[llvm] 0852313 - [NFC] combineX86ShufflesRecursively(): actually address nits for previous patch

Roman Lebedev via llvm-commits llvm-commits at lists.llvm.org
Sun Sep 19 07:29:50 PDT 2021


Author: Roman Lebedev
Date: 2021-09-19T17:29:18+03:00
New Revision: 0852313e47836152b00fb8b8fd62a7e12bf92abd

URL: https://github.com/llvm/llvm-project/commit/0852313e47836152b00fb8b8fd62a7e12bf92abd
DIFF: https://github.com/llvm/llvm-project/commit/0852313e47836152b00fb8b8fd62a7e12bf92abd.diff

LOG: [NFC] combineX86ShufflesRecursively(): actually address nits for previous patch

Added: 
    

Modified: 
    llvm/lib/Target/X86/X86ISelLowering.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/X86/X86ISelLowering.cpp b/llvm/lib/Target/X86/X86ISelLowering.cpp
index 92dfbc17d73a..910b5a05ddf3 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -37911,14 +37911,14 @@ static SDValue combineX86ShufflesRecursively(
     SDValue &Op = I.value();
 
     // What range of shuffle mask element values results in picking from Op?
-    int lo = OpIdx * Mask.size();
-    int hi = lo + Mask.size();
+    int Lo = OpIdx * Mask.size();
+    int Hi = Lo + Mask.size();
 
     // Which elements of Op do we demand, given the mask's granularity?
     APInt OpDemandedElts(Mask.size(), 0);
     for (int MaskElt : Mask) {
-      if (isInRange(MaskElt, lo, hi)) { // Picks from Op?
-        int OpEltIdx = MaskElt - lo;
+      if (isInRange(MaskElt, Lo, Hi)) { // Picks from Op?
+        int OpEltIdx = MaskElt - Lo;
         OpDemandedElts.setBit(OpEltIdx);
       }
     }


        


More information about the llvm-commits mailing list