<div dir="ltr">Ping.<div><br></div><div>Any comments on the updated patch?</div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Apr 9, 2015 at 2:40 PM, Akira Hatanaka <span dir="ltr"><<a href="mailto:ahatanak@gmail.com" target="_blank">ahatanak@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Here is the updated patch.<br>
<br>
In addition to making changes necessary to pass the predicate to the passes' constructors, I moved the piece of code in lib/CodeGen/Passes.cpp which adds passes in InsertedPasses to the pass pipeline. This change was necessary to enable inserting the machineinstr-printer pass via command line option -print-machineinstrs (two tests, CodeGen/ARM/ifcvt-branch-weight-bug.ll and ifcvt-branch-weight.ll, fail without this change).<br>
<span class=""><br>
<br>
<a href="http://reviews.llvm.org/D8717" target="_blank">http://reviews.llvm.org/D8717</a><br>
<br>
Files:<br>
  include/llvm/CodeGen/Passes.h<br>
  include/llvm/Pass.h<br>
</span>  include/llvm/Transforms/Scalar.h<br>
  lib/CodeGen/IfConversion.cpp<br>
  lib/CodeGen/MachineInstrBundle.cpp<br>
  lib/CodeGen/Passes.cpp<br>
  lib/Target/ARM/ARM.h<br>
  lib/Target/ARM/ARMTargetMachine.cpp<br>
  lib/Target/ARM/Thumb2SizeReduction.cpp<br>
<div class="HOEnZb"><div class="h5">  lib/Transforms/Scalar/SimplifyCFGPass.cpp<br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</div></div></blockquote></div><br></div></div>