[libcxx-commits] [PATCH] D156573: [libcxx] [test] Deduplicate setting parameters for clangcl test configs. NFC.
Martin Storsjö via Phabricator via libcxx-commits
libcxx-commits at lists.llvm.org
Fri Jul 28 14:17:07 PDT 2023
mstorsjo created this revision.
mstorsjo added reviewers: Mordante, andrewng.
Herald added a project: All.
mstorsjo requested review of this revision.
Herald added a project: libc++.
Herald added a reviewer: libc++.
This patch is written on top of D155561 <https://reviews.llvm.org/D155561> and D155562 <https://reviews.llvm.org/D155562>, but it can be
rebased and applied first, with the other two coming in on top of
this as well.
I'm undecided about whether this is a good thing or not. The duplicated
logic is clearly not great. But this on the other hand includes a lot
of very obscure variable setting that only makes sense in the context
of the *-clangcl.cfg.in files.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D156573
Files:
libcxx/test/CMakeLists.txt
libcxx/test/configs/llvm-libc++-shared-clangcl.cfg.in
libcxx/test/configs/llvm-libc++-shared-no-vcruntime-clangcl.cfg.in
libcxx/test/configs/llvm-libc++-static-clangcl.cfg.in
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D156573.545280.patch
Type: text/x-patch
Size: 6279 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230728/6f42f7a6/attachment.bin>
More information about the libcxx-commits
mailing list