[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