[PATCH] D65496: AMDGPU/SILoadStoreOptimizer: Add helper functions for working with CombineInfo

[Tom Stellard via Phabricator via llvm-commits]

tstellar marked an inline comment as done.
tstellar added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp:533
 
-bool SILoadStoreOptimizer::findMatchingInst(CombineInfo &CI) {
-  MachineBasicBlock *MBB = CI.I->getParent();
-  MachineBasicBlock::iterator E = MBB->end();
-  MachineBasicBlock::iterator MBBI = CI.I;
+void SILoadStoreOptimizer::setMI(CombineInfo &CI,
+                                 MachineBasicBlock::iterator I) const {
----------------
vpykhtin wrote:
> This can be a member of CombineInfo, or even a constructor of it.
The reason I didn't do it this way originally was because I would also need to pass in TTI and STI to the function or to the constructor for CombineInfo, but I don't mind turning it into a member function.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D65496/new/

https://reviews.llvm.org/D65496