From a31b3de76abd37720921aba2689d2ede8d092032 Mon Sep 17 00:00:00 2001 From: Googler Date: Fri, 29 Dec 2023 06:27:59 -0800 Subject: [PATCH] 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 --- .../devtools/build/lib/rules/cpp/LinkCommandLine.java | 11 ----------- 1 file changed, 11 deletions(-) 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 cd2b1143968214..72a38ddbd270b0 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 @@ public Artifact getParamFile() { 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;