[clang] [Analyzer][CFG] Correctly handle rebuilt default arg and default init expression (PR #91879)

Balazs Benics via cfe-commits cfe-commits at lists.llvm.org
Tue May 14 02:55:53 PDT 2024


================
@@ -1114,17 +1114,16 @@ void fCXX11MemberInitTest1() {
   CXX11MemberInitTest1();
 }
 
+#ifdef PEDANTIC
----------------
steakhal wrote:

I don't think you need to guard this section of code if you were using `// pedantic-note {{...}}` and `// pedantic-warning {{...}}` in the guarded checks.

I also think that the `// TODO: we'd expect the warning: {{2 uninitializeds field}}` comment refers to this new appearing report, so you should just drop that comment from the `fCXX11MemberInitTest2` function.

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


More information about the cfe-commits mailing list