[llvm] d9f6077 - [X86] Fix ambiguous operator ordering to stop -Wparentheses warning. NFCI.

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 26 02:41:02 PST 2023


Author: Simon Pilgrim
Date: 2023-02-26T10:40:43Z
New Revision: d9f6077db05c059d6f24949e18ff13f94dd4ba56

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

LOG: [X86] Fix ambiguous operator ordering to stop -Wparentheses warning. NFCI.

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 53a276e7b8d07..6188d1d55c3aa 100644
--- a/llvm/lib/Target/X86/X86ISelLowering.cpp
+++ b/llvm/lib/Target/X86/X86ISelLowering.cpp
@@ -16960,8 +16960,7 @@ static SDValue splitAndLowerShuffle(const SDLoc &DL, MVT VT, SDValue V1,
     // Because the lowering happens after all combining takes place, we need to
     // manually combine these blend masks as much as possible so that we create
     // a minimal number of high-level vector shuffle nodes.
-
-    assert(!SimpleOnly || (!UseHiV1 && !UseHiV2) && "Shuffle won't be simple");
+    assert((!SimpleOnly || (!UseHiV1 && !UseHiV2)) && "Shuffle isn't simple");
 
     // First try just blending the halves of V1 or V2.
     if (!UseLoV1 && !UseHiV1 && !UseLoV2 && !UseHiV2)


        


More information about the llvm-commits mailing list