<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 10, 2014 at 5:00 PM, Gerolf Hoflehner <span dir="ltr"><<a href="mailto:ghoflehner@apple.com" target="_blank">ghoflehner@apple.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Perhaps I should phrase it differently. DT is an analysis required by other passes like LICM which runs after the combiner.  The combiner does not destroy the Dominator Tree. So what is the root of your concern?</blockquote></div><br>Requiring DT will likely cause DT to be run more times during the compilation (I'm pretty sure there is at least one place where nothing else needs it and there is a pass that invalidates it). As a consequence, this could *in theory* have an impact on compile time.</div><div class="gmail_extra"><br></div><div class="gmail_extra">However, I don't actually expect it to have such an impact. I just think its reasonable to measure the compile time impact because sometimes my expectations are quite wrong. =]</div><div class="gmail_extra"><br></div><div class="gmail_extra">(The only reason I mention a separate thread is to try to help make the code review progress faster -- the code is fine regardless of what the compile time results show, and we could potentially look at ways to get DT some of the time when it will be used later anyways but not always to avoid any compile time regressions.)</div></div>