[llvm] 409ff97 - [InstCombine] Fix comment from #88193 (NFC) (#91427)

via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 17:26:40 PDT 2024


Author: AtariDreams
Date: 2024-05-09T09:26:36+09:00
New Revision: 409ff97aac00e5a677c90353b8b413c2bf46e28f

URL: https://github.com/llvm/llvm-project/commit/409ff97aac00e5a677c90353b8b413c2bf46e28f
DIFF: https://github.com/llvm/llvm-project/commit/409ff97aac00e5a677c90353b8b413c2bf46e28f.diff

LOG: [InstCombine] Fix comment from #88193 (NFC) (#91427)

It is inaccurate and needs to be corrected.

Added: 
    

Modified: 
    llvm/lib/Transforms/InstCombine/InstCombineShifts.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Transforms/InstCombine/InstCombineShifts.cpp b/llvm/lib/Transforms/InstCombine/InstCombineShifts.cpp
index 8847de3667130..ba297111d945f 100644
--- a/llvm/lib/Transforms/InstCombine/InstCombineShifts.cpp
+++ b/llvm/lib/Transforms/InstCombine/InstCombineShifts.cpp
@@ -1259,7 +1259,7 @@ Instruction *InstCombinerImpl::visitLShr(BinaryOperator &I) {
       match(Op1, m_SpecificIntAllowPoison(BitWidth - 1)))
     return new ZExtInst(Builder.CreateIsNotNeg(X, "isnotneg"), Ty);
 
-  // ((X << nuw Z) sub nuw Y) >>u exact Z --> X sub nuw (Y >>u exact Z),
+  // ((X << nuw Z) sub nuw Y) >>u exact Z --> X sub nuw (Y >>u exact Z)
   Value *Y;
   if (I.isExact() &&
       match(Op0, m_OneUse(m_NUWSub(m_NUWShl(m_Value(X), m_Specific(Op1)),
@@ -1279,7 +1279,7 @@ Instruction *InstCombinerImpl::visitLShr(BinaryOperator &I) {
     case Instruction::And:
     case Instruction::Or:
     case Instruction::Xor:
-      // And does not work here, and sub is handled separately.
+      // Sub is handled separately.
       return true;
     }
   };


        


More information about the llvm-commits mailing list