[PATCH] D15007: [analyzer] Improve modelling of nullptr_t in the analyzer. Fix PR25414.

Gábor Horváth via cfe-commits cfe-commits at lists.llvm.org
Thu Nov 26 02:25:37 PST 2015


xazax.hun added a comment.

In http://reviews.llvm.org/D15007#297183, @NoQ wrote:

> There's a little problem with the `shouldNotCrash()` test: the first warning on `invokeF()` on line 107 generates a sink, and the rest of the function never gets executed. It's probably a good idea to split it into three separate tests. Then it'd also warn on line 110.


Thank you for pointing this out! I originally used separate test files and did not notice this problem after merging the into the regression test suite.


http://reviews.llvm.org/D15007





More information about the cfe-commits mailing list