[llvm] [InstCombine] Remove Store when its Ptr is removable (PR #79565)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Wed May 8 11:13:03 PDT 2024


arsenm wrote:

> This issue is Won't Fix as far as I'm concerned. Reaching a fix-point in _all_ cases is an explicit non-goal -- the verification exists to catch bugs like failing to add things back to the worklist.

If this is the case, I think the current behavior of error-by-default isn't really reasonable. You should be able to run instcombine standalone without triggering errors like this

I think this should be fixed, but I feel like it should be nowhere near this complicated to address 

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


More information about the llvm-commits mailing list