diff options
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.java | 22 |
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) { |