[PATCH] D67234: [MergedLoadStoreMotion] Sink stores if they have common GEP
    Denis Bakhvalov via Phabricator via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Wed Sep 18 11:49:38 PDT 2019
    
    
  
dendibakh added a comment.
Thanks for getting back to this.
GVNSink and SimplifyCFG only sink common tails of 2 BBs. I.e. they will not cherry-pick any individual stores from the middle of a BB.
That was the motivation for a little improvement. According to my testing it doesn't have measurable performance impact. I saw fluctuations within 1% which I think is caused by code placement.
I will try to find someone to review.
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D67234/new/
https://reviews.llvm.org/D67234
    
    
More information about the llvm-commits
mailing list