From dc7a4b9a23e956dbfe15a0441709bdc85878d4f1 Mon Sep 17 00:00:00 2001 From: ulfjack Date: Mon, 30 Jul 2018 01:43:56 -0700 Subject: Add the ParallelEvaluator.eval call to the Profiler We want to make sure that the profile captures all Skyframe invocations, so that we can catch cases where we accidentally introduce new ones. PiperOrigin-RevId: 206543307 --- .../google/devtools/build/skyframe/InMemoryMemoizingEvaluator.java | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) (limited to 'src/main/java/com/google/devtools/build/skyframe') diff --git a/src/main/java/com/google/devtools/build/skyframe/InMemoryMemoizingEvaluator.java b/src/main/java/com/google/devtools/build/skyframe/InMemoryMemoizingEvaluator.java index ba3812689c..b5c06b7bfb 100644 --- a/src/main/java/com/google/devtools/build/skyframe/InMemoryMemoizingEvaluator.java +++ b/src/main/java/com/google/devtools/build/skyframe/InMemoryMemoizingEvaluator.java @@ -23,6 +23,8 @@ import com.google.common.collect.Maps; import com.google.common.collect.Sets; import com.google.devtools.build.lib.events.Event; import com.google.devtools.build.lib.events.ExtendedEventHandler; +import com.google.devtools.build.lib.profiler.Profiler; +import com.google.devtools.build.lib.profiler.SilentCloseable; import com.google.devtools.build.skyframe.Differencer.Diff; import com.google.devtools.build.skyframe.InvalidatingNodeVisitor.DeletingInvalidationState; import com.google.devtools.build.skyframe.InvalidatingNodeVisitor.DirtyingInvalidationState; @@ -191,7 +193,10 @@ public final class InMemoryMemoizingEvaluator implements MemoizingEvaluator { numThreads, progressReceiver, graphInconsistencyReceiver); - EvaluationResult result = evaluator.eval(roots); + EvaluationResult result; + try (SilentCloseable c = Profiler.instance().profile("ParallelEvaluator.eval")) { + result = evaluator.eval(roots); + } return EvaluationResult.builder() .mergeFrom(result) .setWalkableGraph(new DelegatingWalkableGraph(graph)) -- cgit v1.2.3