[PATCH] D102240: [analyzer][solver] Prevent use of a null state

Balázs Benics via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue May 11 09:25:21 PDT 2021


steakhal added a comment.

Dam, we will be always haunted by these.
Looks good btw.
The test looks somewhat artificial - like a raw `creduce` result.

Do you think It worth the effort to make it look like it was written by a human?
I also somewhy prefer function parameters to globals for this specific use-case.

This seems to depend on the `-analyzer-config eagerly-assume=true`. We should probably add this option explicitly, just in case.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D102240/new/

https://reviews.llvm.org/D102240



More information about the cfe-commits mailing list