
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Hal, Eric,<div><br></div><div>thanks for your feedback! I attached an updated version.</div><div><br></div><div><blockquote type="cite">It would be really nice to have statistics on the amount of reordering that was done during scheduling.</blockquote><div><br></div>I'll let this open for another patch. Just to let you know: the new scheduling algorithm tries to keep the original instruction order as good as possible.</div><div><br></div><div><blockquote type="cite">I don't like this comment</blockquote><br></div><div>I now initialise everything in the constructor and don't need to call init any more. Therefore this comment is gone anyway.</div><div><br></div><div><blockquote type="cite">You probably want to use a ModRef query here for call sites</blockquote><br></div><div>I'll also want to keep this open for another patch, unless you think it is necessary. </div><div>Also in the old version the alias checking is done this way. I just "copied" it.</div><div><br></div><div><blockquote type="cite">Please go ahead and initialize everything in the constructor</blockquote><br></div><div>done</div><div><br></div><div><blockquote type="cite">std::make_unique?<br></blockquote><div><br></div><div>As I understand make_unique is not yet available in C++11</div><div><br></div><div><blockquote type="cite">It'd also be nice if you could break this up to make it a bit more<br>incremental. I.e. things like the statistic can just be added first.<br></blockquote><br></div><div>I'm not sure I understood.</div><div>I tried to make only little changes in this large function.</div><div><br></div><div><blockquote type="cite">Block comment above this function documenting it?<br></blockquote><br></div><div>Is at the function declaration in the class definition</div><div><br></div><div><blockquote type="cite">"safe"<br></blockquote><br></div><div>done</div><div><br></div><div>Thanks,</div><div>Erik</div><div><br></div></div></body></html>