<div dir="ltr">LGTM with one tweak:<div><br></div><div><div>+  /// createPrintBBPass - Create and return a pass that writes the</div><div>+  /// BB to the specified raw_ostream.</div><div>+  BasicBlockPass *createPrintBBPass(raw_ostream *OS,</div>
<div>+                                    bool DeleteStream=false,</div><div>+                                    const std::string &Banner = ""); </div><div> } // End llvm namespace</div></div><div><br></div>
<div style>In the expected "public" interface, I would expand BB to BasicBlock. So, 'createPrintBasicBlockPass'.</div><div style><br></div><div style>I'm fine with the internal class name being 'PrintBBPass' or whatever is convenient to type. =]</div>
</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Fri, Feb 8, 2013 at 1:16 PM, Sergei Larin <span dir="ltr"><<a href="mailto:slarin@codeaurora.org" target="_blank">slarin@codeaurora.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hello everyone,<br>
<br>
  I am not precisely sure who owns this code (lib/IR/Pass.cpp ...sorry...),<br>
and I am not sure why to this moment we did not have<br>
BasicBlockPass::createPrinterPass() defined, but I got a nasty surprise when<br>
trying to combine -mllvm -print-after-all with -mllvm -vectorize for<br>
clang...<br>
<br>
   The implementation seemed straightforward, so I just decided to do it and<br>
then ask for background later. So... If it is as harmless as I think it is,<br>
please review and OK the patch. If it should not be done for some reason, or<br>
should be implemented differently, please let me know.<br>
<br>
Thanks a lot.<br>
<br>
Sergei Larin<br>
<br>
---<br>
Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, hosted by<br>
The Linux Foundation<br>
<br>
<br>_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br></blockquote></div><br></div>