<div dir="ltr">This looks great. Thanks Arnaud! Please go ahead and commit.<div><br></div><div>- Lang.</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Feb 3, 2015 at 2:34 PM, 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">Based on the feedback from Jonas, and my own experience,  here is a patch which adds some more information to the graph dumps : previously, we were only dumping the node ids, and this made the graph and dot representation not very user friendly nor very useable for debug. With this patch, the node id is dumped together with the register it corresponds to, as well as its register class.<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">I renamed dumpToStrem to dump, and moved the dump & printDot methods up to PBQPRAGraph, because those methods now make use of the metadata information.<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Does this sound OK to you ?<u></u><u></u></p><p class="MsoNormal"><u></u> <u></u></p><p class="MsoNormal">Cheers,<u></u><u></u></p><p class="MsoNormal"><span>--<u></u><u></u></span></p><p class="MsoNormal"><span>Arnaud<u></u><u></u></span></p></div></div></blockquote></div><br></div>