<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Fri, Apr 28, 2017 at 8:00 AM Lei Huang via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">lei added a comment.<br>
<br>
> I think most passes have a cl::opt in `TargetPassConfig.cpp` instead and we don't even add the pass the pipeline if it is disable, you could change to that style too to avoid people seeing the dump for a disabled pass :)<br>
<br>
Will address this in the next patch to turn this on by default for PowerPC.<br>
</blockquote><div><br></div><div>From looking at this it's already turned on by default.</div><div><br></div><div>Since this was unintended I'm going to go ahead and revert for now and then we can add it back in turned off and then turn it back on after the existing issues are fixed.</div><div><br></div><div>Thanks!</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Thanks<br>
<br>
<br>
Repository:<br>
  rL LLVM<br>
<br>
<a href="https://reviews.llvm.org/D28249" rel="noreferrer" target="_blank">https://reviews.llvm.org/D28249</a><br>
<br>
<br>
<br>
</blockquote></div></div>