[PATCH] D134951: [CodeGen] Propagate vreg flags to the regalloc spiller interface

Quentin Colombet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 5 10:03:41 PDT 2022


qcolombet added a comment.

Hi,

The `LiveIntervals` and `LiveRangeEdit` changes make sense to me (though in practice they are NFC.)

However, I'm not a fan of the MachineRegisterInfo change. Since it sounds like you need that field only for fast regalloc, could you do something specifically in that pass/with the related target hooks in that pass?

Cheers,
-Quentin


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134951



More information about the llvm-commits mailing list