[llvm] 6eaea22 - [X86] combineTargetShuffle - break if-else chain. NFC.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sat Jun 11 01:29:09 PDT 2022


Author: Simon Pilgrim
Date: 2022-06-11T09:16:39+01:00
New Revision: 6eaea225c75b59fe3855d10dd458e7ca93417114

URL: https://github.com/llvm/llvm-project/commit/6eaea225c75b59fe3855d10dd458e7ca93417114
DIFF: https://github.com/llvm/llvm-project/commit/6eaea225c75b59fe3855d10dd458e7ca93417114.diff

LOG: [X86] combineTargetShuffle - break if-else chain. NFC.

(style) Both cases always continue.

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 9381da436992..94ed88742348 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -39885,7 +39885,9 @@ static SDValue combineTargetShuffle(SDValue N, SelectionDAG &DAG,
         if ((InsertPSMask & (1u << i)) || (i == (int)DstIdx)) {
           // No change if element is already zero or the inserted element.
           continue;
-        } else if (KnownUndef0[i] || KnownZero0[i]) {
+        }
+
+        if (KnownUndef0[i] || KnownZero0[i]) {
           // If the target mask is undef/zero then we must zero the element.
           InsertPSMask |= (1u << i);
           Updated = true;


        


More information about the llvm-commits mailing list