Address review comments

Simplify action configuration.
bazelbuild · May 23, 2024 · cbf6320 · cbf6320
1 parent 2b542bf
commit cbf6320
Showing 1 changed file with 1 addition and 7 deletions.
diff --git a/swift/internal/compiling.bzl b/swift/internal/compiling.bzl
@@ -262,11 +262,7 @@ def compile_action_configs(
             configurators = [
                 swift_toolchain_config.add_arg("-emit-bc"),
             ],
-            features = [
-                [SWIFT_FEATURE_EMIT_BC],
-                [SWIFT_FEATURE_FULL_LTO],
-                [SWIFT_FEATURE_THIN_LTO],
-            ],
+            features = [SWIFT_FEATURE_EMIT_BC],
         ),
 
         # Add the single object file or object file map, whichever is needed.
@@ -414,15 +410,13 @@ def compile_action_configs(
                 swift_toolchain_config.add_arg("-lto=llvm-thin"),
             ],
             features = [SWIFT_FEATURE_THIN_LTO],
-            not_features = [SWIFT_FEATURE_FULL_LTO],
         ),
         swift_toolchain_config.action_config(
             actions = [swift_action_names.COMPILE],
             configurators = [
                 swift_toolchain_config.add_arg("-lto=llvm-full"),
             ],
             features = [SWIFT_FEATURE_FULL_LTO],
-            not_features = [SWIFT_FEATURE_THIN_LTO],
         ),
     ]