================
@@ -1,13 +1,65 @@
-// RUN: %clang_analyze_cc1 -analyzer-checker=core.DivideZero -verify %s
+// RUN: %clang_analyze_cc1 -analyzer-checker=core.DivideZero -std=c++20 
-verify %s
----------------
necto wrote:

This change reduces test coverage. This PR demonstrates that the difference 
between C++ standards is not negligible, so I would prefer running the test for 
C++20 and pre-C++20 (whichever it was before, C++17?). Yet the added code is 
incorrect pre-C++20, so perhaps it worth it adding a separate test file to 
avoid complicating conditional compilation logic.

https://github.com/llvm/llvm-project/pull/148988
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to