[PATCH] D96898: [NFC][Regalloc] Share the VirtRegAuxInfo object with LiveRangeEdit

Wei Mi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 18 15:21:56 PST 2021


wmi added a comment.

It makes sense to share the same VRAI during the RA course for a particular type of allocator.



================
Comment at: llvm/lib/CodeGen/RegAllocPBQP.cpp:806-809
+  // FIXME: we create DefaultVRAI here to match existing behavior pre-passing
+  // the VRAI through the spiller to the live range editor. However, it probably
+  // makes more sense to pass the PBQP VRAI. The existing behavior had
+  // LiveRangeEdit make its own VirtRegAuxInfo object.
----------------
Did you see any problem by using PBQP VRAI?


================
Comment at: llvm/lib/CodeGen/SplitKit.h:456-457
+  SplitEditor(SplitAnalysis &SA, AAResults &AA, LiveIntervals &LIS,
+              VirtRegMap &VRM, MachineDominatorTree &MDT,
+              MachineBlockFrequencyInfo &MBFI, VirtRegAuxInfo &VRAI);
 
----------------
Unnecessary to pass LIS, VRM and MBFI since they are all included in VRAI?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D96898



More information about the llvm-commits mailing list