[PATCH] D110411: [LiveIntervals] Update subranges in processTiedPairs

Jay Foad via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 24 06:13:00 PDT 2021


foad added reviewers: qcolombet, MatzeB, arsenm, stoklund, bjope, kparzysz.
foad added inline comments.
Herald added a subscriber: wdng.


================
Comment at: llvm/lib/CodeGen/TwoAddressInstructionPass.cpp:1506-1509
+      LiveInterval &LI = LIS->getInterval(RegB);
+      Shrink(LI, LaneBitmask::getAll());
+      for (auto &S : LI.subranges())
+        Shrink(S, S.LaneMask);
----------------
I did wonder if there should be a helper function in LiveIntervals for modifying a range and all relevant subranges, but it's not clear to me exactly what the API would be for that.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D110411



More information about the llvm-commits mailing list