<div dir="ltr">Hello Arnaud,<div><br></div><div>This is a good idea, but the implementation in this patch will make removal of conservatively allocatable nodes more expensive: from O(N) to O(RN^2), where R is the number of registers and N is the number of nodes. We can get that back down to O(RN + NlogN) by creating a VectorMetadata class that contains the expected/worst-case cost, and using a set (or better yet, a priority_queue) to hold the conservatively allocatable nodes. This is the same line of reasoning that was applied (very successfully) to MatrixMetadata to bring down the cost for the conservative-allocatability test itself.</div><div><br></div><div>If you're happy to rewrite your patch along those lines please go for it, otherwise I can find some time to do it in the next few days. It is not a big change at all, though it helps that I'm familiar with the infrastructure.</div><div><br></div><div>Cheers,</div><div>Lang.</div><div> </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Oct 22, 2014 at 6:43 AM, Arnaud A. de Grandmaison <span dir="ltr"><<a href="mailto:arnaud.degrandmaison@arm.com" target="_blank">arnaud.degrandmaison@arm.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-GB" link="blue" vlink="purple"><div><p class="MsoNormal">Hi Lang,<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Here is the last patch in the PBQP improvement serie. Could you please have a look at it ?<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Nodes in ConservativelyAllocatableNodes were processed in the order they<u></u><u></u></p><p class="MsoNormal">have been added in NodeSet; however, it is preferable to process nodes in<u></u><u></u></p><p class="MsoNormal">an order based on their cost, to ensure we keep the total cost under<u></u><u></u></p><p class="MsoNormal">control and provide a reasonable solution.<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal"><span lang="FR">Cheers,<u></u><u></u></span></p><p class="MsoNormal"><span lang="FR">--<u></u><u></u></span></p><p class="MsoNormal"><span lang="FR">Arnaud A. de Grandmaison<u></u><u></u></span></p><p class="MsoNormal"><span lang="FR"><u></u> <u></u></span></p></div></div></blockquote></div><br></div>