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

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

Reply via email to