[PATCH] D26881: [libcxx] Remove CPlusPlus11 requirements from the module.modulemap

Eric Fiselier via cfe-commits cfe-commits at lists.llvm.org
Fri Nov 18 19:42:53 PST 2016


EricWF created this revision.
EricWF added reviewers: rsmith, mclow.lists.
EricWF added a subscriber: cfe-commits.

This patch does two things:

- Allow `std.strstream` to used in C++11. This is needed to get the libc++ test suite passing with modules enabled.  Also enforcing that strstream is not used in C++11 seems like non-conforming behavior.

- Remove the C++11 requirement for `std.atomic`. Libc++ implements almost all of it in C++03.


https://reviews.llvm.org/D26881

Files:
  include/module.modulemap


Index: include/module.modulemap
===================================================================
--- include/module.modulemap
+++ include/module.modulemap
@@ -210,7 +210,6 @@
   module atomic {
     header "atomic"
     export *
-    requires cplusplus11
   }
   module bitset {
     header "bitset"
@@ -387,7 +386,6 @@
   }
   module strstream {
     header "strstream"
-    requires !cplusplus11
   }
   module system_error {
     header "system_error"


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26881.78614.patch
Type: text/x-patch
Size: 458 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20161119/24c85dc7/attachment.bin>


More information about the cfe-commits mailing list