[PATCH] D146859: RegAllocGreedy: Fix detection of lanes read by a bundle

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 13 03:16:27 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/lib/CodeGen/RegAllocGreedy.cpp:1265
+  SmallVector<std::pair<MachineInstr *, unsigned>, 8> Ops;
+  (void)AnalyzeVirtRegInBundle(const_cast<MachineInstr &>(FirstMI), Reg, &Ops);
 
----------------
qcolombet wrote:
> Out-of-curiosity why does `AnalyzeVirtRegInBundle` requires non-const instructions?
It shouldn't. The one possible reason would be if you want a non-const pointer in the output array 


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

https://reviews.llvm.org/D146859



More information about the llvm-commits mailing list