[PATCH] D85424: [Analyzer] Crash fix for alpha.cplusplus.IteratorRange
Balogh, Ádám via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sun Aug 9 23:04:12 PDT 2020
baloghadamsoftware added a comment.
In D85424#2203329 <https://reviews.llvm.org/D85424#2203329>, @NoQ wrote:
> Tests? (edit: ok, i see, you have trouble reducing the test. yes, we absolutely need a test, please finish with creduce/delta.)
I will not commit it without tests, even if someone accepts it. We are working on it using //creduce//.
> There should be no `UndefinedVal`s in function arguments unless you disable `core` checkers (which isn't supported).
Hmm, then it means that instead of this patch we should investigate why the argument is `UndefinedVal` and fix the issue there. All the `core` checkers are enabled, of course.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85424/new/
https://reviews.llvm.org/D85424
More information about the cfe-commits
mailing list