Scarlet1ssimo added a comment.
In D143867#4121835 <https://reviews.llvm.org/D143867#4121835>, @steakhal wrote:
> Looks sensible to me.
> Do you have a test for triggering the previous assertion?
I do have prepared a test case like:
int testA(void) { return 0/0; }
// RUN: %clang_analyze_cc1 -analyzer-checker=core %s -analyzer-output=sarif
-o -
Notice you must put the buggy code at the very first line to trigger the
assertion.
Do you think it's necessary to put this test case into the test suite?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143867/new/
https://reviews.llvm.org/D143867
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
