<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 9, 2014 at 3:01 PM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">> The problem with making DT optional is that you might never get it.<br>
> If any pass that runs before InstCombine invalidates DT, it won’t be<br>
> available to any following pass until there is a pass that<br>
> explicitly requires it.<br>
<br>
</span>That's true. There have been objections in the past to making it required, and I don't recall by whom or why exactly. In the current standard pass manager configuration, it does happen at least once.</blockquote></div><br>Just to reiterate what I said in the code review, I actually agree with Juergen, but I think it would be fine to make it required. I just want to get the business logic into instcombine right away, and then double check that the compile time is OK when we make DT required.</div></div>