[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