Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
While at it, could you switch those UndefValue with PoisonValue if possible? Thank you!
Comment Actions
I'm not sure UndefValue will work because the backend would only check isUndef() to generate correct code...
Comment Actions
oops.. all tests need to updated and include all intrinsic IR tests...
Is there any benefit to switch UndefValue to PoisonValue?
maybe those changed should be in other follow-up patches?
Comment Actions
Thank you!
The benefit is that we want to remove undef altogether, so I'm trying to avoid having more undefs introduced, and if possible have everyone help reducing its usage whenever possible.
Thank you for your help!