[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