[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