<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Jun 13, 2017 at 2:01 PM, Eli Friedman via Phabricator <span dir="ltr"><<a href="mailto:reviews@reviews.llvm.org" target="_blank">reviews@reviews.llvm.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">efriedma added inline comments.<br>
<br>
<br>
================<br>
Comment at: include/llvm/Transforms/Utils/<wbr>OrderedInstructions.h:40<br>
+  /// Constructors.<br>
+  OrderedInstructions() = default;<br>
   OrderedInstructions(<wbr>DominatorTree *DT) : DT(DT) {}<br>
----------------<br>
Not sure I like this constructor... easy to misuse.<br></blockquote><div><br></div><div>IMHO, we always require DT.</div><div><br></div><div>Is there anything that wants local but not global dominance?<br><br></div><div>It's usually IMHO, the other way around, things have global dominance and then want local.</div><div><br></div></div></div></div>