steakhal added a comment.

LGTM

In D85424#2199471 <https://reviews.llvm.org/D85424#2199471>, 
@baloghadamsoftware wrote:

> Unfortunately, I could not create test for it. It is extremely rare that the 
> //Analyzer// creates an `UndefinedVal`.

You can work around the issue by creating a unit-test instead.
In that test, you could create a custom checker, which returns the required 
`Undefined` SVal of your desired type.

We need a test to approve this change.


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