Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[TIR] Handle DeclBuffer in IRSubstitute #15054

Merged
merged 1 commit into from
Jun 8, 2023

Conversation

Lunderberg
Copy link
Contributor

Previously, occurrences of DeclBuffer were not handled by IRSubstitute, and the buffer object was left unhandled. This commit updates IRSubstitute to modify the buffer object in DeclBuffer, similar to how it is already modified in BufferLoad and BufferStore instances.

This is a subset of changes being split out from #14778 into independent portions.

Previously, occurrences of `DeclBuffer` were not handled by
`IRSubstitute`, and the buffer object was left unhandled.  This commit
updates `IRSubstitute` to modify the buffer object in `DeclBuffer`,
similar to how it is already modified in `BufferLoad` and
`BufferStore` instances.

This is a subset of changes being split out from
apache#14778 into independent portions.
@tvm-bot
Copy link
Collaborator

tvm-bot commented Jun 7, 2023

Thanks for contributing to TVM! Please refer to the contributing guidelines https://tvm.apache.org/docs/contribute/ for useful information and tips. Please request code reviews from Reviewers by @-ing them in a comment.

Generated by tvm-bot

@Hzfengsy Hzfengsy merged commit fa4d73a into apache:main Jun 8, 2023
@Lunderberg Lunderberg deleted the declbuffer_ir_substitute branch June 8, 2023 11:27
junrushao pushed a commit to junrushao/tvm that referenced this pull request Jun 22, 2023
Previously, occurrences of `DeclBuffer` were not handled by
`IRSubstitute`, and the buffer object was left unhandled.  This commit
updates `IRSubstitute` to modify the buffer object in `DeclBuffer`,
similar to how it is already modified in `BufferLoad` and
`BufferStore` instances.

This is a subset of changes being split out from
apache#14778 into independent portions.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants