[PATCH] D105545: [MergedLoadStoreMotion] Conditional store elimination

Momchil Velikov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 2 07:52:27 PDT 2021


chill added a comment.

Hello,

Thanks for the comments and the suggestions.
The main objective of this patch is to simplify the CFG, but, indeed, the patch itself does not
make the CFG simpler.
As it turns out, there's already a very similar transformation done in SimplifyCFG, so I have opted
to adding extra correctness checks (respectively, trigger in more cases) *there*.

I'm abandoning this patch in favour of  https://reviews.llvm.org/D107281


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

https://reviews.llvm.org/D105545



More information about the llvm-commits mailing list