[libcxx-commits] [PATCH] D151046: [libcxx][tests] Introduce 32-bit feature and use it for stringstream gcount test

Azat Khuzhin via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon May 22 22:23:43 PDT 2023


azat updated this revision to Diff 524566.
azat added a comment.

Use sizeof(void*)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D151046/new/

https://reviews.llvm.org/D151046

Files:
  libcxx/test/std/input.output/string.streams/stringstream.members/gcount.pass.cpp
  libcxx/utils/libcxx/test/features.py


Index: libcxx/utils/libcxx/test/features.py
===================================================================
--- libcxx/utils/libcxx/test/features.py
+++ libcxx/utils/libcxx/test/features.py
@@ -381,4 +381,12 @@
   # not other forms of aligned allocation.
   Feature(name='availability-aligned_allocation-missing',
     when=lambda cfg: BooleanExpression.evaluate('stdlib=apple-libc++ && target={{.+}}-apple-macosx10.{{(9|10|11|12)(.0)?}}', cfg.available_features)),
+
+  # Tests that require 64-bit architecture
+  Feature(name='32-bit-pointer',
+          when=lambda cfg: sourceBuilds(cfg, """
+            int main(int, char**) {
+              static_assert(sizeof(void *) == 4);
+            }
+          """)),
 ]
Index: libcxx/test/std/input.output/string.streams/stringstream.members/gcount.pass.cpp
===================================================================
--- libcxx/test/std/input.output/string.streams/stringstream.members/gcount.pass.cpp
+++ libcxx/test/std/input.output/string.streams/stringstream.members/gcount.pass.cpp
@@ -6,12 +6,7 @@
 //
 //===----------------------------------------------------------------------===//
 
-// All 32 bit arches that CI has:
-//
-// UNSUPPORTED: target=powerpc-ibm-aix
-// UNSUPPORTED: target={{.*}}armv7l{{.*}}
-// UNSUPPORTED: target={{.*}}armv8l{{.*}}
-// UNSUPPORTED: target=i686-w64-windows-gnu
+// UNSUPPORTED: 32-bit-pointer
 
 // Test that tellp() does not break the stringstream after INT_MAX, due to use
 // of pbump() that accept int.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D151046.524566.patch
Type: text/x-patch
Size: 1511 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230523/0e8111d8/attachment.bin>


More information about the libcxx-commits mailing list