diff options
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/rules/cpp')
12 files changed, 1 insertions, 79 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java index d1f0e4b8f6..e8356092ce 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileAction.java @@ -1067,11 +1067,6 @@ public class CppCompileAction extends AbstractAction return context.getDeclaredIncludeSrcs(); } - @Override - public ResourceSet estimateResourceConsumption(Executor executor) { - return executor.getContext(actionContext).estimateResourceConsumption(this); - } - @VisibleForTesting public Class<? extends CppCompileActionContext> getActionContext() { return actionContext; diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionContext.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionContext.java index 192a789c24..903829275c 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionContext.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppCompileActionContext.java @@ -19,7 +19,6 @@ import com.google.devtools.build.lib.actions.ActionExecutionException; import com.google.devtools.build.lib.actions.Artifact; import com.google.devtools.build.lib.actions.ExecException; import com.google.devtools.build.lib.actions.Executor.ActionContext; -import com.google.devtools.build.lib.actions.ResourceSet; import java.io.IOException; import javax.annotation.Nullable; @@ -44,7 +43,7 @@ public interface CppCompileActionContext extends ActionContext { * <p>Returns null if additional inputs will only be found during action execution, not before. */ @Nullable - public Iterable<Artifact> findAdditionalInputs( + Iterable<Artifact> findAdditionalInputs( CppCompileAction action, ActionExecutionContext actionExecutionContext, IncludeProcessing includeProcessing) @@ -61,9 +60,4 @@ public interface CppCompileActionContext extends ActionContext { */ @Nullable Reply getReplyFromException( ExecException e, CppCompileAction action); - - /** - * Returns the estimated resource consumption of the action. - */ - ResourceSet estimateResourceConsumption(CppCompileAction action); } diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkAction.java index 2082192068..0fbd2d05ec 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkAction.java @@ -485,11 +485,6 @@ public final class CppLinkAction extends AbstractAction return (isLTOIndexing ? "LTO indexing " : "Linking ") + linkOutput.prettyPrint(); } - @Override - public ResourceSet estimateResourceConsumption(Executor executor) { - return executor.getContext(CppLinkActionContext.class).estimateResourceConsumption(this); - } - /** * Estimate the resources consumed when this action is run locally. */ diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkActionContext.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkActionContext.java index cf64db187f..bf0bd3e9e2 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkActionContext.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppLinkActionContext.java @@ -18,19 +18,12 @@ import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionExecutionException; import com.google.devtools.build.lib.actions.ExecException; import com.google.devtools.build.lib.actions.Executor.ActionContext; -import com.google.devtools.build.lib.actions.ResourceSet; /** * Context for executing {@link CppLinkAction}s. */ @ActionContextMarker(name = "C++ link") public interface CppLinkActionContext extends ActionContext { - - /** - * Returns the estimated resource consumption of the action. - */ - ResourceSet estimateResourceConsumption(CppLinkAction action); - /** * Executes the specified action. */ diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModuleMapAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModuleMapAction.java index 817ee99c60..a99751a8b1 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModuleMapAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CppModuleMapAction.java @@ -21,9 +21,7 @@ import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionOwner; import com.google.devtools.build.lib.actions.Artifact; import com.google.devtools.build.lib.actions.Artifact.ArtifactExpander; -import com.google.devtools.build.lib.actions.ResourceSet; import com.google.devtools.build.lib.analysis.actions.AbstractFileWriteAction; -import com.google.devtools.build.lib.analysis.actions.AbstractFileWriteAction.DeterministicWriter; import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; import com.google.devtools.build.lib.util.Fingerprint; import com.google.devtools.build.lib.vfs.PathFragment; @@ -223,11 +221,6 @@ public final class CppModuleMapAction extends AbstractFileWriteAction { return f.hexDigestAndReset(); } - @Override - public ResourceSet estimateResourceConsumptionLocal() { - return ResourceSet.createWithRamCpuIo(/*memoryMb=*/0, /*cpuUsage=*/0, /*ioUsage=*/0.02); - } - @VisibleForTesting public Collection<Artifact> getPublicHeaders() { return publicHeaders; diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/CreateIncSymlinkAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/CreateIncSymlinkAction.java index 443787863a..ee8e92b2ef 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/CreateIncSymlinkAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/CreateIncSymlinkAction.java @@ -23,8 +23,6 @@ import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionExecutionException; import com.google.devtools.build.lib.actions.ActionOwner; import com.google.devtools.build.lib.actions.Artifact; -import com.google.devtools.build.lib.actions.Executor; -import com.google.devtools.build.lib.actions.ResourceSet; import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; import com.google.devtools.build.lib.util.Fingerprint; import com.google.devtools.build.lib.vfs.FileSystemUtils; @@ -82,15 +80,6 @@ public final class CreateIncSymlinkAction extends AbstractAction { } @Override - public ResourceSet estimateResourceConsumption(Executor executor) { - // We're mainly doing I/O, so CPU usage should be very low; most of the - // time we'll be blocked waiting for the OS. - // The only exception is the fingerprint digest calculation for the stamp - // file contents. - return ResourceSet.createWithRamCpuIo(/*memoryMb=*/0, /*cpuUsage=*/0.005, /*ioUsage=*/0.0); - } - - @Override public String computeKey() { Fingerprint key = new Fingerprint(); for (Map.Entry<Artifact, Artifact> entry : symlinks.entrySet()) { diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/ExtractInclusionAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/ExtractInclusionAction.java index 0a3d428bb8..6a16ea78e3 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/ExtractInclusionAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/ExtractInclusionAction.java @@ -22,9 +22,7 @@ import com.google.devtools.build.lib.actions.ActionOwner; import com.google.devtools.build.lib.actions.Artifact; import com.google.devtools.build.lib.actions.ExecException; import com.google.devtools.build.lib.actions.Executor; -import com.google.devtools.build.lib.actions.ResourceSet; import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; - import java.io.IOException; /** @@ -67,11 +65,6 @@ final class ExtractInclusionAction extends AbstractAction { } @Override - public ResourceSet estimateResourceConsumption(Executor executor) { - return ResourceSet.ZERO; - } - - @Override public void execute(ActionExecutionContext actionExecutionContext) throws ActionExecutionException, InterruptedException { Executor executor = actionExecutionContext.getExecutor(); diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java index cf57fd67d3..7df15c067d 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/FakeCppCompileAction.java @@ -269,9 +269,4 @@ public class FakeCppCompileAction extends CppCompileAction { public ResourceSet estimateResourceConsumptionLocal() { return ResourceSet.createWithRamCpuIo(/*memoryMb=*/1, /*cpuUsage=*/0.1, /*ioUsage=*/0.0); } - - @Override - public ResourceSet estimateResourceConsumption(Executor executor) { - return executor.getContext(actionContext).estimateResourceConsumption(this); - } } diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoStubAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoStubAction.java index ee2f194f03..be9624625b 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoStubAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/FdoStubAction.java @@ -18,8 +18,6 @@ import com.google.devtools.build.lib.actions.AbstractAction; import com.google.devtools.build.lib.actions.ActionExecutionContext; import com.google.devtools.build.lib.actions.ActionOwner; import com.google.devtools.build.lib.actions.Artifact; -import com.google.devtools.build.lib.actions.Executor; -import com.google.devtools.build.lib.actions.ResourceSet; import com.google.devtools.build.lib.concurrent.ThreadSafety.Immutable; import com.google.devtools.build.lib.vfs.Path; @@ -59,11 +57,6 @@ public final class FdoStubAction extends AbstractAction { } @Override - public ResourceSet estimateResourceConsumption(Executor executor) { - return ResourceSet.ZERO; - } - - @Override public void prepare(Path execRoot) { // The superclass would delete the output files here. We can't let that happen, since this // action does not in fact create those files; it is only a placeholder and the actual files diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java index 98db892c9a..c2bca35809 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/SolibSymlinkAction.java @@ -22,8 +22,6 @@ import com.google.devtools.build.lib.actions.ActionExecutionException; import com.google.devtools.build.lib.actions.ActionOwner; import com.google.devtools.build.lib.actions.Actions; import com.google.devtools.build.lib.actions.Artifact; -import com.google.devtools.build.lib.actions.Executor; -import com.google.devtools.build.lib.actions.ResourceSet; import com.google.devtools.build.lib.actions.Root; import com.google.devtools.build.lib.analysis.RuleContext; import com.google.devtools.build.lib.analysis.config.BuildConfiguration; @@ -90,11 +88,6 @@ public final class SolibSymlinkAction extends AbstractAction { } @Override - public ResourceSet estimateResourceConsumption(Executor executor) { - return ResourceSet.ZERO; - } - - @Override protected String computeKey() { Fingerprint f = new Fingerprint(); f.addPath(symlink.getPath()); diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnGccStrategy.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnGccStrategy.java index 40f3194828..7ec2b85121 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnGccStrategy.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnGccStrategy.java @@ -76,11 +76,6 @@ public class SpawnGccStrategy implements CppCompileActionContext { } @Override - public ResourceSet estimateResourceConsumption(CppCompileAction action) { - return action.estimateResourceConsumptionLocal(); - } - - @Override public Reply getReplyFromException(ExecException e, CppCompileAction action) { return null; } diff --git a/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnLinkStrategy.java b/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnLinkStrategy.java index c88a3c6447..9360d2a185 100644 --- a/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnLinkStrategy.java +++ b/src/main/java/com/google/devtools/build/lib/rules/cpp/SpawnLinkStrategy.java @@ -20,7 +20,6 @@ import com.google.devtools.build.lib.actions.BaseSpawn; import com.google.devtools.build.lib.actions.ExecException; import com.google.devtools.build.lib.actions.ExecutionStrategy; import com.google.devtools.build.lib.actions.Executor; -import com.google.devtools.build.lib.actions.ResourceSet; import com.google.devtools.build.lib.actions.Spawn; import com.google.devtools.build.lib.actions.SpawnActionContext; @@ -47,9 +46,4 @@ public final class SpawnLinkStrategy implements CppLinkActionContext { action.estimateResourceConsumptionLocal()); spawnActionContext.exec(spawn, actionExecutionContext); } - - @Override - public ResourceSet estimateResourceConsumption(CppLinkAction action) { - return action.estimateResourceConsumptionLocal(); - } } |