<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 8, 2020 at 1:40 AM Serge Guelton <<a href="mailto:sguelton@redhat.com">sguelton@redhat.com</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 Tue, Jan 07, 2020 at 04:57:29PM -0800, Eric Christopher wrote:<br>
>     >  ; CHECK-EP-VECTORIZER-START-NEXT: Running pass: NoOpFunctionPass<br>
>     > +; CHECK-EXT: Running pass: {{.*}}::Bye on foo<br>
>     ><br>
>     > Why is this running on every test of the pass manager? It should be an<br>
>     example<br>
>     > run in the examples directory and not on by default? Same for every other<br>
>     PM<br>
>     > test. This seems like a bug?<br>
> <br>
>     It's not. When the examples are active and if the appropriate cmake flag is<br>
>     set<br>
>     (which is not the case by default), the pass is linked in statically, and<br>
>     is run<br>
>     in the default pipeline. The CHECK-EXT prefix is disabled otherwise. That's<br>
>     one<br>
>     of the configuration I did test :-)<br>
> <br>
> <br>
> <br>
> I really don't think this is ideal. The examples directory shouldn't affect<br>
> tests being run or not or in what way. Can we back this part out and talk about<br>
> it a bit more? I don't think we should need to do this to test the<br>
> functionality.<br>
<br>
I can just error if LLVM_BYE_LINK_INTO_TOOLS=ON, and remove that part. It means<br>
less testing, but somehow Polly is already some kind of test case. Would<br>
that be okay with you?<br></blockquote><div><br></div><div>I'd really like to avoid linking bye into any of the tools by default and instead just come up with a different test mechanism to show it's working.</div><div><br></div><div>-eric </div></div></div>