[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