[llvm] [InstCombine] Remove Store when its Ptr is removable (PR #79565)
Matt Arsenault via llvm-commits
llvm-commits at lists.llvm.org
Thu May 9 12:08:45 PDT 2024
arsenm wrote:
> Possibly the error message just needs to be improved to make it clear that this is not necessary a problem?
I think the message definitely should be improved. Downgrade the message to warning?
> > Put up a WIP https://github.com/arsenm/llvm-project/tree/issue68120-worklist-issue
>
> This will miscompile cases if there are multiple uses along the stripped chain, I think?
I started guarding against these, but so far I haven't had much luck avoiding the second iteration in the mixed multiple use cases.
https://github.com/llvm/llvm-project/pull/79565
More information about the llvm-commits
mailing list