[Mlir-commits] [clang] [compiler-rt] [libc] [libclc] [libcxxabi] [lld] [lldb] [llvm] [mlir] Add clarifying parenthesis around non-trivial conditions in ternary expressions. (PR #90391)

Jessica Clarke llvmlistbot at llvm.org
Sat May 4 12:30:23 PDT 2024


================
@@ -3802,7 +3802,7 @@ bool X86AsmParser::validateInstruction(MCInst &Inst, const OperandVector &Ops) {
     //    VFMULCPHZrr   Dest, Src1, Src2
     //    VFMULCPHZrrk  Dest, Dest, Mask, Src1, Src2
     //    VFMULCPHZrrkz Dest, Mask, Src1, Src2
-    for (unsigned i = TSFlags & X86II::EVEX_K ? 2 : 1;
----------------
jrtc27 wrote:

The second level seems like a bug, it's not consistent with how `type identifier = expr ? expr : expr;` is handled in a block.

https://github.com/llvm/llvm-project/pull/90391


More information about the Mlir-commits mailing list