balazske added inline comments.
================
Comment at: clang/test/Analysis/return-ptr-range.cpp:1
-// RUN: %clang_analyze_cc1 -analyzer-checker=alpha.security.ReturnPtrRange
-verify %s
+// RUN1: %clang_analyze_cc1 -analyzer-checker=alpha.security.ReturnPtrRange
-verify %s
+// RUN: %clang_analyze_cc1 -analyzer-checker=alpha.security.ReturnPtrRange
-analyzer-output text -verify=notes %s
----------------
Szelethus wrote:
> steakhal wrote:
> > Is `RUN1` intentional? If so, what does it do?
> We could just delete it. I guess that was the intent, to make this RUN line
> non-functional.
I wanted to make two runs, one for warnings only and one for notes only. But
could not find out how to disable the warning messages and show only notes.
Because the same warnings appear anyway it should be enough to use only one run
with text output and remove the custom prefix.
================
Comment at: clang/test/Analysis/return-ptr-range.cpp:19-20
+ ptr = arr1 + x; // notes-note{{Value assigned to 'ptr'}}
+ if (x != 20) // notes-note{{Assuming 'x' is equal to 20}}
+ // notes-note@-1{{Taking false branch}}
+ return arr1; // no-warning
----------------
Szelethus wrote:
> steakhal wrote:
> > This is probably more of a taste.
> > I would prefer fewer indentations.
> > The same applies everywhere.
> I disagree, and prefer it as it is written now.
The current style will be used, the comments are better grouped.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D107051/new/
https://reviews.llvm.org/D107051
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
