[llvm] [X86] Remove def32 and drop redundant zero extensions after `isel` (PR #225462)

Craig Topper via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 25 22:18:14 PDT 2026


topperc wrote:

> > Is #218382 also fixed by #223750. I see the description also mentioned shrinkAndImmediate.
> 
> Yes, it is the same all-ones path in `shrinkAndImmediate` with a truncate operand, so #223750 covers #218382 as well. I'll rebase on top of it, drop the guard and the `SDNode` `isDef32` again since the `zext` no longer depends on its operand, and keep the reduced case from #218382 here as a regression test later.

Can we close #218382 since it's fixed?

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


More information about the llvm-commits mailing list