Move enabling of coverage related features to CcCommon

Move dealing with coverage related features from CppConfiguration.configurationEnabledFeatures to CcCommon.configureFeatures.
Remove configurationEnabledFeatures.

RELNOTES: None.
PiperOrigin-RevId: 186744803

1 files changed, 0 insertions, 28 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
index 3bec3daa6b..64ba290c29 100644
--- a/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
+++ b/src/main/java/com/google/devtools/build/lib/analysis/config/BuildConfiguration.java
@@ -37,7 +37,6 @@ import com.google.devtools.build.lib.actions.ArtifactRoot;
 import com.google.devtools.build.lib.actions.BuildConfigurationInterface;
 import com.google.devtools.build.lib.analysis.BlazeDirectories;
 import com.google.devtools.build.lib.analysis.ConfiguredRuleClassProvider;
-import com.google.devtools.build.lib.analysis.RuleContext;
 import com.google.devtools.build.lib.analysis.actions.FileWriteAction;
 import com.google.devtools.build.lib.analysis.config.transitions.ComposingPatchTransition;
 import com.google.devtools.build.lib.analysis.config.transitions.PatchTransition;
@@ -199,12 +198,6 @@ public class BuildConfiguration implements BuildConfigurationInterface {
       return ImmutableMap.of();
     }
 
-    /** Return set of features enabled by this configuration. */
-    public ImmutableSet<String> configurationEnabledFeatures(
-        RuleContext ruleContext, ImmutableSet<String> disabledFeatures) {
-      return ImmutableSet.of();
-    }
-
     /**
      * @return false if a Fragment understands that it won't be able to work with a given strategy,
      *     or true otherwise.
@@ -703,23 +696,6 @@ public class BuildConfiguration implements BuildConfigurationInterface {
     public Label coverageReportGenerator;
 
     @Option(
-      name = "experimental_use_llvm_covmap",
-      defaultValue = "false",
-      category = "experimental",
-      documentationCategory = OptionDocumentationCategory.OUTPUT_PARAMETERS,
-      effectTags = {
-          OptionEffectTag.CHANGES_INPUTS,
-          OptionEffectTag.AFFECTS_OUTPUTS,
-          OptionEffectTag.LOADING_AND_ANALYSIS
-      },
-      metadataTags = { OptionMetadataTag.EXPERIMENTAL },
-      help =
-          "If specified, Bazel will generate llvm-cov coverage map information rather than "
-              + "gcov when collect_code_coverage is enabled."
-    )
-    public boolean useLLVMCoverageMapFormat;
-
-    @Option(
       name = "build_runfile_manifests",
       defaultValue = "true",
       category = "strategy",
@@ -1906,10 +1882,6 @@ public class BuildConfiguration implements BuildConfigurationInterface {
     return options.experimentalJavaCoverage;
   }
 
-  public boolean isLLVMCoverageMapFormatEnabled() {
-    return options.useLLVMCoverageMapFormat;
-  }
-
   /** If false, AnalysisEnvironment doesn't register any actions created by the ConfiguredTarget. */
   public boolean isActionsEnabled() {
     return actionsEnabled;