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

[Wenlei He via Phabricator via llvm-commits]

wenlei added a comment.

In D114831#3168435 <https://reviews.llvm.org/D114831#3168435>, @mtrofin wrote:

> 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?

I agree with Matthias that the advisor now is really tied to Greedy and #1 makes sense. Handle this in a later patch is also reasonable, but leaving it in current state for too long probably isn't ideal. If you plan to handle it later, perhaps stick in a TODO comment?

Other than that, the refactoring seems straightforward.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D114831