aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java
diff options
context:
space:
mode:
authorGravatar plf <plf@google.com>2018-01-09 07:02:46 -0800
committerGravatar Copybara-Service <copybara-piper@google.com>2018-01-09 07:04:10 -0800
commit91c82b34171542e6c45a9e81dddaf9db0bc185d4 (patch)
tree3b0fe39895b13b7b49cc76f867bfb429a92f775b /src/main/java
parent9c21b5cf530b7d2a2070a11465ca22e79d6d8729 (diff)
Removes unnecessary check for LinkTargetType.EXECUTABLE.
The precondition check in line 1274 makes sure that it has Staticness.STATIC. EXECUTABLE has Staticness.DYNAMIC which precludes the checks from ever being executed. NO_SQ=Kokoro failing without giving an actual error. RELNOTES:none PiperOrigin-RevId: 181314900
Diffstat (limited to 'src/main/java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java12
1 files changed, 2 insertions, 10 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java
index 587df20b27..80d313792a 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModel.java
@@ -1318,9 +1318,7 @@ public final class CppModel {
.addVariablesExtensions(variablesExtensions)
.build();
env.registerAction(maybePicAction);
- if (linkType != LinkTargetType.EXECUTABLE) {
- result.addStaticLibrary(maybePicAction.getOutputLibrary());
- }
+ result.addStaticLibrary(maybePicAction.getOutputLibrary());
// Create a second static library (.pic.a). Only in case (2) do we need both PIC and non-PIC
// static libraries. In that case, the first static library contains the non-PIC code, and this
@@ -1344,9 +1342,7 @@ public final class CppModel {
.addVariablesExtensions(variablesExtensions)
.build();
env.registerAction(picAction);
- if (linkType != LinkTargetType.EXECUTABLE) {
- result.addPicStaticLibrary(picAction.getOutputLibrary());
- }
+ result.addPicStaticLibrary(picAction.getOutputLibrary());
}
if (!createDynamicLibrary) {
@@ -1457,10 +1453,6 @@ public final class CppModel {
CppLinkAction dynamicLinkAction = dynamicLinkActionBuilder.build();
env.registerAction(dynamicLinkAction);
- if (linkType == LinkTargetType.EXECUTABLE) {
- return result.build();
- }
-
LibraryToLink dynamicLibrary = dynamicLinkAction.getOutputLibrary();
LibraryToLink interfaceLibrary = dynamicLinkAction.getInterfaceOutputLibrary();
if (interfaceLibrary == null) {