Fix merge conflicts and remove const values gating

Author: luispadron

swift/internal/compiling.bzl

@@ -49,7 +49,6 @@ load(
     "SWIFT_FEATURE_USE_EXPLICIT_SWIFT_MODULE_MAP",
     "SWIFT_FEATURE_VFSOVERLAY",
     "SWIFT_FEATURE__NUM_THREADS_0_IN_SWIFTCOPTS",
-    "SWIFT_FEATURE__SUPPORTS_CONST_VALUE_EXTRACTION",
     "SWIFT_FEATURE__WMO_IN_SWIFTCOPTS",
 )
 load(
@@ -492,13 +491,7 @@ def compile(
         feature_name = SWIFT_FEATURE_EMIT_SWIFTSOURCEINFO,
     )
 
-    if is_feature_enabled(
-        feature_configuration = feature_configuration,
-        feature_name = SWIFT_FEATURE__SUPPORTS_CONST_VALUE_EXTRACTION,
-    ):
-        const_protocols_to_gather_file = swift_toolchain.const_protocols_to_gather
-    else:
-        const_protocols_to_gather_file = []
+    const_protocols_to_gather_file = swift_toolchain.const_protocols_to_gather
 
     compile_outputs, other_outputs = _declare_compile_outputs(
         srcs = srcs,

swift/internal/feature_names.bzl

@@ -336,11 +336,6 @@ SWIFT_FEATURE_EMIT_SWIFTSOURCEINFO = "swift.emit_swiftsourceinfo"
 # multiple frameworks with different target name, but same module name.
 SWIFT_FEATURE_ADD_TARGET_NAME_TO_OUTPUT = "swift.add_target_name_to_output"
 
-# A private feature that is set by the toolchain if it supports constant values
-# extraction (Swift 5.8 and above). Users should never manually enable, disable, or query this
-# feature.
-SWIFT_FEATURE__SUPPORTS_CONST_VALUE_EXTRACTION = "swift._supports_const_value_extraction"
-
 # Enable thin LTO and update output-file-map correctly
 SWIFT_FEATURE_THIN_LTO = "swift.thin_lto"
 

swift/internal/swift_autoconfiguration.bzl

@@ -40,7 +40,6 @@ load(
     "SWIFT_FEATURE_USE_AUTOLINK_EXTRACT",
     "SWIFT_FEATURE_USE_MODULE_WRAP",
     "SWIFT_FEATURE_USE_OLD_DRIVER",
-    "SWIFT_FEATURE__SUPPORTS_MACROS",
 )
 
 def _scratch_file(repository_ctx, temp_dir, name, content = ""):
@@ -259,10 +258,6 @@ def _create_linux_toolchain(repository_ctx):
     feature_values.append(SWIFT_FEATURE_USE_AUTOLINK_EXTRACT)
     feature_values.append(SWIFT_FEATURE_USE_MODULE_WRAP)
 
-    swift_features_config = _fetch_supported_features(repository_ctx, path_to_swiftc)
-    if "load-plugin-executable" in swift_features_config:
-        feature_values.append(SWIFT_FEATURE__SUPPORTS_MACROS)
-
     repository_ctx.file(
         "BUILD",
         """

swift/swift_compiler_plugin.bzl

@@ -24,7 +24,6 @@ load(
 load(
     "@build_bazel_rules_swift//swift/internal:feature_names.bzl",
     "SWIFT_FEATURE_ADD_TARGET_NAME_TO_OUTPUT",
-    "SWIFT_FEATURE__SUPPORTS_MACROS",
 )
 load("//swift/internal:features.bzl", "is_feature_enabled")
 load(
@@ -59,12 +58,6 @@ def _swift_compiler_plugin_impl(ctx):
         unsupported_features = ctx.disabled_features,
     )
 
-    if not is_feature_enabled(
-        feature_configuration = feature_configuration,
-        feature_name = SWIFT_FEATURE__SUPPORTS_MACROS,
-    ):
-        fail("Swift compiler plugins require Swift 5.9+")
-
     deps = ctx.attr.deps
     srcs = ctx.files.srcs
     module_contexts = []

swift/toolchains/config/compile_config.bzl

@@ -75,7 +75,6 @@ load(
     "SWIFT_FEATURE_USE_PCH_OUTPUT_DIR",
     "SWIFT_FEATURE_VFSOVERLAY",
     "SWIFT_FEATURE__NUM_THREADS_0_IN_SWIFTCOPTS",
-    "SWIFT_FEATURE__SUPPORTS_CONST_VALUE_EXTRACTION",
     "SWIFT_FEATURE__WMO_IN_SWIFTCOPTS",
 )
 load(":action_config.bzl", "ActionConfigInfo", "ConfigResultInfo", "add_arg")
@@ -253,7 +252,6 @@ def compile_action_configs(
         ActionConfigInfo(
             actions = [SWIFT_ACTION_COMPILE],
             configurators = [_constant_value_extraction_configurator],
-            features = [SWIFT_FEATURE__SUPPORTS_CONST_VALUE_EXTRACTION],
         ),
 
         # Link Time Optimization (LTO).

swift/toolchains/xcode_swift_toolchain.bzl

@@ -47,9 +47,10 @@ load(
 load("//swift/internal:attrs.bzl", "swift_toolchain_driver_attrs")
 load(
     "//swift/internal:feature_names.bzl",
+    "SWIFT_FEATURE__FORCE_ALWAYSLINK_TRUE",
     "SWIFT_FEATURE_CACHEABLE_SWIFTMODULES",
-    "SWIFT_FEATURE_COVERAGE",
     "SWIFT_FEATURE_COVERAGE_PREFIX_MAP",
+    "SWIFT_FEATURE_COVERAGE",
     "SWIFT_FEATURE_DEBUG_PREFIX_MAP",
     "SWIFT_FEATURE_DISABLE_SWIFT_SANDBOX",
     "SWIFT_FEATURE_DISABLE_SYSTEM_INDEX",
@@ -62,8 +63,8 @@ load(
     "SWIFT_FEATURE_OBJC_LINK_FLAGS",
     "SWIFT_FEATURE_OPT_USES_WMO",
     "SWIFT_FEATURE_REMAP_XCODE_PATH",
+    "SWIFT_FEATURE_SUPPORTS_BARE_SLASH_REGEX",
     "SWIFT_FEATURE_USE_GLOBAL_MODULE_CACHE",
-    "SWIFT_FEATURE__FORCE_ALWAYSLINK_TRUE",
 )
 load("//swift/internal:features.bzl", "features_for_build_modes")
 load("//swift/internal:target_triples.bzl", "target_triples")
@@ -695,14 +696,16 @@ def _xcode_swift_toolchain_impl(ctx):
         SWIFT_FEATURE_CACHEABLE_SWIFTMODULES,
         SWIFT_FEATURE_COVERAGE_PREFIX_MAP,
         SWIFT_FEATURE_DEBUG_PREFIX_MAP,
+        SWIFT_FEATURE_DISABLE_SYSTEM_INDEX,
         SWIFT_FEATURE_EMIT_SWIFTDOC,
         SWIFT_FEATURE_EMIT_SWIFTSOURCEINFO,
         SWIFT_FEATURE_ENABLE_BATCH_MODE,
         SWIFT_FEATURE_ENABLE_SKIP_FUNCTION_BODIES,
-        SWIFT_FEATURE_DISABLE_SYSTEM_INDEX,
+        SWIFT_FEATURE_FILE_PREFIX_MAP,
         SWIFT_FEATURE_OBJC_LINK_FLAGS,
         SWIFT_FEATURE_OPT_USES_WMO,
         SWIFT_FEATURE_REMAP_XCODE_PATH,
+        SWIFT_FEATURE_SUPPORTS_BARE_SLASH_REGEX,
         SWIFT_FEATURE_USE_GLOBAL_MODULE_CACHE,
     ])
 

test/const_values_tests.bzl

@@ -26,14 +26,6 @@ load(
 
 const_values_test = make_action_command_line_test_rule()
 
-no_const_values_test = make_action_command_line_test_rule(
-    config_settings = {
-        "//command_line_option:features": [
-            "-swift._supports_const_value_extraction",
-        ],
-    },
-)
-
 const_values_wmo_test = make_provider_test_rule(
     config_settings = {
         str(Label("@build_bazel_rules_swift//swift:copt")): [
@@ -98,19 +90,6 @@ def const_values_test_suite(name, tags = []):
         target_under_test = "@build_bazel_rules_swift//test/fixtures/basic:first",
     )
 
-    no_const_values_test(
-        name = "{}_not_expected_argv".format(name),
-        not_expected_argv = [
-            "-Xfrontend -const-gather-protocols-file",
-            "-Xfrontend swift/toolchains/config/const_protocols_to_gather.json",
-            "-emit-const-values-path",
-            "first.swift.swiftconstvalues",
-        ],
-        mnemonic = "SwiftCompile",
-        tags = all_tags,
-        target_under_test = "@build_bazel_rules_swift//test/fixtures/basic:first",
-    )
-
     native.test_suite(
         name = name,
         tags = all_tags,