[llvm] 5482228 - [gn build] fix bug in fb0b19c3de3b

Nico Weber via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 13 06:06:55 PDT 2021


Author: Nico Weber
Date: 2021-04-13T09:06:29-04:00
New Revision: 5482228e5cb645dbcf1f3a8b2e8bf48b191d49b4

URL: https://github.com/llvm/llvm-project/commit/5482228e5cb645dbcf1f3a8b2e8bf48b191d49b4
DIFF: https://github.com/llvm/llvm-project/commit/5482228e5cb645dbcf1f3a8b2e8bf48b191d49b4.diff

LOG: [gn build] fix bug in fb0b19c3de3b

Since __config is no longer in the concatenated generated __config_site,
it now needs to be copied.

(Also fix a comment typo while here.)

Added: 
    

Modified: 
    llvm/utils/gn/secondary/libcxx/include/BUILD.gn

Removed: 
    


################################################################################
diff  --git a/llvm/utils/gn/secondary/libcxx/include/BUILD.gn b/llvm/utils/gn/secondary/libcxx/include/BUILD.gn
index 0c3b1128b0c45..02896bd7d7954 100644
--- a/llvm/utils/gn/secondary/libcxx/include/BUILD.gn
+++ b/llvm/utils/gn/secondary/libcxx/include/BUILD.gn
@@ -6,7 +6,7 @@ libcxx_generated_include_dir = "$root_build_dir/include/c++/v1"
 # This is a bit weird. For now, we assume that __config_site is identical
 # in all toolchains, and only copy it (and all other libcxx headers)
 # to 'include' in the root build dir, so that it's the same for all toolchains.
-# Maybe we wnt to make this per-toolchain eventually (and then use root_out_dir
+# Maybe we want to make this per-toolchain eventually (and then use root_out_dir
 # in libcxx_generated_include_dir) -- e.g. for cross-builds that for example
 # use for-linux-configured libc++ for the host build but for-windows-configured
 # libc++ for the target build.
@@ -64,6 +64,7 @@ if (current_toolchain == default_toolchain) {
       "__bits",
       "__bsd_locale_defaults.h",
       "__bsd_locale_fallbacks.h",
+      "__config",
       "__debug",
       "__errc",
       "__functional_03",


        


More information about the llvm-commits mailing list