[libcxx-commits] [libcxx] [libc++][modules] Removes some validation quirks. (PR #88031)

via libcxx-commits libcxx-commits at lists.llvm.org
Mon Apr 8 11:45:52 PDT 2024


llvmbot wrote:


<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-libcxx

Author: Mark de Wever (mordante)

<details>
<summary>Changes</summary>

Recent unrelated header cleanups caused these quirks to become obsolete.

---
Full diff: https://github.com/llvm/llvm-project/pull/88031.diff


1 Files Affected:

- (modified) libcxx/utils/libcxx/test/modules.py (-5) 


``````````diff
diff --git a/libcxx/utils/libcxx/test/modules.py b/libcxx/utils/libcxx/test/modules.py
index 3f3c7999a1a21d..6c8f1d9a8e1d50 100644
--- a/libcxx/utils/libcxx/test/modules.py
+++ b/libcxx/utils/libcxx/test/modules.py
@@ -54,8 +54,6 @@
     "std::operator==",
 ]
 
-# Declared in the forward header since std::string uses std::allocator
-SkipDeclarations["string"] = ["std::allocator"]
 # TODO MODULES remove zombie names
 # https://libcxx.llvm.org/Status/Cxx20.html#note-p0619
 SkipDeclarations["memory"] = [
@@ -63,9 +61,6 @@
     "std::get_temporary_buffer",
 ]
 
-# TODO MODULES this should be part of ios instead
-SkipDeclarations["streambuf"] = ["std::basic_ios"]
-
 # include/__type_traits/is_swappable.h
 SkipDeclarations["type_traits"] = [
     "std::swap",

``````````

</details>


https://github.com/llvm/llvm-project/pull/88031


More information about the libcxx-commits mailing list