[PATCH] D37755: Change default implementation of doMemOpsHaveSameBasePtr
Stanislav Mekhanoshin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 13 09:06:25 PDT 2017
rampitec added a comment.
In https://reviews.llvm.org/D37755#869341, @hfinkel wrote:
> Please see my comments in https://reviews.llvm.org/D37698. If the primary purpose of this on AArch64 is to enable the AArch64LoadStoreOptimizer, I'm not sure that this makes sense.
Right. After I did it and run testing I have realized only to targets in the tree use these mutations, and AArch64 needs exactly current implementation. So I would keep it AMD specific untill there are other users.
https://reviews.llvm.org/D37755
More information about the llvm-commits
mailing list