<div dir="ltr">Yeah, that review hasn't landed yet. It should be called auto, but we need to make it actually "automatic". Currently "none" really means don't help me, not that we will automatically figure it out for you. I agree with your requested change, it's just a bunch of work that should probably be done on it's own.<div><br></div><div><br></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 22, 2015 at 10:57 AM, Jonathan Roelofs <span dir="ltr"><<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><br>
<br>
On 10/22/15 2:50 PM, Eric Fiselier via cfe-commits wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: ericwf<br>
Date: Thu Oct 22 15:50:07 2015<br>
New Revision: 251062<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=251062&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=251062&view=rev</a><br>
Log:<br>
Disable linker scripts when the ABI library is not specified or is none.<br>
<br>
Modified:<br>
     libcxx/trunk/CMakeLists.txt<br>
<br>
Modified: libcxx/trunk/CMakeLists.txt<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/libcxx/trunk/CMakeLists.txt?rev=251062&r1=251061&r2=251062&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/libcxx/trunk/CMakeLists.txt?rev=251062&r1=251061&r2=251062&view=diff</a><br>
==============================================================================<br>
--- libcxx/trunk/CMakeLists.txt (original)<br>
+++ libcxx/trunk/CMakeLists.txt Thu Oct 22 15:50:07 2015<br>
@@ -74,9 +74,12 @@ option(LIBCXX_ENABLE_STATIC_ABI_LIBRARY<br>
  # Generate and install a linker script inplace of libc++.so. The linker script<br>
  # will link libc++ to the correct ABI library. This option is on by default<br>
  # On UNIX platforms other than Apple unless 'LIBCXX_ENABLE_STATIC_ABI_LIBRARY'<br>
-# is on.<br>
+# is on. This option is also disabled when the ABI library is not specified<br>
+# or is specified to be "none".<br>
  set(ENABLE_LINKER_SCRIPT_DEFAULT_VALUE OFF)<br>
  if (LLVM_HAVE_LINK_VERSION_SCRIPT AND NOT LIBCXX_ENABLE_STATIC_ABI_LIBRARY<br>
+      AND LIBCXX_CXX_ABI<br>
+      AND NOT LIBCXX_CXX_ABI STREQUAL "none"<br>
</blockquote>
<br></div></div>
I remember suggesting in a review that this be called "auto" rather than "none"... but now I can't find that review. Thoughts?<br>
<br>
Jon<div class="HOEnZb"><div class="h5"><br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
        AND PYTHONINTERP_FOUND)<br>
      set(ENABLE_LINKER_SCRIPT_DEFAULT_VALUE ON)<br>
  endif()<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@lists.llvm.org" target="_blank">cfe-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits</a><br>
<br>
</blockquote>
<br></div></div><span class="HOEnZb"><font color="#888888">
-- <br>
Jon Roelofs<br>
<a href="mailto:jonathan@codesourcery.com" target="_blank">jonathan@codesourcery.com</a><br>
CodeSourcery / Mentor Embedded<br>
</font></span></blockquote></div><br></div>