[llvm] Revert "[IndVarsSimplify] sinkUnusedInvariants is skipping instructio… (PR #184436)

Vigneshwar Jayakumar via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 3 13:09:21 PST 2026


VigneshwarJ wrote:

> I don't think this change should be reverted. We might remove the transform entirely if it's not, on average, beneficial to performance, but we should not go back to a version of it that will only try to apply sinking to some subset of instructions due to an implementation bug.

Yes, Thats something I want to do.  I have a patch for removing this entirely, #169250

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


More information about the llvm-commits mailing list