[Mlir-commits] [llvm] [mlir] eliminating g++ warnings (PR #105520)

[Frank Schlimbach]

================
@@ -893,7 +893,7 @@ FlatLinearValueConstraints::FlatLinearValueConstraints(IntegerSet set,
                             set.getNumDims(), set.getNumSymbols(),
                             /*numLocals=*/0) {
   assert(operands.empty() ||
-         set.getNumInputs() == operands.size() && "operand count mismatch");
+         (set.getNumInputs() == operands.size() && "operand count mismatch"));
----------------
fschlimb wrote:

Yes, the point here is that the braces should not change the semantics. gcc "suggests" braces to clarify. I tried to add braces without changing semantics. What you are proposing looks more correct, but is not the same as without.

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