NoQ added inline comments.

================
Comment at: clang/test/Analysis/return-value-guaranteed.cpp:20
+struct Foo { int Field; };
+bool error();
+bool problem();
----------------
NoQ wrote:
> `core.builtin.NoReturnFunctions` reacts on this function. You can easily see 
> it in the explodedgraph dump, as the last sink node in `test_calls::parseFoo` 
> is tagged with that checker. You'll have to pick a different name for testing 
> purposes.
(yes, this is why your tests aren't working)
(see also D63965)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63915/new/

https://reviews.llvm.org/D63915



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to