[llvm] [DAGCombiner][X86] Correctly clean up high bits in `combinei64TruncSrlAdd` (PR #128353)

Phoebe Wang via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 22 05:34:49 PST 2025


phoebewang wrote:

> A counterexample for original implementation: https://alive2.llvm.org/ce/z/YowPZY
> We should keep low `64 - shamt` bits instead of `shamt - 32`.
> Proof: https://alive2.llvm.org/ce/z/z_jdHD

The alive2 just proves the problem in calculation of NewAddConstVal, but is the problem in #128309 actually caused by zext vs. anyext?

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


More information about the llvm-commits mailing list