[llvm] [IndVars] Teach widenLoopCompare to use sext if narrow IV is positive and other operand is already sext. (PR #142703)
Craig Topper via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 9 09:13:20 PDT 2025
================
@@ -1630,6 +1630,12 @@ bool WidenIV::widenLoopCompare(WidenIV::NarrowIVDefUse DU) {
// Widen the other operand of the compare, if necessary.
if (CastWidth < IVWidth) {
+ // If the narrow IV is always postive and the other operand is sext, widen
----------------
topperc wrote:
I wrote "always postive" here because that's what is used 2 other times in the comment block earlier in this function. Should that be updated to use non-negative too?
https://github.com/llvm/llvm-project/pull/142703
More information about the llvm-commits
mailing list