<div dir="ltr">Hrm. This is still failing for me on linux.<div><br></div><div>I've disabled the test harder here:</div><div><br></div><div>echristo@athyra ~/r/llvm-project> git push<br>To github.com:llvm/llvm-project.git<br>   6563826ff0f..c23c8746d0c  master -> master<br></div><div><br></div><div>while investigation is happening. Let me know what I can do to help (cmake configuration? something else?)</div><div><br></div><div>Thanks!</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jan 7, 2020 at 12:58 PM serge via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.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">serge-sans-paille added a comment.<br>
<br>
In D72327#1808263 <<a href="https://reviews.llvm.org/D72327#1808263" rel="noreferrer" target="_blank">https://reviews.llvm.org/D72327#1808263</a>>, @fhahn wrote:<br>
<br>
> Does this fix `llvm/test/Feature/load_extension.ll` on macOS by any chance?<br>
<br>
<br>
Now it should, but I'm waiting for my pre-commit validation to state if that's actually the case.<br>
<br>
<br>
<br>
================<br>
Comment at: llvm/examples/Bye/CMakeLists.txt:1<br>
-add_llvm_pass_plugin(Bye<br>
+if(LLVM_BUILD_EXAMPLES)<br>
+  add_llvm_pass_plugin(Bye<br>
----------------<br>
Meinersbur wrote:<br>
> [serious] Whether example targets are generated is controlled by `LLVM_INCLUDE_EXAMPLES`. `LLVM_BUILD_EXAMPLES` should control `EXCLUDE_FROM_ALL` or similar. Is there a corresponding option for pass plugins, especially if `LLVM_BYE_LINK_INTO_TOOLS` is off?<br>
> <br>
> I'd assume that if `LLVM_BUILD_EXAMPLES` is off, but `LLVM_BYE_LINK_INTO_TOOLS` is on, that latter would effectively override `EXCLUDE_FROM_ALL` since the plugin is needed to built opt,bugpoint,etc.<br>
> <br>
> Btw, what is the "Fix validation issue when building examples"?<br>
> I'd assume that if LLVM_BUILD_EXAMPLES is off, but LLVM_BYE_LINK_INTO_TOOLS is on, that latter would effectively override EXCLUDE_FROM_ALL since the plugin is needed to built opt,bugpoint,etc.<br>
<br>
The problem I wanted to solve here is having LLVM_BYE_LINK_INTO_TOOLS=ON in release build, because Bye is just an example library. But it's nice to be able to test it. Maybe I should just issue a warning in that case?<br>
<br>
> Btw, what is the "Fix validation issue when building examples"?<br>
<br>
There was a mismatch between the lit feature « requires build_example » and « LLVM_BUILD_EXAMPLES » I've updated the patch in that aspect, taking into account your remark.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D72327/new/" rel="noreferrer" target="_blank">https://reviews.llvm.org/D72327/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D72327" rel="noreferrer" target="_blank">https://reviews.llvm.org/D72327</a><br>
<br>
<br>
<br>
</blockquote></div>