Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(87)

Unified Diff: pkg/kernel/test/treeshaker_bench.dart

Issue 2918593003: Pass ClassHierarchy instead of creating it. (Closed)
Patch Set: Created 3 years, 7 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: pkg/kernel/test/treeshaker_bench.dart
diff --git a/pkg/kernel/test/treeshaker_bench.dart b/pkg/kernel/test/treeshaker_bench.dart
index 3b2fd090cc4f08cac61ba73c6005c9c74daba75c..07f660ef9597d8f315be0dd577d3af4304c61ba2 100644
--- a/pkg/kernel/test/treeshaker_bench.dart
+++ b/pkg/kernel/test/treeshaker_bench.dart
@@ -58,8 +58,8 @@ void main(List<String> args) {
var watch = new Stopwatch()..start();
ClassHierarchy sharedClassHierarchy = buildClassHierarchy();
int coldHierarchyTime = watch.elapsedMicroseconds;
- var shaker = new TreeShaker(coreTypes, program,
- hierarchy: sharedClassHierarchy, strongMode: strongMode);
+ var shaker = new TreeShaker(coreTypes, sharedClassHierarchy, program,
+ strongMode: strongMode);
if (options['diagnose']) {
print(shaker.getDiagnosticString());
}
@@ -80,8 +80,7 @@ void main(List<String> args) {
watch.reset();
var hierarchy = getClassHierarchy();
hotHierarchyTime += watch.elapsedMicroseconds;
- new TreeShaker(coreTypes, program,
- hierarchy: hierarchy, strongMode: strongMode);
+ new TreeShaker(coreTypes, hierarchy, program, strongMode: strongMode);
hotTreeShakingTime += watch.elapsedMicroseconds;
}
hotHierarchyTime ~/= numberOfTrials;

Powered by Google App Engine
This is Rietveld 408576698