[libcxx-commits] [PATCH] D102798: [SystemZ][z/OS] XFAIL aligned allocation tests on z/OS

[Daniel McIntosh via Phabricator via libcxx-commits]

DanielMcIntosh-IBM created this revision.
DanielMcIntosh-IBM added a reviewer: ldionne.
DanielMcIntosh-IBM requested review of this revision.
Herald added a project: libc++.
Herald added a subscriber: libcxx-commits.
Herald added a reviewer: libc++.

zOS doesn't support aligned allocation, so these tests are failing.
For more details on aligned allocation in zOS, see
https://reviews.llvm.org/D87611 and https://reviews.llvm.org/D90178


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D102798

Files:
  libcxx/test/libcxx/language.support/support.dynamic/libcpp_deallocate.sh.cpp
  libcxx/test/libcxx/language.support/support.dynamic/new_faligned_allocation.pass.cpp


Index: libcxx/test/libcxx/language.support/support.dynamic/new_faligned_allocation.pass.cpp
===================================================================
--- libcxx/test/libcxx/language.support/support.dynamic/new_faligned_allocation.pass.cpp
+++ libcxx/test/libcxx/language.support/support.dynamic/new_faligned_allocation.pass.cpp
@@ -13,13 +13,14 @@
 // definitions, which does not yet provide aligned allocation
 // XFAIL: LIBCXX-WINDOWS-FIXME
 
-// The dylibs shipped before macosx10.13 do not contain the aligned allocation
-// functions, so trying to force using those with -faligned-allocation results
-// in a link error.
+// The dynamic library shipped with z/OS and Mac OS X before macosx10.13 do not
+// contain the aligned allocation functions, so trying to force using those
+// with -faligned-allocation results in a link error.
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.12
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.11
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.10
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.9
+// XFAIL: -zos
 
 // REQUIRES: -faligned-allocation
 // ADDITIONAL_COMPILE_FLAGS: -faligned-allocation
Index: libcxx/test/libcxx/language.support/support.dynamic/libcpp_deallocate.sh.cpp
===================================================================
--- libcxx/test/libcxx/language.support/support.dynamic/libcpp_deallocate.sh.cpp
+++ libcxx/test/libcxx/language.support/support.dynamic/libcpp_deallocate.sh.cpp
@@ -13,13 +13,14 @@
 // definitions, which does not yet provide aligned allocation
 // XFAIL: LIBCXX-WINDOWS-FIXME
 
-// The dylibs shipped before macosx10.13 do not contain the aligned allocation
-// functions, so trying to force using those with -faligned-allocation results
-// in a link error.
+// The dynamic library shipped with z/OS and Mac OS X before macosx10.13 do not
+// contain the aligned allocation functions, so trying to force using those
+// with -faligned-allocation results in a link error.
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.12
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.11
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.10
 // XFAIL: use_system_cxx_lib && x86_64-apple-macosx10.9
+// XFAIL: -zos
 
 // AppleClang < 10 incorrectly warns that aligned allocation is not supported
 // even when it is supported.


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D102798.346529.patch
Type: text/x-patch
Size: 2368 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20210519/36b7d1ff/attachment.bin>