Don't create the LIPO context collector subgraph for instrumentation builds.

The consumers of ":lipo_context_collector" only do anything with it in
optimization builds.

Also remove an outdated comment in FdoSupport.

PiperOrigin-RevId: 153748937

1 files changed, 4 insertions, 6 deletions

diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoSupport.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoSupport.java
index ca0ab31cf7..04b998498d 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoSupport.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoSupport.java
@@ -510,10 +510,6 @@ public class FdoSupport {
       CcToolchainFeatures.Variables.Builder buildVariables, RuleContext ruleContext,
       PathFragment sourceName, PathFragment sourceExecPath, boolean usePic,
       FeatureConfiguration featureConfiguration, FdoSupportProvider fdoSupportProvider) {
-    // It is a bug if this method is called with useLipo if lipo is disabled. However, it is legal
-    // if is is called with !useLipo, even though lipo is enabled.
-    LipoContextProvider lipoInputProvider = CppHelper.getLipoContextProvider(ruleContext);
-    Preconditions.checkArgument(lipoInputProvider == null || lipoMode != LipoMode.OFF);
 
     // FDO is disabled -> do nothing.
     if ((fdoInstrument == null) && (fdoRoot == null)) {
@@ -532,7 +528,7 @@ public class FdoSupport {
         return;
       }
       Iterable<Artifact> auxiliaryInputs = getAuxiliaryInputs(
-          ruleContext, sourceName, sourceExecPath, usePic, lipoInputProvider, fdoSupportProvider);
+          ruleContext, sourceName, sourceExecPath, usePic, fdoSupportProvider);
       builder.addMandatoryInputs(auxiliaryInputs);
       if (!Iterables.isEmpty(auxiliaryInputs)) {
         if (featureConfiguration.isEnabled(CppRuleClasses.AUTOFDO)) {
@@ -557,7 +553,9 @@ public class FdoSupport {
    */
   private Iterable<Artifact> getAuxiliaryInputs(
       RuleContext ruleContext, PathFragment sourceName, PathFragment sourceExecPath, boolean usePic,
-      LipoContextProvider lipoContextProvider, FdoSupportProvider fdoSupportProvider) {
+      FdoSupportProvider fdoSupportProvider) {
+    LipoContextProvider lipoContextProvider = CppHelper.getLipoContextProvider(ruleContext);
+
     // If --fdo_optimize was not specified, we don't have any additional inputs.
     if (fdoProfile == null) {
       return ImmutableSet.of();