[libcxx-commits] [libcxx] [libc++] Make __config_site modular (PR #134699)

[Louis Dionne via libcxx-commits]

================
@@ -2101,8 +2100,16 @@ set(files
 configure_file("__config_site.in" "${LIBCXX_GENERATED_INCLUDE_TARGET_DIR}/__config_site" @ONLY)
 configure_file("${LIBCXX_ASSERTION_HANDLER_FILE}" "${LIBCXX_GENERATED_INCLUDE_DIR}/__assertion_handler" COPYONLY)
 
+# We generate the modulemap file so that we can include __config_site in it. For now, we don't know how to
+# make __config_site modular when per-target runtime directories are used.
+if (NOT LLVM_ENABLE_PER_TARGET_RUNTIME_DIR)
+  set(LIBCXX_CONFIG_SITE_MODULE_ENTRY "header \"__config_site\"")
----------------
ldionne wrote:

After discussion we decided to make it a textual header for now.

https://github.com/llvm/llvm-project/pull/134699