<div dir="ltr">I've gone ahead and done this thusly:<div><br></div><div><div>Committing to <a href="https://llvm.org/svn/llvm-project/llvm/trunk">https://llvm.org/svn/llvm-project/llvm/trunk</a> ...</div><div><span style="white-space:pre">     </span>R<span style="white-space:pre">    </span>lib/Target/PowerPC/PPCBranchCoalescing.cpp => lib/CodeGen/BranchCoalescing.cpp</div><div><span style="white-space:pre">     </span>M<span style="white-space:pre">    </span>include/llvm/CodeGen/Passes.h</div><div><span style="white-space:pre"> </span>M<span style="white-space:pre">    </span>include/llvm/InitializePasses.h</div><div><span style="white-space:pre">       </span>M<span style="white-space:pre">    </span>lib/CodeGen/CMakeLists.txt</div><div><span style="white-space:pre">    </span>M<span style="white-space:pre">    </span>lib/CodeGen/CodeGen.cpp</div><div><span style="white-space:pre">       </span>M<span style="white-space:pre">    </span>lib/CodeGen/TargetPassConfig.cpp</div><div><span style="white-space:pre">      </span>M<span style="white-space:pre">    </span>lib/Target/PowerPC/CMakeLists.txt</div><div><span style="white-space:pre">     </span>M<span style="white-space:pre">    </span>lib/Target/PowerPC/PPC.h</div><div><span style="white-space:pre">      </span>M<span style="white-space:pre">    </span>lib/Target/PowerPC/PPCTargetMachine.cpp</div><div><span style="white-space:pre">       </span>M<span style="white-space:pre">    </span>test/CodeGen/PowerPC/branch_coalesce.ll</div><div><span style="white-space:pre">       </span>M<span style="white-space:pre">    </span>test/CodeGen/PowerPC/select-i1-vs-i1.ll</div><div>Committed r312214</div></div><div><br></div><div>And we can revisit here tomorrow :)</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 30, 2017 at 10:50 PM Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><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" target="_blank">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></div><div dir="ltr"><div class="gmail_quote"><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></div><div dir="ltr"><div class="gmail_quote"><div><br></div><div>-eric</div></div></div><div dir="ltr"><div class="gmail_quote"><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></blockquote></div>