NoQ added inline comments.
================ Comment at: clang/test/Analysis/bstring.c:310 - clang_analyzer_eval(p == &dst[4]); // expected-warning{{TRUE}} + clang_analyzer_eval(p == &dst[4]); // no-warning (above is fatal) } ---------------- Hmm, given that your change suppresses some existing tests, maybe disable your checker on this file and copy the updated tests to a new file that would have the checker enabled? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D120489/new/ https://reviews.llvm.org/D120489 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits