[PATCH] D114831: [NFC][regalloc] Move ExtraRegInfo and related to LiveRangeStageManager

[Mircea Trofin via Phabricator via llvm-commits]

mtrofin added a comment.

In D114831#3168425 <https://reviews.llvm.org/D114831#3168425>, @MatzeB wrote:

>> Option 1 would mean extracting RAGreedy in that header, and that looks like a larger factoring in a .h, in comparison to the ExtraRegInfo stuff.
>
> Yes, but it's just moving the declaration around without changing things otherwise. Which seems like the "smaller" factoring to me. (at the price of higher entanglement as I stated)

Ack.

I wonder if we can decouple this design problem from this patch. Should there end up being a strong preference for going to a `RegAllocGreedy.h`, that should be easy to do at any later point, wdyt?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114831