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

Unified Diff: src/compiler/pipeline.cc

Issue 2769743002: [wasm] Allow --turbo-stats to collect wasm compilation info. (Closed)
Patch Set: Created 3 years, 9 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
« no previous file with comments | « no previous file | src/flag-definitions.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/compiler/pipeline.cc
diff --git a/src/compiler/pipeline.cc b/src/compiler/pipeline.cc
index c9a6a897d16bf1007e4788246c683e14284a8917..76cba95f35e0f5a6a3e243bb0d596972d0fcf274 100644
--- a/src/compiler/pipeline.cc
+++ b/src/compiler/pipeline.cc
@@ -115,6 +115,7 @@ class PipelineData {
// For WASM compile entry point.
PipelineData(ZoneStats* zone_stats, CompilationInfo* info, JSGraph* jsgraph,
+ PipelineStatistics* pipeline_statistics,
SourcePositionTable* source_positions,
ZoneVector<trap_handler::ProtectedInstructionData>*
protected_instructions)
@@ -122,6 +123,7 @@ class PipelineData {
info_(info),
debug_name_(info_->GetDebugName()),
zone_stats_(zone_stats),
+ pipeline_statistics_(pipeline_statistics),
graph_zone_scope_(zone_stats_, ZONE_NAME),
graph_(jsgraph->graph()),
source_positions_(source_positions),
@@ -648,7 +650,9 @@ class PipelineWasmCompilationJob final : public CompilationJob {
: CompilationJob(info->isolate(), info, "TurboFan",
State::kReadyToExecute),
zone_stats_(info->isolate()->allocator()),
- data_(&zone_stats_, info, jsgraph, source_positions, protected_insts),
+ pipeline_statistics_(CreatePipelineStatistics(info, &zone_stats_)),
+ data_(&zone_stats_, info, jsgraph, pipeline_statistics_.get(),
+ source_positions, protected_insts),
pipeline_(&data_),
linkage_(descriptor),
allow_signalling_nan_(allow_signalling_nan) {}
@@ -660,6 +664,7 @@ class PipelineWasmCompilationJob final : public CompilationJob {
private:
ZoneStats zone_stats_;
+ std::unique_ptr<PipelineStatistics> pipeline_statistics_;
Mircea Trofin 2017/03/22 18:12:21 can we move this guy up in CompilationJob?
bradn 2017/03/22 18:46:45 Ugh. If you mean CompilationJob, that's tricky as
PipelineData data_;
PipelineImpl pipeline_;
Linkage linkage_;
« no previous file with comments | « no previous file | src/flag-definitions.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698