Get rid of `undef` and use `poison`

Open bcardosolopes opened this issue 6 months ago • 3 comments

We have been getting PR warnings such as https://github.com/llvm/clangir/pull/1701#issuecomment-2992625611, we should move out existing uses of undef as much as possible.

Originally posted by @bcardosolopes in https://github.com/llvm/clangir/pull/1701#discussion_r2162559191

Jun 23 '25 21:06 bcardosolopes

hi @bcardosolopes i am interested in contributing to this issue. May i know about any prerequisites that might be required before tackling this issue. I am going through the repo currently for the same

Jun 28 '25 17:06 saeitoshi-10

@saeitoshi-10 great! I'd find any places where ClangIR uses undef when lowering to LLVM, check whether traditional codegen generated poison instead, and based on those cases fix what's needed.

Jul 14 '25 19:07 bcardosolopes

@saeitoshi-10 are you still interested in this issue? If no, I can pick it up.

Jul 19 '25 10:07 bruteforceboy