<div dir="ltr"><div>Hi Davide,</div><div><br></div><div dir="ltr"><a href="https://reviews.llvm.org/D49340">https://reviews.llvm.org/D49340</a></div><div dir="ltr">It was fixed here but I do not have the commit access so I did ask for help. Maybe it slipped your attention</div><div dir="ltr"><br clear="all"><div><div dir="ltr" class="gmail_signature"><div dir="ltr">Son Tuan Vu</div></div></div><br></div></div><br><div class="gmail_quote"><div dir="ltr">On Mon, Dec 17, 2018 at 7:28 PM Davide Italiano <<a href="mailto:davide@freebsd.org">davide@freebsd.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">On Fri, Jul 13, 2018 at 12:59 PM Duncan P. N. Exon Smith<br>
<<a href="mailto:dexonsmith@apple.com" target="_blank">dexonsmith@apple.com</a>> wrote:<br>
><br>
> +Son Tuan Vu<br>
><br>
> On Jul. 13, 2018, at 11:08, Davide Italiano <<a href="mailto:davide@freebsd.org" target="_blank">davide@freebsd.org</a>> wrote:<br>
><br>
> On Wed, Jul 11, 2018 at 4:35 PM Duncan P. N. Exon Smith via<br>
> llvm-commits <<a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a>> wrote:<br>
><br>
><br>
> Author: dexonsmith<br>
> Date: Wed Jul 11 16:30:25 2018<br>
> New Revision: 336869<br>
><br>
> URL: <a href="http://llvm.org/viewvc/llvm-project?rev=336869&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=336869&view=rev</a><br>
> Log:<br>
> IR: Skip -print-*-all after -print-*<br>
><br>
> This changes `-print-*` from transformation passes to analysis passes so<br>
> that `-print-after-all` and `-print-before-all` don't trigger.  This<br>
> avoids some redundant output.<br>
><br>
> Patch by Son Tuan Vu!<br>
><br>
> Added:<br>
>    llvm/trunk/test/Other/printer.ll<br>
> Modified:<br>
>    llvm/trunk/lib/IR/IRPrintingPasses.cpp<br>
><br>
> Modified: llvm/trunk/lib/IR/IRPrintingPasses.cpp<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/IRPrintingPasses.cpp?rev=336869&r1=336868&r2=336869&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/IR/IRPrintingPasses.cpp?rev=336869&r1=336868&r2=336869&view=diff</a><br>
> ==============================================================================<br>
> --- llvm/trunk/lib/IR/IRPrintingPasses.cpp (original)<br>
> +++ llvm/trunk/lib/IR/IRPrintingPasses.cpp Wed Jul 11 16:30:25 2018<br>
> @@ -127,13 +127,13 @@ public:<br>
><br>
> char PrintModulePassWrapper::ID = 0;<br>
> INITIALIZE_PASS(PrintModulePassWrapper, "print-module",<br>
> -                "Print module to stderr", false, false)<br>
> +                "Print module to stderr", false, true)<br>
> char PrintFunctionPassWrapper::ID = 0;<br>
> INITIALIZE_PASS(PrintFunctionPassWrapper, "print-function",<br>
> -                "Print function to stderr", false, false)<br>
> +                "Print function to stderr", false, true)<br>
> char PrintBasicBlockPass::ID = 0;<br>
> INITIALIZE_PASS(PrintBasicBlockPass, "print-bb", "Print BB to stderr", false,<br>
> -                false)<br>
> +                true)<br>
><br>
> ModulePass *llvm::createPrintModulePass(llvm::raw_ostream &OS,<br>
>                                         const std::string &Banner,<br>
><br>
> Added: llvm/trunk/test/Other/printer.ll<br>
> URL: <a href="http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Other/printer.ll?rev=336869&view=auto" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/llvm/trunk/test/Other/printer.ll?rev=336869&view=auto</a><br>
> ==============================================================================<br>
> --- llvm/trunk/test/Other/printer.ll (added)<br>
> +++ llvm/trunk/test/Other/printer.ll Wed Jul 11 16:30:25 2018<br>
> @@ -0,0 +1,13 @@<br>
> +; RUN: opt -mem2reg -instcombine -print-after-all -S < %s 2>&1 | FileCheck %s<br>
> +define void @tester(){<br>
> +  ret void<br>
> +}<br>
> +<br>
> +define void @foo(){<br>
> +  ret void<br>
> +}<br>
> +<br>
> +;CHECK: IR Dump After Promote Memory to Register<br>
> +;CHECK: IR Dump After Combine redundant instructions<br>
> +;CHECK: IR Dump After Module Verifier<br>
> +;CHECK-NOT: IR Dump After Print Module IR<br>
><br>
><br>
> I'm afraid this will break, as CHECK-NOT(s) are really fragile. Is there<br>
> a better way we can test this? probably `CHECK-NEXT` & adding a "end<br>
> of pipeline" message?<br>
><br>
><br>
> I agree, that seems better.  Son, can you take a care of that?<br>
><br>
<br>
Was this ever fixed?<br>
<br>
--<br>
Davide<br>
</blockquote></div>