================
@@ -0,0 +1,81 @@
+// RUN: %clang_analyze_cc1
-analyzer-checker=core,alpha.cplusplus.ReportDanglingPtrDeref \
+// RUN: -analyzer-config cfg-lifetime=true -analyzer-output=text -verify %s
+
+void test_case_one() {
+ int *ptr = nullptr;
+ {
+ int num = 5;
+ ptr = #
+ }
+ *ptr = 6;
+ // expected-warning@-1 {{Use of 'num' after its lifetime ended}}
+ // expected-note@-2 {{Use of 'num' after its lifetime ended}}
+}
+
+void test_case_two() {
+ int *ptr_one = nullptr;
+ int *ptr_two = nullptr;
+ {
+ int n = 1;
+ int m = 2;
+ ptr_one = &n;
+ ptr_two = &m;
+ }
+ *ptr_one = 6;
+ *ptr_two = 7;
+ // expected-warning@-2 {{Use of 'n' after its lifetime ended}}
+ // expected-warning@-2 {{Use of 'm' after its lifetime ended}}
+ // expected-note@-4 {{Use of 'n' after its lifetime ended}}
+ // expected-note@-4 {{Use of 'm' after its lifetime ended}}
----------------
steakhal wrote:
Could you align these - and in other cases where the notes repeat the warning?
Also, I think it's more readable if the warning is directly followed by its
matching note.
```suggestion
// expected-note@-4 {{Use of 'n' after its lifetime ended}}
// expected-note@-4 {{Use of 'm' after its lifetime ended}}
```
https://github.com/llvm/llvm-project/pull/209278
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits