diff options
author | Nathan Harmata <nharmata@google.com> | 2015-09-08 20:03:22 +0000 |
---|---|---|
committer | Damien Martin-Guillerez <dmarting@google.com> | 2015-09-08 22:27:47 +0000 |
commit | d4803010a6c07f262e3a326bde9c43f0f29aa5ff (patch) | |
tree | 8efd74d9d96c0460929bbf2b9d1d8ebfa884bb2b /src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java | |
parent | ea2f63c98f4334c7dfe81fc9a0a20fcf4732ed59 (diff) |
Use AutoProfiler in the Bazel codebase.
--
MOS_MIGRATED_REVID=102584924
Diffstat (limited to 'src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java')
-rw-r--r-- | src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java | 27 |
1 files changed, 12 insertions, 15 deletions
diff --git a/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java b/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java index fbbe90b28b..6461e3f085 100644 --- a/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java +++ b/src/main/java/com/google/devtools/build/lib/skyframe/SequencedSkyframeExecutor.java @@ -38,7 +38,7 @@ import com.google.devtools.build.lib.packages.PackageFactory; import com.google.devtools.build.lib.packages.Preprocessor; import com.google.devtools.build.lib.pkgcache.PackageCacheOptions; import com.google.devtools.build.lib.pkgcache.PathPackageLocator; -import com.google.devtools.build.lib.profiler.Profiler; +import com.google.devtools.build.lib.profiler.AutoProfiler; import com.google.devtools.build.lib.skyframe.DirtinessCheckerUtils.BasicFilesystemDirtinessChecker; import com.google.devtools.build.lib.skyframe.DirtinessCheckerUtils.MissingDiffDirtinessChecker; import com.google.devtools.build.lib.skyframe.DirtinessCheckerUtils.UnionDirtinessChecker; @@ -476,22 +476,19 @@ public final class SequencedSkyframeExecutor extends SkyframeExecutor { * recreate them if necessary). */ private void discardAnalysisCache(Collection<ConfiguredTarget> topLevelTargets) { - long startTime = Profiler.nanoTimeMaybe(); - lastAnalysisDiscarded = true; - for (Map.Entry<SkyKey, SkyValue> entry : memoizingEvaluator.getValues().entrySet()) { - if (!entry.getKey().functionName().equals(SkyFunctions.CONFIGURED_TARGET)) { - continue; - } - ConfiguredTargetValue ctValue = (ConfiguredTargetValue) entry.getValue(); - // ctValue may be null if target was not successfully analyzed. - if (ctValue != null && !topLevelTargets.contains(ctValue.getConfiguredTarget())) { - ctValue.clear(); + try (AutoProfiler p = AutoProfiler.logged("discarding analysis cache", LOG)) { + lastAnalysisDiscarded = true; + for (Map.Entry<SkyKey, SkyValue> entry : memoizingEvaluator.getValues().entrySet()) { + if (!entry.getKey().functionName().equals(SkyFunctions.CONFIGURED_TARGET)) { + continue; + } + ConfiguredTargetValue ctValue = (ConfiguredTargetValue) entry.getValue(); + // ctValue may be null if target was not successfully analyzed. + if (ctValue != null && !topLevelTargets.contains(ctValue.getConfiguredTarget())) { + ctValue.clear(); + } } } - long duration = Profiler.nanoTimeMaybe() - startTime; - if (duration > 0) { - LOG.info("Spent " + (duration / 1000 / 1000) + " ms discarding analysis cache"); - } } @Override |