[libcxx-commits] [PATCH] D153042: [NFC][libc++][test] Improve name test file.

Mark de Wever via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Thu Jun 15 09:17:16 PDT 2023


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

With C++20 modules in libc++ the old name modules is ambiguous. This
rename makes it clear this is regarding the clang modules.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D153042

Files:
  libcxx/test/libcxx/clang_modules_include.gen.py
  libcxx/test/libcxx/modules_include.gen.py


Index: libcxx/test/libcxx/modules_include.gen.py
===================================================================
--- /dev/null
+++ libcxx/test/libcxx/modules_include.gen.py
@@ -1,43 +0,0 @@
-#===----------------------------------------------------------------------===##
-#
-# Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
-# See https://llvm.org/LICENSE.txt for license information.
-# SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
-#
-#===----------------------------------------------------------------------===##
-
-# Test that we can include each header in a TU while using modules.
-# This is important notably because the LLDB data formatters use
-# libc++ headers with modules enabled.
-
-# RUN: %{python} %s %{libcxx}/utils
-
-import sys
-sys.path.append(sys.argv[1])
-from libcxx.test.header_information import lit_header_restrictions, public_headers
-
-for header in public_headers:
-  BLOCKLIT = '' # block Lit from interpreting a RUN/XFAIL/etc inside the generation script
-  print(f"""\
-//--- {header}.compile.pass.cpp
-// RUN{BLOCKLIT}: %{{cxx}} %s %{{flags}} %{{compile_flags}} -fmodules -fcxx-modules -fmodules-cache-path=%t -fsyntax-only
-
-// UNSUPPORTED{BLOCKLIT}: use_module_std
-
-// GCC doesn't support -fcxx-modules
-// UNSUPPORTED{BLOCKLIT}: gcc
-
-// The Windows headers don't appear to be compatible with modules
-// UNSUPPORTED{BLOCKLIT}: windows
-// UNSUPPORTED{BLOCKLIT}: buildhost=windows
-
-// The Android headers don't appear to be compatible with modules yet
-// XFAIL{BLOCKLIT}: LIBCXX-ANDROID-FIXME
-
-// TODO: Investigate this failure
-// UNSUPPORTED{BLOCKLIT}: LIBCXX-FREEBSD-FIXME
-
-{lit_header_restrictions.get(header, '')}
-
-#include <{header}>
-""")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153042.531790.patch
Type: text/x-patch
Size: 1741 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/libcxx-commits/attachments/20230615/ddd75371/attachment.bin>


More information about the libcxx-commits mailing list