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] Add tir::builtin::undef #12266

Merged
merged 9 commits into from
Aug 6, 2022
Merged

Conversation

Lunderberg
Copy link
Contributor

This PR introduces tir::builtin::undef, which is used to represent values that are an arbitrary value of a known datatype, along with a transformation to remove instances of tir::builtin::undef.

This PR is part of the handling of padded buffer layout transformations (tracking issue, rfc).

@Lunderberg
Copy link
Contributor Author

This is currently a draft PR, as the unit tests depend on functionality introduced in #12264.

@Lunderberg Lunderberg marked this pull request as ready for review August 2, 2022 14:16
src/tir/transforms/remove_store_undef.cc Outdated Show resolved Hide resolved
src/tir/transforms/remove_store_undef.cc Show resolved Hide resolved
src/tir/transforms/remove_store_undef.cc Show resolved Hide resolved
@Lunderberg
Copy link
Contributor Author

Rebased onto main to resolve conflict with #12267.

@wrongtest-intellif wrongtest-intellif merged commit c4aab62 into apache:main Aug 6, 2022
@Lunderberg Lunderberg deleted the tir_undef branch August 22, 2022 15:33
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
* [UnitTest] RemoveStoreUndef, simplest behavior

* [RemoveStoreUndef] First implementation

* [UnitTest] RemoveStoreUndef, stores that depend through LetStmt

* [UnitTest] RemoveStoreUndef, LetStmt handling, error on illegal usage

* [RemoveStoreUndef] Added error checking for illegal T.undef() usage

* Fix lint error

* Use const ref for list of stores to remove

* Verify that removed expression has no other side effects

* Fix lint error
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.

2 participants