Move the C++ toolchain into its own execution group so that it does not have to be bound by the same constraints as the execution platform selected for the Swift toolchain.

PiperOrigin-RevId: 745988579
(cherry picked from commit bb0f38b)
Signed-off-by: Brentley Jones <[email protected]>

Author: allevato

swift/toolchains/swift_toolchain.bzl

@@ -25,7 +25,7 @@ load("@bazel_skylib//lib:paths.bzl", "paths")
 load("@bazel_skylib//rules:common_settings.bzl", "BuildSettingInfo")
 load(
     "@bazel_tools//tools/cpp:toolchain_utils.bzl",
-    "find_cpp_toolchain",
+    "CPP_TOOLCHAIN_TYPE",
     "use_cpp_toolchain",
 )
 load("@rules_cc//cc/common:cc_common.bzl", "cc_common")
@@ -410,7 +410,7 @@ def _parse_target_system_name(*, arch, os, target_system_name):
 
 def _swift_toolchain_impl(ctx):
     toolchain_root = ctx.attr.root
-    cc_toolchain = find_cpp_toolchain(ctx)
+    cc_toolchain = ctx.exec_groups["default"].toolchains[CPP_TOOLCHAIN_TYPE].cc
     target_system_name = _parse_target_system_name(
         arch = ctx.attr.arch,
         os = ctx.attr.os,
@@ -706,6 +706,14 @@ The version of XCTest that the toolchain packages.
         },
     ),
     doc = "Represents a Swift compiler toolchain.",
+    exec_groups = {
+        # An execution group that has no specific platform requirements. This
+        # ensures that the execution platform of this Swift toolchain does not
+        # unnecessarily constrain the execution platform of the C++ toolchain.
+        "default": exec_group(
+            toolchains = use_cpp_toolchain(),
+        ),
+    },
     fragments = [] if bazel_features.cc.swift_fragment_removed else ["swift"],
     toolchains = use_cpp_toolchain(),
     implementation = _swift_toolchain_impl,

swift/toolchains/xcode_swift_toolchain.bzl

@@ -25,7 +25,7 @@ load("@bazel_skylib//lib:paths.bzl", "paths")
 load("@bazel_skylib//rules:common_settings.bzl", "BuildSettingInfo")
 load(
     "@bazel_tools//tools/cpp:toolchain_utils.bzl",
-    "find_cpp_toolchain",
+    "CPP_TOOLCHAIN_TYPE",
     "use_cpp_toolchain",
 )
 load("@rules_cc//cc/common:cc_common.bzl", "cc_common")
@@ -626,7 +626,7 @@ def _dsym_provider(*, ctx):
 def _xcode_swift_toolchain_impl(ctx):
     cpp_fragment = ctx.fragments.cpp
     apple_toolchain = apple_common.apple_toolchain()
-    cc_toolchain = find_cpp_toolchain(ctx)
+    cc_toolchain = ctx.exec_groups["default"].toolchains[CPP_TOOLCHAIN_TYPE].cc
 
     target_triple = ctx.var.get("CC_TARGET_TRIPLE") or target_triples.normalize_for_swift(
         target_triples.parse(cc_toolchain.target_gnu_system_name),
@@ -998,6 +998,14 @@ for incremental compilation using a persistent mode.
         },
     ),
     doc = "Represents a Swift compiler toolchain provided by Xcode.",
+    exec_groups = {
+        # An execution group that has no specific platform requirements. This
+        # ensures that the execution platform of this Swift toolchain does not
+        # unnecessarily constrain the execution platform of the C++ toolchain.
+        "default": exec_group(
+            toolchains = use_cpp_toolchain(),
+        ),
+    },
     fragments = [
         "cpp",
         "objc",

test/fixtures/linking/fake_framework.bzl

@@ -2,7 +2,7 @@
 
 load(
     "@bazel_tools//tools/cpp:toolchain_utils.bzl",
-    "find_cpp_toolchain",
+    "CPP_TOOLCHAIN_TYPE",
     "use_cpp_toolchain",
 )
 load("@rules_cc//cc/common:cc_common.bzl", "cc_common")
@@ -15,7 +15,7 @@ def _impl(ctx):
     binary2 = ctx.actions.declare_file("framework2.framework/framework2")
     ctx.actions.write(binary2, "empty")
 
-    cc_toolchain = find_cpp_toolchain(ctx)
+    cc_toolchain = ctx.exec_groups["default"].toolchains[CPP_TOOLCHAIN_TYPE].cc
     feature_configuration = cc_common.configure_features(
         ctx = ctx,
         cc_toolchain = cc_toolchain,
@@ -56,6 +56,14 @@ fake_framework = rule(
             doc = "The C++ toolchain to use.",
         ),
     },
+    exec_groups = {
+        # An execution group that has no specific platform requirements. This
+        # ensures that the execution platform of this Swift toolchain does not
+        # unnecessarily constrain the execution platform of the C++ toolchain.
+        "default": exec_group(
+            toolchains = use_cpp_toolchain(),
+        ),
+    },
     toolchains = use_cpp_toolchain(),
     fragments = ["cpp"],
 )