[PATCH] D37698: Add target callback doMemOpsHaveSameBasePtr

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 12 09:21:14 PDT 2017


rampitec added a comment.

In https://reviews.llvm.org/D37698#868147, @vpykhtin wrote:

> SIInstrInfo::doMemOpsHaveSameBasePtr looks generic enough to be default indeed but should we commit this first and then make doMemOpsHaveSameBasePtr  default so we could rollback to this one in case of severe regressions? LGTM by the way.


Good point. I will create a separate patch to switch the default implementation.


https://reviews.llvm.org/D37698





More information about the llvm-commits mailing list