[PATCH] D156479: GlobalISel: Pass MachineIRBuilder to applyMappingImpl

Pierre van Houtryve via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 01:47:09 PDT 2023


Pierre-vh added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPURegisterBankInfo.h:56
 
-  bool executeInWaterfallLoop(
-    MachineIRBuilder &B,
-    iterator_range<MachineBasicBlock::iterator> Range,
-    SmallSet<Register, 4> &SGPROperandRegs,
-    MachineRegisterInfo &MRI) const;
+  bool executeInWaterfallLoop(MachineIRBuilder &B,
+                              iterator_range<MachineBasicBlock::iterator> Range,
----------------
Could we make the builder a member of the class? 


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

https://reviews.llvm.org/D156479



More information about the llvm-commits mailing list