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



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

Reply via email to