<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class="">Today, the code includes the eprintf archive on all linker invocations for i386 that include the OS X archive.<div class=""><br class=""></div><div class="">Removing the eprintf library from the command and including eprintf only for i386 in the OS X archive should be equivalent to what we have today.</div><div class=""><br class=""></div><div class="">-Chris<br class=""><div class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Aug 16, 2016, at 8:27 AM, Daniel Dunbar via cfe-commits <<a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class="">The original motivation for organizing things this way was to try and always be in the situation where, when linking against the latest deployment target, we should never need the extra library (because the content will be present in libSystem). This is useful in helping to ensure we don't unnecessarily embed copies of system provided functions.<div class=""><br class=""></div><div class="">I know it is quite cumbersome to maintain that policy, but have we explicitly given it up? Nick, what is your opinion here?</div><div class=""><div class=""><br class=""></div><div class=""> - Daniel</div></div></div><div class="gmail_extra"><br class=""><div class="gmail_quote">On Mon, Aug 15, 2016 at 3:51 PM, Chris Bieneman <span dir="ltr" class=""><<a href="mailto:beanz@apple.com" target="_blank" class="">beanz@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">beanz created this revision.<br class="">
beanz added reviewers: ddunbar, bob.wilson.<br class="">
beanz added a subscriber: cfe-commits.<br class="">
<br class="">
The eprintf library was added before the general OS X builtins library existed as a place to store one builtin function. Since we have for several years had an actual mandated builtin library for OS X > 10.5, we should just merge eprintf into the main library.<br class="">
<br class="">
This change will resolve PR28855.<br class="">
<br class="">
As a follow up I'll also patch compiler-rt to not generate the eprintf library anymore.<br class="">
<br class="">
<a href="https://reviews.llvm.org/D23531" rel="noreferrer" target="_blank" class="">https://reviews.llvm.org/<wbr class="">D23531</a><br class="">
<br class="">
Files:<br class="">
lib/Driver/ToolChains.cpp<br class="">
<br class="">
Index: lib/Driver/ToolChains.cpp<br class="">
==============================<wbr class="">==============================<wbr class="">=======<br class="">
--- lib/Driver/ToolChains.cpp<br class="">
+++ lib/Driver/ToolChains.cpp<br class="">
@@ -483,8 +483,6 @@<br class="">
if (isMacosxVersionLT(10, 5)) {<br class="">
AddLinkRuntimeLib(Args, CmdArgs, "libclang_rt.10.4.a");<br class="">
} else {<br class="">
- if (getTriple().getArch() == llvm::Triple::x86)<br class="">
- AddLinkRuntimeLib(Args, CmdArgs, "libclang_rt.eprintf.a");<br class="">
AddLinkRuntimeLib(Args, CmdArgs, "libclang_rt.osx.a");<br class="">
}<br class="">
}<br class="">
<br class="">
<br class="">
</blockquote></div><br class=""></div>
_______________________________________________<br class="">cfe-commits mailing list<br class=""><a href="mailto:cfe-commits@lists.llvm.org" class="">cfe-commits@lists.llvm.org</a><br class="">http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits<br class=""></div></blockquote></div><br class=""></div></div></body></html>