[PATCH 0/5] LiveInterval refactoring

Matthias Braun mbraun at apple.com
Tue Sep 3 17:15:48 PDT 2013


On Sep 3, 2013, at 3:11 PM, Jakob Stoklund Olesen <stoklund at 2pi.dk> wrote:

> 
> On Sep 3, 2013, at 2:41 PM, Matthias Braun <mbraun at apple.com> wrote:
>> Attached is a patch for the LiveRange -> LiveInterval::Segment renaming. I’m starting the rebasing of my other patches now.
> 
> Looks good.
> 
>> If we are renaming anyway, I would propose to rename LiveInterval to RegLiveRange (or RegisterLiveRange), which is more logical as it will become a subclass of LiveRange with just a register number and spill weight added.
> 
> Maybe you should wait until you know all the classes you want in the new design?

> 
> Also remember that LiveInterval is currently used for stack slots as well.
Ok, I’ll wait with that :) Yes, well I guess I’ll wait until I know better whether I want subregister liveness for the stackslots as well.

Anyway here is the updated complete patchset.

Greetings,
	Matthias
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-rename-LiveRange-to-LiveInterval-Segment.patch
Type: application/octet-stream
Size: 79720 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130903/c206f137/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-refactor-LiveInterval-introduce-new-LiveRange-class.patch
Type: application/octet-stream
Size: 47533 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130903/c206f137/attachment-0001.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-Pass-LiveQueryResult-by-value.patch
Type: application/octet-stream
Size: 20441 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130903/c206f137/attachment-0002.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-change-MachineVerifier-to-work-on-LiveRange-LiveInte.patch
Type: application/octet-stream
Size: 15818 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130903/c206f137/attachment-0003.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0005-work-on-LiveRange-instead-of-LiveInterval-where-poss.patch
Type: application/octet-stream
Size: 19644 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130903/c206f137/attachment-0004.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0006-Represent-RegUnit-liveness-with-LiveRange-instance.patch
Type: application/octet-stream
Size: 30376 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130903/c206f137/attachment-0005.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0007-print-register-in-LiveInterval-print.patch
Type: application/octet-stream
Size: 6951 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130903/c206f137/attachment-0006.obj>


More information about the llvm-commits mailing list