diff options
Diffstat (limited to 'src/main/java/com')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java | 13 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CcExecutionDynamicLibraries.java (renamed from src/main/java/com/google/devtools/build/lib/rules/cpp/CcExecutionDynamicLibrariesInfo.java) | 12 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibrary.java | 4 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingHelper.java | 20 | ||||
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingInfo.java | 22 |
5 files changed, 34 insertions, 37 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java index 80b3f0e379..647fa911eb 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcBinary.java @@ -901,8 +901,8 @@ public abstract class CcBinary implements RuleConfiguredTargetFactory { ccCompilationInfoBuilder.setCcCompilationContext(ccCompilationContext); CcLinkingInfo.Builder ccLinkingInfoBuilder = CcLinkingInfo.Builder.create(); - ccLinkingInfoBuilder.setCcExecutionDynamicLibrariesInfo( - new CcExecutionDynamicLibrariesInfo( + ccLinkingInfoBuilder.setCcExecutionDynamicLibraries( + new CcExecutionDynamicLibraries( collectExecutionDynamicLibraryArtifacts( ruleContext, linkingOutputs.getExecutionDynamicLibraries()))); @@ -946,11 +946,10 @@ public abstract class CcBinary implements RuleConfiguredTargetFactory { NestedSetBuilder<Artifact> builder = NestedSetBuilder.stableOrder(); for (CcLinkingInfo ccLinkingInfo : ruleContext.getPrerequisites("deps", Mode.TARGET, CcLinkingInfo.PROVIDER)) { - CcExecutionDynamicLibrariesInfo ccExecutionDynamicLibrariesInfo = - ccLinkingInfo.getCcExecutionDynamicLibrariesInfo(); - if (ccExecutionDynamicLibrariesInfo != null) { - builder.addTransitive( - ccExecutionDynamicLibrariesInfo.getExecutionDynamicLibraryArtifacts()); + CcExecutionDynamicLibraries ccExecutionDynamicLibraries = + ccLinkingInfo.getCcExecutionDynamicLibraries(); + if (ccExecutionDynamicLibraries != null) { + builder.addTransitive(ccExecutionDynamicLibraries.getExecutionDynamicLibraryArtifacts()); } } diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcExecutionDynamicLibrariesInfo.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcExecutionDynamicLibraries.java index a40c387279..fbf52685eb 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcExecutionDynamicLibrariesInfo.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcExecutionDynamicLibraries.java @@ -22,22 +22,22 @@ import com.google.devtools.build.lib.skyframe.serialization.autocodec.AutoCodec; import com.google.devtools.build.lib.skylarkinterface.SkylarkModule; import com.google.devtools.build.lib.skylarkinterface.SkylarkModuleCategory; -/** A target that provides the execution-time dynamic libraries of a C++ rule. */ +/** An object that contains the execution-time dynamic libraries of a C++ rule. */ @Immutable @AutoCodec @SkylarkModule( - name = "cc_execution_dynamic_libraries_info ", + name = "cc_execution_dynamic_libraries ", documented = false, category = SkylarkModuleCategory.PROVIDER, doc = "." ) -public final class CcExecutionDynamicLibrariesInfo { - public static final CcExecutionDynamicLibrariesInfo EMPTY = - new CcExecutionDynamicLibrariesInfo(NestedSetBuilder.<Artifact>emptySet(Order.STABLE_ORDER)); +public final class CcExecutionDynamicLibraries { + public static final CcExecutionDynamicLibraries EMPTY = + new CcExecutionDynamicLibraries(NestedSetBuilder.<Artifact>emptySet(Order.STABLE_ORDER)); private final NestedSet<Artifact> ccExecutionDynamicLibraries; - public CcExecutionDynamicLibrariesInfo(NestedSet<Artifact> ccExecutionDynamicLibraries) { + public CcExecutionDynamicLibraries(NestedSet<Artifact> ccExecutionDynamicLibraries) { this.ccExecutionDynamicLibraries = ccExecutionDynamicLibraries; } diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibrary.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibrary.java index 245af2ee14..e8df81b7bb 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibrary.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLibrary.java @@ -375,8 +375,8 @@ public abstract class CcLibrary implements RuleConfiguredTargetFactory { (CcLinkingInfo) linkingInfo.getProviders().getProvider(CcLinkingInfo.PROVIDER.getKey()); CcLinkingInfo.Builder ccLinkingInfoBuilder = CcLinkingInfo.Builder.create(); ccLinkingInfoBuilder.setCcLinkParamsStore(ccLinkingInfo.getCcLinkParamsStore()); - ccLinkingInfoBuilder.setCcExecutionDynamicLibrariesInfo( - ccLinkingInfo.getCcExecutionDynamicLibrariesInfo()); + ccLinkingInfoBuilder.setCcExecutionDynamicLibraries( + ccLinkingInfo.getCcExecutionDynamicLibraries()); ccLinkingInfoBuilder.setCcRunfiles(new CcRunfiles(staticRunfiles, sharedRunfiles)); return ccLinkingInfoBuilder.build(); diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingHelper.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingHelper.java index 23abb5b4c8..8deebe314f 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingHelper.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingHelper.java @@ -541,7 +541,7 @@ public final class CcLinkingHelper { CcLinkingInfo.Builder ccLinkingInfoBuilder = CcLinkingInfo.Builder.create(); ccLinkingInfoBuilder.setCcRunfiles(new CcRunfiles(cppStaticRunfiles, cppSharedRunfiles)); - ccLinkingInfoBuilder.setCcExecutionDynamicLibrariesInfo( + ccLinkingInfoBuilder.setCcExecutionDynamicLibraries( collectExecutionDynamicLibraryArtifacts(ccLinkingOutputs.getExecutionDynamicLibraries())); CppConfiguration cppConfiguration = ruleContext.getFragment(CppConfiguration.class); @@ -672,26 +672,24 @@ public final class CcLinkingHelper { return result.build(); } - private CcExecutionDynamicLibrariesInfo collectExecutionDynamicLibraryArtifacts( + private CcExecutionDynamicLibraries collectExecutionDynamicLibraryArtifacts( List<LibraryToLink> executionDynamicLibraries) { Iterable<Artifact> artifacts = LinkerInputs.toLibraryArtifacts(executionDynamicLibraries); if (!Iterables.isEmpty(artifacts)) { - return new CcExecutionDynamicLibrariesInfo( - NestedSetBuilder.wrap(Order.STABLE_ORDER, artifacts)); + return new CcExecutionDynamicLibraries(NestedSetBuilder.wrap(Order.STABLE_ORDER, artifacts)); } NestedSetBuilder<Artifact> builder = NestedSetBuilder.stableOrder(); for (CcLinkingInfo dep : AnalysisUtils.getProviders(deps, CcLinkingInfo.PROVIDER)) { - CcExecutionDynamicLibrariesInfo ccExecutionDynamicLibrariesInfo = - dep.getCcExecutionDynamicLibrariesInfo(); - if (ccExecutionDynamicLibrariesInfo != null) { - builder.addTransitive( - ccExecutionDynamicLibrariesInfo.getExecutionDynamicLibraryArtifacts()); + CcExecutionDynamicLibraries ccExecutionDynamicLibraries = + dep.getCcExecutionDynamicLibraries(); + if (ccExecutionDynamicLibraries != null) { + builder.addTransitive(ccExecutionDynamicLibraries.getExecutionDynamicLibraryArtifacts()); } } return builder.isEmpty() - ? CcExecutionDynamicLibrariesInfo.EMPTY - : new CcExecutionDynamicLibrariesInfo(builder.build()); + ? CcExecutionDynamicLibraries.EMPTY + : new CcExecutionDynamicLibraries(builder.build()); } /** diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingInfo.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingInfo.java index a331e7025e..840c32d702 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingInfo.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CcLinkingInfo.java @@ -38,18 +38,18 @@ public final class CcLinkingInfo extends NativeInfo { private final CcLinkParamsStore ccLinkParamsStore; private final CcRunfiles ccRunfiles; - private final CcExecutionDynamicLibrariesInfo ccExecutionDynamicLibrariesInfo; + private final CcExecutionDynamicLibraries ccExecutionDynamicLibraries; @AutoCodec.Instantiator @VisibleForSerialization CcLinkingInfo( CcLinkParamsStore ccLinkParamsStore, CcRunfiles ccRunfiles, - CcExecutionDynamicLibrariesInfo ccExecutionDynamicLibrariesInfo) { + CcExecutionDynamicLibraries ccExecutionDynamicLibraries) { super(PROVIDER); this.ccLinkParamsStore = ccLinkParamsStore; this.ccRunfiles = ccRunfiles; - this.ccExecutionDynamicLibrariesInfo = ccExecutionDynamicLibrariesInfo; + this.ccExecutionDynamicLibraries = ccExecutionDynamicLibraries; } public CcLinkParamsStore getCcLinkParamsStore() { @@ -60,15 +60,15 @@ public final class CcLinkingInfo extends NativeInfo { return ccRunfiles; } - public CcExecutionDynamicLibrariesInfo getCcExecutionDynamicLibrariesInfo() { - return ccExecutionDynamicLibrariesInfo; + public CcExecutionDynamicLibraries getCcExecutionDynamicLibraries() { + return ccExecutionDynamicLibraries; } /** A Builder for {@link CcLinkingInfo}. */ public static class Builder { CcLinkParamsStore ccLinkParamsStore; CcRunfiles ccRunfiles; - CcExecutionDynamicLibrariesInfo ccExecutionDynamicLibrariesInfo; + CcExecutionDynamicLibraries ccExecutionDynamicLibraries; public static CcLinkingInfo.Builder create() { return new CcLinkingInfo.Builder(); @@ -86,15 +86,15 @@ public final class CcLinkingInfo extends NativeInfo { return this; } - public Builder setCcExecutionDynamicLibrariesInfo( - CcExecutionDynamicLibrariesInfo ccExecutionDynamicLibrariesInfo) { - Preconditions.checkState(this.ccExecutionDynamicLibrariesInfo == null); - this.ccExecutionDynamicLibrariesInfo = ccExecutionDynamicLibrariesInfo; + public Builder setCcExecutionDynamicLibraries( + CcExecutionDynamicLibraries ccExecutionDynamicLibraries) { + Preconditions.checkState(this.ccExecutionDynamicLibraries == null); + this.ccExecutionDynamicLibraries = ccExecutionDynamicLibraries; return this; } public CcLinkingInfo build() { - return new CcLinkingInfo(ccLinkParamsStore, ccRunfiles, ccExecutionDynamicLibrariesInfo); + return new CcLinkingInfo(ccLinkParamsStore, ccRunfiles, ccExecutionDynamicLibraries); } } } |