<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Dec 10, 2014 at 10:31 AM, Matthias Braun <span dir="ltr"><<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><br><div><span class=""><blockquote type="cite"><div>On Dec 10, 2014, at 9:28 AM, David Blaikie <<a href="mailto:dblaikie@gmail.com" target="_blank">dblaikie@gmail.com</a>> wrote:</div><br><div><div dir="ltr" style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px"><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Dec 9, 2014 at 5:32 PM, Matthias Braun<span> </span><span dir="ltr"><<a href="mailto:matze@braunis.de" target="_blank">matze@braunis.de</a>></span><span> </span>wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">Ok,<br><br>I just commited the changes to svn trunk. Thanks everyone, esp. Renato and Quentin, for the detailed reviews!<br><br>SubRegisterLiveness is available as a subtarget switch now, but not enabled by any of the targets yet, as most of them need some additional fixing/testcase changes which I'll address in subsequent commits.<br><br>For the record: I did fix many but not all issues mentioned in the reviews. The following is still missing:<br><br>- Manually creating a single linked list in the LiveInterval class. The alternatives to this are:<br>   <span> </span>a) Using the double linked list provided by llvm.<br>   <span> </span>b) Turning the manual single linked list into a full featued ADT in llvm.<br></blockquote><div><br>Why not std::forward_list ?<br></div></div></div></div></div></blockquote><div><br></div></span><div>As far as I know the STL list containers are non-intrusive which is probably even more wasteful as you need an extra object (with memory allocator alignment padding) for each list element.</div></div></div></blockquote><div><br>They aren't intrusive, but that doesn't mean you need an extra object - it's still just a single allocation per node, containing the element type and a pointer.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><span class="HOEnZb"><font color="#888888"><div><br></div><div>- Matthias</div></font></span></div></div></blockquote></div><br></div></div>