<div dir="auto"><div>IIRC LLVM_LINK_COMPONENTS is already used in a few places. Should they be normalized to LINK_COMPONENTS as well?<br><br><div class="gmail_quote"><div dir="ltr">On Fri, Jul 26, 2019, 10:57 AM Chris Bieneman via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">beanz added inline comments.<br>
Herald added a subscriber: s.egerton.<br>
<br>
<br>
================<br>
Comment at: llvm/lib/Analysis/CMakeLists.txt:1<br>
+set(LLVM_LINK_COMPONENTS<br>
+  BinaryFormat<br>
----------------<br>
It would be better to use the `LINK_COMPONENTS` parameter to `add_llvm_library` rather than setting this. CMake's variable scoping behavior is non-intuitive, which can lead to bugs.<br>
<br>
<br>
Repository:<br>
  rG LLVM Github Monorepo<br>
<br>
CHANGES SINCE LAST ACTION<br>
  <a href="https://reviews.llvm.org/D28855/new/" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D28855/new/</a><br>
<br>
<a href="https://reviews.llvm.org/D28855" rel="noreferrer noreferrer" target="_blank">https://reviews.llvm.org/D28855</a><br>
<br>
<br>
<br>
</blockquote></div></div></div>