[PATCH 0/5] LiveInterval refactoring

[Jakob Stoklund Olesen]

On Aug 29, 2013, at 6:07 PM, Matthias Braun <matze at braunis.de> wrote:

> 
> This patchset is a step towards having subregister liveness in the
> LiveIntervall class. It factors out some functionality of the LiveInterval
> class into a new LiveRanges class to enable reusage. As a proof of concept
> the final patch rewrite the Register Unit liveness tracking to use this new
> class and thereby avoid having unnecessary register number and spill weight
> fields there.
> 
> Matthias Braun (5):
>  refactor LiveInterval: introduce new LiveRanges class
>  Pass LiveQueryResult by value
>  change MachineVerifier to work on LiveRanges + LiveInterval
>  work on LiveRanges instead of LiveInterval where possible
>  Represent RegUnit liveness with LiveRanges instances

Hi Matthias,

These patches all LGTM, except for the LiveRanges class name.

The struct LiveRange is badly misnamed, and it should really just be an implementation detail of LiveInterval. Could you start by moving it into that class as struct LiveInterval::Segment?

Then you can use the proper name LiveRange for your new 'LiveRanges' class.

Thanks,
/jakob