hfinkel added inline comments. ================ Comment at: lib/Target/AMDGPU/SIInstrInfo.cpp:360 +bool SIInstrInfo::doMemOpsHaveSameBasePtr(const MachineInstr &MI1, + unsigned BaseReg1, ---------------- Can you make *this* the default implementation? https://reviews.llvm.org/D37698