[all-commits] [llvm/llvm-project] ea4c19: [libcxx] [test] Make filesystem tests not rely on ...
Martin Storsjö via All-commits
all-commits at lists.llvm.org
Wed Mar 2 00:42:32 PST 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: ea4c198a0f56b3fe63600384a63c6ccaa0560b56
https://github.com/llvm/llvm-project/commit/ea4c198a0f56b3fe63600384a63c6ccaa0560b56
Author: Martin Storsjö <martin at martin.st>
Date: 2022-03-02 (Wed, 02 Mar 2022)
Changed paths:
M libcxx/test/configs/llvm-libc++-shared-clangcl.cfg.in
M libcxx/test/configs/llvm-libc++-static-clangcl.cfg.in
M libcxx/test/libcxx/input.output/filesystems/convert_file_time.pass.cpp
M libcxx/test/std/input.output/filesystems/class.directory_entry/directory_entry.obs/last_write_time.pass.cpp
M libcxx/test/std/input.output/filesystems/fs.op.funcs/fs.op.last_write_time/last_write_time.pass.cpp
M libcxx/test/support/filesystem_test_helper.h
M libcxx/utils/libcxx/test/config.py
Log Message:
-----------
[libcxx] [test] Make filesystem tests not rely on libc++ internals
As part of https://reviews.llvm.org/D119036
(506cf6dc048835c598b654e43ed8f723a42e39ba), `-DNOMINMAX` was
dropped from the Windows CI configurations, replaced with a
block with `_LIBCPP_PUSH_MACROS`, `#include <__undef_macros>`
and `_LIBCPP_POP_MACROS` (and
`ADDITIONAL_COMPILE_FLAGS: -DNOMINMAX` left in two tests).
However, this workaround breaks the running the libc++ tests
against a different C++ standard library than libc++, as those
macros and that header are libc++ internals.
Therefore, reinstate `-DNOMINMAX` for clang-cl configurations
and remove the libc++ specific bits in filesystem_test_helper.h.
Differential Revision: https://reviews.llvm.org/D120478
More information about the All-commits
mailing list