From 751d53d9d2806600029c85e810fcbe2c22401e4b Mon Sep 17 00:00:00 2001 From: "fschneider@chromium.org" Date: Fri, 2 Mar 2012 13:05:59 +0000 Subject: [PATCH] Make trace-hydrogen output more visualizer-friendly. Review URL: https://chromiumcodereview.appspot.com/9580005 git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@10904 ce2b1a6d-e550-0410-aec6-3dcde31c8c00 --- src/arm/lithium-arm.cc | 4 ++-- src/arm/lithium-codegen-arm.cc | 2 +- src/hydrogen.cc | 28 ++++++++++++++-------------- src/ia32/lithium-codegen-ia32.cc | 2 +- src/ia32/lithium-ia32.cc | 4 ++-- src/lithium-allocator.cc | 16 ++++++++-------- src/mips/lithium-codegen-mips.cc | 2 +- src/mips/lithium-mips.cc | 4 ++-- src/x64/lithium-codegen-x64.cc | 2 +- src/x64/lithium-x64.cc | 4 ++-- 10 files changed, 34 insertions(+), 34 deletions(-) diff --git a/src/arm/lithium-arm.cc b/src/arm/lithium-arm.cc index 8654a3b866..b747ab8155 100644 --- a/src/arm/lithium-arm.cc +++ b/src/arm/lithium-arm.cc @@ -440,7 +440,7 @@ LOperand* LChunk::GetNextSpillSlot(bool is_double) { void LChunk::MarkEmptyBlocks() { - HPhase phase("L Mark empty blocks", this); + HPhase phase("L_Mark empty blocks", this); for (int i = 0; i < graph()->blocks()->length(); ++i) { HBasicBlock* block = graph()->blocks()->at(i); int first = block->first_instruction_index(); @@ -552,7 +552,7 @@ Representation LChunk::LookupLiteralRepresentation( LChunk* LChunkBuilder::Build() { ASSERT(is_unused()); chunk_ = new(zone()) LChunk(info(), graph()); - HPhase phase("L Building chunk", chunk_); + HPhase phase("L_Building chunk", chunk_); status_ = BUILDING; const ZoneList* blocks = graph()->blocks(); for (int i = 0; i < blocks->length(); i++) { diff --git a/src/arm/lithium-codegen-arm.cc b/src/arm/lithium-codegen-arm.cc index d03410bf0c..fcbc90da5f 100644 --- a/src/arm/lithium-codegen-arm.cc +++ b/src/arm/lithium-codegen-arm.cc @@ -62,7 +62,7 @@ class SafepointGenerator : public CallWrapper { #define __ masm()-> bool LCodeGen::GenerateCode() { - HPhase phase("Z Code generation", chunk()); + HPhase phase("Z_Code generation", chunk()); ASSERT(is_unused()); status_ = GENERATING; CpuFeatures::Scope scope1(VFP3); diff --git a/src/hydrogen.cc b/src/hydrogen.cc index ee02b68ec3..ccddd88690 100644 --- a/src/hydrogen.cc +++ b/src/hydrogen.cc @@ -730,7 +730,7 @@ HBasicBlock* HGraph::CreateBasicBlock() { void HGraph::Canonicalize() { if (!FLAG_use_canonicalizing) return; - HPhase phase("H Canonicalize", this); + HPhase phase("H_Canonicalize", this); for (int i = 0; i < blocks()->length(); ++i) { HInstruction* instr = blocks()->at(i)->first(); while (instr != NULL) { @@ -743,7 +743,7 @@ void HGraph::Canonicalize() { void HGraph::OrderBlocks() { - HPhase phase("H Block ordering"); + HPhase phase("H_Block ordering"); BitVector visited(blocks_.length(), zone()); ZoneList reverse_result(8); @@ -805,7 +805,7 @@ void HGraph::Postorder(HBasicBlock* block, void HGraph::AssignDominators() { - HPhase phase("H Assign dominators", this); + HPhase phase("H_Assign dominators", this); for (int i = 0; i < blocks_.length(); ++i) { HBasicBlock* block = blocks_[i]; if (block->IsLoopHeader()) { @@ -824,7 +824,7 @@ void HGraph::AssignDominators() { // Mark all blocks that are dominated by an unconditional soft deoptimize to // prevent code motion across those blocks. void HGraph::PropagateDeoptimizingMark() { - HPhase phase("H Propagate deoptimizing mark", this); + HPhase phase("H_Propagate deoptimizing mark", this); MarkAsDeoptimizingRecursively(entry_block()); } @@ -837,7 +837,7 @@ void HGraph::MarkAsDeoptimizingRecursively(HBasicBlock* block) { } void HGraph::EliminateRedundantPhis() { - HPhase phase("H Redundant phi elimination", this); + HPhase phase("H_Redundant phi elimination", this); // Worklist of phis that can potentially be eliminated. Initialized with // all phi nodes. When elimination of a phi node modifies another phi node @@ -871,7 +871,7 @@ void HGraph::EliminateRedundantPhis() { void HGraph::EliminateUnreachablePhis() { - HPhase phase("H Unreachable phi elimination", this); + HPhase phase("H_Unreachable phi elimination", this); // Initialize worklist. ZoneList phi_list(blocks_.length()); @@ -1010,7 +1010,7 @@ void HRangeAnalysis::TraceRange(const char* msg, ...) { void HRangeAnalysis::Analyze() { - HPhase phase("H Range analysis", graph_); + HPhase phase("H_Range analysis", graph_); Analyze(graph_->entry_block()); } @@ -1831,7 +1831,7 @@ Representation HInferRepresentation::TryChange(HValue* value) { void HInferRepresentation::Analyze() { - HPhase phase("H Infer representations", graph_); + HPhase phase("H_Infer representations", graph_); // (1) Initialize bit vectors and count real uses. Each phi gets a // bit-vector of length . @@ -1910,7 +1910,7 @@ void HInferRepresentation::Analyze() { void HGraph::InitializeInferredTypes() { - HPhase phase("H Inferring types", this); + HPhase phase("H_Inferring types", this); InitializeInferredTypes(0, this->blocks_.length() - 1); } @@ -2047,7 +2047,7 @@ void HGraph::InsertRepresentationChangesForValue(HValue* value) { void HGraph::InsertRepresentationChanges() { - HPhase phase("H Insert representation changes", this); + HPhase phase("H_Insert representation changes", this); // Compute truncation flag for phis: Initially assume that all @@ -2104,7 +2104,7 @@ void HGraph::RecursivelyMarkPhiDeoptimizeOnUndefined(HPhi* phi) { void HGraph::MarkDeoptimizeOnUndefined() { - HPhase phase("H MarkDeoptimizeOnUndefined", this); + HPhase phase("H_MarkDeoptimizeOnUndefined", this); // Compute DeoptimizeOnUndefined flag for phis. // Any phi that can reach a use with DeoptimizeOnUndefined set must // have DeoptimizeOnUndefined set. Currently only HCompareIDAndBranch, with @@ -2430,7 +2430,7 @@ HGraph* HGraphBuilder::CreateGraph() { if (FLAG_hydrogen_stats) HStatistics::Instance()->Initialize(info()); { - HPhase phase("H Block building"); + HPhase phase("H_Block building"); current_block_ = graph()->entry_block(); Scope* scope = info()->scope(); @@ -2515,7 +2515,7 @@ HGraph* HGraphBuilder::CreateGraph() { // Perform common subexpression elimination and loop-invariant code motion. if (FLAG_use_gvn) { - HPhase phase("H Global value numbering", graph()); + HPhase phase("H_Global value numbering", graph()); HGlobalValueNumberer gvn(graph(), info()); bool removed_side_effects = gvn.Analyze(); // Trigger a second analysis pass to further eliminate duplicate values that @@ -2548,7 +2548,7 @@ HGraph* HGraphBuilder::CreateGraph() { void HGraph::ReplaceCheckedValues() { - HPhase phase("H Replace checked values", this); + HPhase phase("H_Replace checked values", this); for (int i = 0; i < blocks()->length(); ++i) { HInstruction* instr = blocks()->at(i)->first(); while (instr != NULL) { diff --git a/src/ia32/lithium-codegen-ia32.cc b/src/ia32/lithium-codegen-ia32.cc index 13b788c450..6ddc024258 100644 --- a/src/ia32/lithium-codegen-ia32.cc +++ b/src/ia32/lithium-codegen-ia32.cc @@ -67,7 +67,7 @@ class SafepointGenerator : public CallWrapper { #define __ masm()-> bool LCodeGen::GenerateCode() { - HPhase phase("Z Code generation", chunk()); + HPhase phase("Z_Code generation", chunk()); ASSERT(is_unused()); status_ = GENERATING; CpuFeatures::Scope scope(SSE2); diff --git a/src/ia32/lithium-ia32.cc b/src/ia32/lithium-ia32.cc index 94e992f04f..6db041319d 100644 --- a/src/ia32/lithium-ia32.cc +++ b/src/ia32/lithium-ia32.cc @@ -388,7 +388,7 @@ LOperand* LChunk::GetNextSpillSlot(bool is_double) { void LChunk::MarkEmptyBlocks() { - HPhase phase("L Mark empty blocks", this); + HPhase phase("L_Mark empty blocks", this); for (int i = 0; i < graph()->blocks()->length(); ++i) { HBasicBlock* block = graph()->blocks()->at(i); int first = block->first_instruction_index(); @@ -551,7 +551,7 @@ Representation LChunk::LookupLiteralRepresentation( LChunk* LChunkBuilder::Build() { ASSERT(is_unused()); chunk_ = new(zone()) LChunk(info(), graph()); - HPhase phase("L Building chunk", chunk_); + HPhase phase("L_Building chunk", chunk_); status_ = BUILDING; const ZoneList* blocks = graph()->blocks(); for (int i = 0; i < blocks->length(); i++) { diff --git a/src/lithium-allocator.cc b/src/lithium-allocator.cc index a15435c18c..83805dc729 100644 --- a/src/lithium-allocator.cc +++ b/src/lithium-allocator.cc @@ -1105,7 +1105,7 @@ bool LAllocator::Allocate(LChunk* chunk) { void LAllocator::MeetRegisterConstraints() { - HPhase phase("L Register constraints", chunk_); + HPhase phase("L_Register constraints", chunk_); first_artificial_register_ = next_virtual_register_; const ZoneList* blocks = graph_->blocks(); for (int i = 0; i < blocks->length(); ++i) { @@ -1117,7 +1117,7 @@ void LAllocator::MeetRegisterConstraints() { void LAllocator::ResolvePhis() { - HPhase phase("L Resolve phis", chunk_); + HPhase phase("L_Resolve phis", chunk_); // Process the blocks in reverse order. const ZoneList* blocks = graph_->blocks(); @@ -1207,7 +1207,7 @@ HBasicBlock* LAllocator::GetBlock(LifetimePosition pos) { void LAllocator::ConnectRanges() { - HPhase phase("L Connect ranges", this); + HPhase phase("L_Connect ranges", this); for (int i = 0; i < live_ranges()->length(); ++i) { LiveRange* first_range = live_ranges()->at(i); if (first_range == NULL || first_range->parent() != NULL) continue; @@ -1247,7 +1247,7 @@ bool LAllocator::CanEagerlyResolveControlFlow(HBasicBlock* block) const { void LAllocator::ResolveControlFlow() { - HPhase phase("L Resolve control flow", this); + HPhase phase("L_Resolve control flow", this); const ZoneList* blocks = graph_->blocks(); for (int block_id = 1; block_id < blocks->length(); ++block_id) { HBasicBlock* block = blocks->at(block_id); @@ -1268,7 +1268,7 @@ void LAllocator::ResolveControlFlow() { void LAllocator::BuildLiveRanges() { - HPhase phase("L Build live ranges", this); + HPhase phase("L_Build live ranges", this); InitializeLivenessAnalysis(); // Process the blocks in reverse order. const ZoneList* blocks = graph_->blocks(); @@ -1373,7 +1373,7 @@ bool LAllocator::SafePointsAreInOrder() const { void LAllocator::PopulatePointerMaps() { - HPhase phase("L Populate pointer maps", this); + HPhase phase("L_Populate pointer maps", this); const ZoneList* pointer_maps = chunk_->pointer_maps(); ASSERT(SafePointsAreInOrder()); @@ -1492,14 +1492,14 @@ void LAllocator::ProcessOsrEntry() { void LAllocator::AllocateGeneralRegisters() { - HPhase phase("L Allocate general registers", this); + HPhase phase("L_Allocate general registers", this); num_registers_ = Register::kNumAllocatableRegisters; AllocateRegisters(); } void LAllocator::AllocateDoubleRegisters() { - HPhase phase("L Allocate double registers", this); + HPhase phase("L_Allocate double registers", this); num_registers_ = DoubleRegister::kNumAllocatableRegisters; mode_ = DOUBLE_REGISTERS; AllocateRegisters(); diff --git a/src/mips/lithium-codegen-mips.cc b/src/mips/lithium-codegen-mips.cc index a463cd3257..c8d37b6ece 100644 --- a/src/mips/lithium-codegen-mips.cc +++ b/src/mips/lithium-codegen-mips.cc @@ -62,7 +62,7 @@ class SafepointGenerator : public CallWrapper { #define __ masm()-> bool LCodeGen::GenerateCode() { - HPhase phase("Z Code generation", chunk()); + HPhase phase("Z_Code generation", chunk()); ASSERT(is_unused()); status_ = GENERATING; CpuFeatures::Scope scope(FPU); diff --git a/src/mips/lithium-mips.cc b/src/mips/lithium-mips.cc index 2607522424..ae07c1e456 100644 --- a/src/mips/lithium-mips.cc +++ b/src/mips/lithium-mips.cc @@ -440,7 +440,7 @@ LOperand* LChunk::GetNextSpillSlot(bool is_double) { void LChunk::MarkEmptyBlocks() { - HPhase phase("L Mark empty blocks", this); + HPhase phase("L_Mark empty blocks", this); for (int i = 0; i < graph()->blocks()->length(); ++i) { HBasicBlock* block = graph()->blocks()->at(i); int first = block->first_instruction_index(); @@ -552,7 +552,7 @@ Representation LChunk::LookupLiteralRepresentation( LChunk* LChunkBuilder::Build() { ASSERT(is_unused()); chunk_ = new(zone()) LChunk(info(), graph()); - HPhase phase("L Building chunk", chunk_); + HPhase phase("L_Building chunk", chunk_); status_ = BUILDING; const ZoneList* blocks = graph()->blocks(); for (int i = 0; i < blocks->length(); i++) { diff --git a/src/x64/lithium-codegen-x64.cc b/src/x64/lithium-codegen-x64.cc index 8615a4d6a9..985cdc6d98 100644 --- a/src/x64/lithium-codegen-x64.cc +++ b/src/x64/lithium-codegen-x64.cc @@ -67,7 +67,7 @@ class SafepointGenerator : public CallWrapper { #define __ masm()-> bool LCodeGen::GenerateCode() { - HPhase phase("Z Code generation", chunk()); + HPhase phase("Z_Code generation", chunk()); ASSERT(is_unused()); status_ = GENERATING; diff --git a/src/x64/lithium-x64.cc b/src/x64/lithium-x64.cc index 8cddb364c5..19bb58ae64 100644 --- a/src/x64/lithium-x64.cc +++ b/src/x64/lithium-x64.cc @@ -382,7 +382,7 @@ LOperand* LChunk::GetNextSpillSlot(bool is_double) { void LChunk::MarkEmptyBlocks() { - HPhase phase("L Mark empty blocks", this); + HPhase phase("L_Mark empty blocks", this); for (int i = 0; i < graph()->blocks()->length(); ++i) { HBasicBlock* block = graph()->blocks()->at(i); int first = block->first_instruction_index(); @@ -545,7 +545,7 @@ Representation LChunk::LookupLiteralRepresentation( LChunk* LChunkBuilder::Build() { ASSERT(is_unused()); chunk_ = new(zone()) LChunk(info(), graph()); - HPhase phase("L Building chunk", chunk_); + HPhase phase("L_Building chunk", chunk_); status_ = BUILDING; const ZoneList* blocks = graph()->blocks(); for (int i = 0; i < blocks->length(); i++) {