diff options
author | Louis Dionne <ldionne.2@gmail.com> | 2023-03-18 13:34:29 -0400 |
---|---|---|
committer | Louis Dionne <ldionne.2@gmail.com> | 2023-03-30 06:57:56 -0400 |
commit | ed61d6a46611cfb144b260e0dd0fb1b5783562d9 (patch) | |
tree | 635c0867a3a270e74ec444c18eb003e4811ee3b9 /libunwind | |
parent | e64cc756819d567f453467bf7cc16599ad296fdd (diff) | |
download | llvm-ed61d6a46611cfb144b260e0dd0fb1b5783562d9.zip llvm-ed61d6a46611cfb144b260e0dd0fb1b5783562d9.tar.gz llvm-ed61d6a46611cfb144b260e0dd0fb1b5783562d9.tar.bz2 |
[libc++] Use the stdlib=<LIB> Lit feature instead of use_system_cxx_lib
The use_system_cxx_lib Lit feature was only used for back-deployment
testing. However, one immense hole in that setup was that we didn't
have a proper way to test Apple's own libc++ outside of back-deployment,
which was embodied by the fact that we needed to define _LIBCPP_DISABLE_AVAILABILITY
when testing (see change in libcxx/utils/libcxx/test/params.py).
This led to the apple-system testing configuration not checking for
availability markup, which is obviously quite bad since the library
we ship actually has availability markup.
Using stdlib=<VENDOR>-libc++ instead to encode back-deployment restrictions
on tests is simpler and it makes it possible to naturally support tests
such as availability markup checking even in the tip-of-trunk Apple-libc++
configuration.
Differential Revision: https://reviews.llvm.org/D146366
Diffstat (limited to 'libunwind')
-rw-r--r-- | libunwind/test/configs/apple-libunwind-backdeployment.cfg.in | 2 | ||||
-rw-r--r-- | libunwind/test/libunwind_01.pass.cpp | 2 |
2 files changed, 3 insertions, 1 deletions
diff --git a/libunwind/test/configs/apple-libunwind-backdeployment.cfg.in b/libunwind/test/configs/apple-libunwind-backdeployment.cfg.in index 4e37a27..4484573 100644 --- a/libunwind/test/configs/apple-libunwind-backdeployment.cfg.in +++ b/libunwind/test/configs/apple-libunwind-backdeployment.cfg.in @@ -54,6 +54,8 @@ config.substitutions.append(('%{exec}', '%{executor} --execdir %T --env DYLD_LIBRARY_PATH="%{cxx-runtime-root}:%{abi-runtime-root}:%{unwind-runtime-root}" -- ' )) +config.stdlib = 'apple-libc++' + import os, site import libcxx.test.params, libcxx.test.config libcxx.test.config.configure( diff --git a/libunwind/test/libunwind_01.pass.cpp b/libunwind/test/libunwind_01.pass.cpp index 4661ac9..96f12d1 100644 --- a/libunwind/test/libunwind_01.pass.cpp +++ b/libunwind/test/libunwind_01.pass.cpp @@ -8,7 +8,7 @@ //===----------------------------------------------------------------------===// // TODO: Investigate this failure on x86_64 macOS back deployment -// XFAIL: use_system_cxx_lib && target=x86_64-apple-macosx{{10.9|10.10|10.11|10.12|10.13|10.14|10.15|11.0|12.0}} +// XFAIL: stdlib=apple-libc++ && target=x86_64-apple-macosx{{10.9|10.10|10.11|10.12|10.13|10.14|10.15|11.0|12.0}} // TODO: Figure out why this fails with Memory Sanitizer. // XFAIL: msan |