[clang] [analyzer] EvalBinOpLL should return Unknown less often (PR #114222)

Balazs Benics via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 30 06:31:24 PDT 2024


steakhal wrote:

> Makes sense for me. It would be nice if we had a minimal reproducer for a regression test instead of maintaining a large-ish test.

I wish I could add a LIT test instead.
Out of the get min/max val, or getKnownValue APIs, only the latter is used slightly more broadly, where the callsites can't assume it should return a valid result. Still, these APIs are barely used.
In my case I have additional context in my downstream checker, where I can make this assumption and I was surprised to see crashes. So I had no other options but to craft a unittest for this.

https://github.com/llvm/llvm-project/pull/114222


More information about the cfe-commits mailing list