<div dir="ltr">D'oh, forgot to cc Marshall :-)</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, May 13, 2014 at 12:00 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>r207606 changed the __need_foo macros to behave like they do with gcc: If they are set, _only_ the __need_foo stuff gets defined. libc++'s cstddef sets a few __need_foo settings, so cstddef now doesn't work right with libc++ after clang r207606.<br>

</div><div><br></div><div>The attached patch removes the __need_foo defines from libc++'s cstddef. They were added with a cryptic commit message ""for ubuntu" years ago – I think they're incorrect, and things still seem to work without them. (libstdc++ doesn't set them either.) Maybe they were needed before clang had r207606 – if so, this patch should also improve how libc++ works when built with gcc.</div>

<div><br></div><div>This fixes PR19723, see the bug for some more information and discussion.</div><div><br></div><div>Ok?</div><span class="HOEnZb"><font color="#888888"><div><br></div><div>Nico</div></font></span></div>

</blockquote></div><br></div>