aboutsummaryrefslogtreecommitdiffhomepage
path: root/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java
diff options
context:
space:
mode:
authorGravatar Benjamin Peterson <bp@benjamin.pe>2018-06-28 08:16:50 -0700
committerGravatar Copybara-Service <copybara-piper@google.com>2018-06-28 08:18:23 -0700
commit384d4549f97cfb3fcec40a6fd809a574f7df59fe (patch)
tree89ede3766d7b0b585dd59f40b18651838d293f21 /src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java
parent817e32900687e05a000d31f8b7596c49693f7b88 (diff)
Delete some more LIPO crumbs.
Closes #5478. Change-Id: I3ac44605ef16a7c2e6bdc63d26fdf968bef651aa PiperOrigin-RevId: 202482493
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java')
-rw-r--r--src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java22
1 files changed, 2 insertions, 20 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java
index fda27a7f16..9254e3f9ac 100644
--- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java
+++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcCompilationContext.java
@@ -128,8 +128,8 @@ public final class CcCompilationContext implements CcCompilationContextApi {
* Builder#mergeDependentCcCompilationContexts(Iterable)}, they are consolidated into a single
* middleman Artifact when {@link Builder#build()} is called.
*
- * <p>The returned set can be empty if there are no prerequisites. Usually it contains a single
- * middleman, but if LIPO is used there can be two.
+ * <p>The returned set can be empty if there are no prerequisites. Usually, it contains a single
+ * middleman.
*/
public ImmutableSet<Artifact> getTransitiveCompilationPrerequisites() {
return compilationPrerequisites;
@@ -426,8 +426,6 @@ public final class CcCompilationContext implements CcCompilationContextApi {
* headers.
*/
public Builder addCompilationPrerequisites(Iterable<Artifact> prerequisites) {
- // LIPO collector must not add compilation prerequisites in order to avoid
- // the creation of a middleman action.
for (Artifact prerequisite : prerequisites) {
String basename = prerequisite.getFilename();
Preconditions.checkArgument(!Link.OBJECT_FILETYPES.matches(basename));
@@ -793,22 +791,6 @@ public final class CcCompilationContext implements CcCompilationContextApi {
}
/**
- * Merges a {@link ModuleInfo} into this one. In contrast to addTransitive, this doesn't add
- * the dependent module to transitiveModules, but just merges the transitive sets. The main
- * usage is to merge multiple {@link ModuleInfo} instances for Lipo.
- */
- public Builder merge(ModuleInfo other) {
- if (headerModule == null) {
- headerModule = other.headerModule;
- }
- modularHeaders.addAll(other.modularHeaders);
- textualHeaders.addAll(other.textualHeaders);
- transitiveModules.addTransitive(other.transitiveModules);
- transitiveModuleHeaders.addTransitive(other.transitiveModuleHeaders);
- return this;
- }
-
- /**
* Adds the {@link ModuleInfo} of a dependency and builds up the transitive data structures.
*/
public Builder addTransitive(ModuleInfo moduleInfo) {