[clang] [Clang][diagnostics] Improve the diagnostics for chained comparisons (PR #129285)

Amr Hesham via cfe-commits cfe-commits at lists.llvm.org
Sat Mar 1 03:02:16 PST 2025


================
@@ -85,7 +85,7 @@ void f(int x, int y, int z) {
   if ((a<y) != -1) {}// expected-warning {{comparison of constant -1 with boolean expression is always true}}
 
   if ((a<y) == z) {} // no warning
-  if (a>y<z)      {} // expected-error {{comparisons like 'X<=Y<=Z' don't have their mathematical meaning}}
+  if (a>y<z)      {} // expected-error {{comparisons like 'a > y < z' don't have their mathematical meaning}}
----------------
AmrDeveloper wrote:

Yes, I got your point also it will not be great with large expressions :D, I am thinking should we change the message to be different or just the old one with actual operators 🤔

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


More information about the cfe-commits mailing list