[PATCH] D138037: [analyzer] Remove unjustified assertion from EQClass::simplify

Vince Bridgers via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 17 02:39:19 PST 2022


vabridgers added a comment.

Hi @steakhal, thanks for the suggested change.
How we can help move this forward? From what I'm comprehending from the notes, perhaps we could try running this change through our internal systems level test and fuzzer. Unfortunately, I'd not be able to say more than "trust me, we saw no problems" if we see no problems. But if I do find additional cases I can make simplified reproducers and we can work on addressing those.
Best!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D138037



More information about the cfe-commits mailing list