commit a31b3de76abd37720921aba2689d2ede8d092032
author Googler <ilist@google.com> Fri Dec 29 06:27:59 2023 -0800
committer Copybara-Service <copybara-worker@google.com> Fri Dec 29 06:29:42 2023 -0800
tree f0c14168851d7662ded1b595f1e5fb127618eecc
parent 6846317fded0de1e404ae02de822e9188b0866b6

Remove public getFeatureConfiguration from LinkCommandLine

It's used in CppCompileActionTest only. The tests are actually more robust if they check if CppLinkAction actually used pic or not. That pic is used exactly when main.pic.o is an input to linking action.

PiperOrigin-RevId: 594448244
Change-Id: Iaba5c7ae0d3e5bb3199f3b56c625083fbef3a66a

src/main/java/com/google/devtools/build/lib/rules/cpp/LinkCommandLine.java

diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkCommandLine.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkCommandLine.java
index cd2b114..72a38dd 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkCommandLine.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/LinkCommandLine.java
@@ -82,21 +82,10 @@
     return paramFile;
   }
 
-  @Nullable
-  @VisibleForTesting
-  public FeatureConfiguration getFeatureConfiguration() {
-    return featureConfiguration;
-  }
-
   public String getActionName() {
     return actionName;
   }
 
-  /** Returns the current type of link target set. */
-  public LinkTargetType getLinkTargetType() {
-    return linkTargetType;
-  }
-
   /** Returns the "staticness" of the link. */
   public Link.LinkingMode getLinkingMode() {
     return linkingMode;