[llvm] [InstSimplify] Remove most of undef handling logic (PR #119884)

Nuno Lopes via llvm-commits llvm-commits at lists.llvm.org
Tue Dec 17 02:13:59 PST 2024


nunoplopes wrote:

This looks great, thank you!
As we move away from undef, most of the remaining uses are related with bitfields, so arithmetic on those isn't important (only bitwise operations).

That said, I wouldn't remove every single undef transformation because `freeze poison` with a single use can be optimized like undef in most cases. Right now, instcombine eagerly transformes those into 0, but that's suboptimal. We need to teach instcombine how to optimize `freeze poison` to eventually remove that `freeze poison -> 0` transformation.

https://github.com/llvm/llvm-project/pull/119884


More information about the llvm-commits mailing list