<div dir="ltr">On Tue, Jun 25, 2013 at 10:46 PM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank" class="cremed">bob.wilson@apple.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word"><div><div class="h5"><br><div><div>On Jun 25, 2013, at 5:11 PM, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank" class="cremed">chandlerc@gmail.com</a>> wrote:</div>
<br><blockquote type="cite"><div style="font-family:Helvetica;font-size:12px;font-style:normal;font-variant:normal;font-weight:normal;letter-spacing:normal;line-height:normal;text-align:start;text-indent:0px;text-transform:none;white-space:normal;word-spacing:0px">
<div dir="ltr">On Tue, Jun 25, 2013 at 2:44 PM, Bob Wilson<span> </span><span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank" class="cremed">bob.wilson@apple.com</a>></span><span> </span>wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word">
OK, I think I managed to get our buildbots unblocked.  We have some internal code to add armv7s, which wasn't disabled by the hack in 184816.  I've fixed that now, so hopefully our buildbots will start passing again.<div>
<br></div><div>But, we still need a real solution.  It looks like we just need to add errno.h to the fake SDK for Darwin.  Or is the problem more fundamental than that?</div></div></blockquote><div><br></div><div>I don't have strong opinions here.</div>
<div><br></div><div>I can't readily add an errno.h (or anything else) to the fake SDK for Darwin because I have no idea what to put in it. If you guys want to do that, cool. Personally, I don't like the idea of using a fake SDK for this at all. Instead, I much prefer that the build system test for the existence of the real SDK and use it to build the library.</div>
<div><br></div><div>That said, I'm also fine with making the use of the actual libc system headers be Linux-only if you guys really need to continue to do the fake SDK thing. I think Bill has already implemented this, and it looks like a good solution to me.</div>
<div><br></div><div>Ultimately, the only thing I really care about is being able to add the code to this library necessary to make failures encountered in it debuggable on Linux. I actually think the current approach works just fine for this (using __APPLE__ to hide the errno business) but will leave it to you guys to revert the changes to<span> </span><a href="http://clang_darwin.mk/" target="_blank" class="cremed">clang_darwin.mk</a><span> </span>when you're comfortable with tho solution.</div>
</div></div></div></div></blockquote><br></div></div></div><div dir="auto">Unfortunately we’re going to need to continue using the fake SDK approach for Darwin, at least for Apple’s builds.  I’m not a fan either, but we haven’t been able to come up with any reasonable alternative.</div>
<br><div>We can either add errno.h to the fake SDK for Darwin or keep Bill’s __APPLE__ ifdef workaround.  With that in place now, can we revert at least the <a href="http://clang_darwin.mk" target="_blank" class="cremed">clang_darwin.mk</a> parts of 184805?</div>
</div></blockquote><div><br></div><div>Yep. I'll switch it back.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div style="word-wrap:break-word">
<div>  I assume you want to keep the <a href="http://clang_linux.mk" target="_blank" class="cremed">clang_linux.mk</a> change to avoid using the fake SDK for the profiling library, right?</div></div></blockquote><div><br>
</div><div>That's currently my preference. It leaves us the freedom to add any other logic useful on Linux behind the ifdef workarounds.</div></div></div></div>