[clang] [Clang] omit parentheses in fold expressions with a single expansion (PR #110761)

via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 3 13:11:27 PDT 2024


================
@@ -36,7 +36,7 @@ using r1i2 = r1<char>; // expected-error {{constraints not satisfied for class t
 template<typename... Ts> requires
 false_v<requires (Ts... ts) {requires ((sizeof(ts) == 2) && ...);}>
 // expected-note at -1 {{because 'false_v<requires (short ts, unsigned short ts) { requires (sizeof (ts) == 2) && (sizeof (ts) == 2); }>'}}
-// expected-note at -2 {{because 'false_v<requires (short ts) { requires (sizeof (ts) == 2); }>' evaluated to false}}
+// expected-note at -2 {{because 'false_v<requires (short ts) { requires sizeof (ts) == 2; }>' evaluated to false}}
----------------
cor3ntin wrote:

Yeah... I don't think we should modify the AST to avoid a warning. Can we instead tweak when the warning is emitted?

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


More information about the cfe-commits mailing list