diff options
author | 2018-01-09 07:02:46 -0800 | |
---|---|---|
committer | 2018-01-09 07:04:10 -0800 | |
commit | 91c82b34171542e6c45a9e81dddaf9db0bc185d4 (patch) | |
tree | 3b0fe39895b13b7b49cc76f867bfb429a92f775b /src/main/java | |
parent | 9c21b5cf530b7d2a2070a11465ca22e79d6d8729 (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.java | 12 |
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) { |