GN: Use special toolchain for nacl64.exe, force is_component_build off

The nacl64.exe build within the Windows 32-bit x86 Chrome build is not
compatible with component-style builds.  Build nacl64.exe in a special
variant of the x64 toolchain, wherein is_component_build is always false.

BUG=512869
CQ_EXTRA_TRYBOTS=tryserver.chromium.win:win8_chromium_gn_dbg,win_chromium_gn_x64_rel,win_clang_dbg

Review URL: https://codereview.chromium.org/1650633003

Cr-Commit-Position: refs/heads/master@{#372485}

1 files changed, 56 insertions, 16 deletions

diff --git a/build/toolchain/win/BUILD.gn b/build/toolchain/win/BUILD.gn
index 4fb7279..858e9b4 100644
--- a/build/toolchain/win/BUILD.gn
+++ b/build/toolchain/win/BUILD.gn
@@ -225,9 +225,12 @@ template("msvc_toolchain") {
       if (defined(invoker.toolchain_os)) {
         current_os = invoker.toolchain_os
       }
-      if (defined(invoker.is_clang)) {
-        is_clang = invoker.is_clang
-      }
+
+      forward_variables_from(invoker,
+                             [
+                               "is_clang",
+                               "is_component_build",
+                             ])
 
       # This value needs to be passed through unchanged.
       host_toolchain = host_toolchain
@@ -284,21 +287,58 @@ x64_toolchain_data = exec_script("setup_toolchain.py",
                                  ],
                                  "scope")
 
-msvc_toolchain("x64") {
-  environment = "environment.x64"
-  toolchain_cpu = "x64"
-  cl = "${goma_prefix}\"${x64_toolchain_data.vc_bin_dir}/cl.exe\""
-  is_clang = false
+template("win_x64_toolchains") {
+  # TODO(mcgrathr): These assignments are only required because of
+  # crbug.com/395883.  Drop them if that ever gets fixed in GN.
+  concurrent_links = invoker.concurrent_links
+  goma_prefix = invoker.goma_prefix
+  x64_toolchain_data = invoker.x64_toolchain_data
+  clang_cl = invoker.clang_cl
+
+  msvc_toolchain(target_name) {
+    environment = "environment.x64"
+    toolchain_cpu = "x64"
+    cl = "${goma_prefix}\"${x64_toolchain_data.vc_bin_dir}/cl.exe\""
+    is_clang = false
+
+    forward_variables_from(invoker, [ "is_component_build" ])
+  }
+
+  msvc_toolchain("clang_" + target_name) {
+    environment = "environment.x64"
+    toolchain_cpu = "x64"
+    prefix = rebase_path("//third_party/llvm-build/Release+Asserts/bin",
+                         root_build_dir)
+    cl = "${goma_prefix}$prefix/${clang_cl}"
+    toolchain_os = "win"
+    is_clang = true
+
+    forward_variables_from(invoker, [ "is_component_build" ])
+  }
+}
+
+win_x64_toolchains("x64") {
+  # TODO(mcgrathr): These assignments are only required because of
+  # crbug.com/395883.  Drop them if that ever gets fixed in GN.
+  concurrent_links = concurrent_links
+  goma_prefix = goma_prefix
+  x64_toolchain_data = x64_toolchain_data
 }
 
-msvc_toolchain("clang_x64") {
-  environment = "environment.x64"
-  toolchain_cpu = "x64"
-  prefix = rebase_path("//third_party/llvm-build/Release+Asserts/bin",
-                       root_build_dir)
-  cl = "${goma_prefix}$prefix/${clang_cl}"
-  toolchain_os = "win"
-  is_clang = true
+# The nacl_win64 toolchain is nearly identical to the plain x64 toolchain.
+# It's used solely for building nacl64.exe (//components/nacl/broker:nacl64).
+# The only reason it's a separate toolchain is so that it can force
+# is_component_build to false in the toolchain_args() block, because
+# building nacl64.exe in component style does not work.
+win_x64_toolchains("nacl_win64") {
+  is_component_build = false
+
+  # TODO(mcgrathr): These assignments are only required because of
+  # crbug.com/395883.  Drop them if that ever gets fixed in GN.
+  concurrent_links = concurrent_links
+  goma_prefix = goma_prefix
+  x64_toolchain_data = x64_toolchain_data
+  clang_cl = clang_cl
 }
 
 # WinRT toolchains. Only define these when targeting them.