<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Tue, Sep 23, 2014 at 3:49 PM, Richard Smith <span dir="ltr"><<a href="mailto:richard@metafoo.co.uk" target="_blank">richard@metafoo.co.uk</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"><div class="gmail_extra"><div class="gmail_quote"><div><div class="h5">On Fri, Sep 19, 2014 at 6:35 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div dir="ltr">Hi,<div><br></div><div>when using modules in c++98, libc++ currently fails to build:</div><div><br></div><div><div>While building module 'std' imported from src/disk_interface.h:18:</div><div>In file included from <module-includes>:3:</div><div>/Users/thakis/src/chrome/src/third_party/llvm-build/Release+Asserts/bin/../include/c++/v1/atomic:539:2: error: <atomic> is not implemented</div><div>#error <atomic> is not implemented</div><div> ^</div><div>In file included from src/disk_interface.cc:15:</div><div>src/disk_interface.h:18:10: fatal error: could not build module 'std'</div><div>#include <map></div><div> ~~~~~~~~^</div></div><div><br></div><div>The attached patch fixes this by making atomic depend on cxx_atomics. Fixes PR21002.</div></div></blockquote><div><br></div></div></div><div>Doesn't this just make the atomic submodule always unavailable? We don't have a cxx_atomics feature, as far as I can see. And cxx_atomic (the corresponding __has_feature flag) wouldn't work either: module features are not the same as __has_feature features (see the relevant documentation here: <a href="http://clang.llvm.org/docs/Modules.html#requires-declaration" target="_blank">http://clang.llvm.org/docs/Modules.html#requires-declaration</a>). cplusplus11 is the best check we have for this.</div></div></div></div>
</blockquote></div><br></div><div class="gmail_extra">Aw, bummer. Is the patch ok with s/cxx_atomics/cplusplus11/?</div></div>