[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