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

Unified Diff: src/hydrogen-gvn.cc

Issue 17827005: Get rid of ZoneScope completely. (Closed) Base URL: https://v8.googlecode.com/svn/branches/bleeding_edge
Patch Set: Suggestions from danno Created 7 years, 6 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 | « src/hydrogen-gvn.h ('k') | src/isolate.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/hydrogen-gvn.cc
diff --git a/src/hydrogen-gvn.cc b/src/hydrogen-gvn.cc
index 9d5e866de443ce226d331fa5ecb1054542c01828..b27b86f14b623da9a96cc395fe21219d64b8c031 100644
--- a/src/hydrogen-gvn.cc
+++ b/src/hydrogen-gvn.cc
@@ -365,16 +365,15 @@ HGlobalValueNumberer::HGlobalValueNumberer(HGraph* graph, CompilationInfo* info)
: graph_(graph),
info_(info),
removed_side_effects_(false),
- phase_zone_(info->phase_zone()),
- phase_zone_scope_(phase_zone_, DELETE_ON_EXIT),
- block_side_effects_(graph->blocks()->length(), phase_zone_),
- loop_side_effects_(graph->blocks()->length(), phase_zone_),
- visited_on_paths_(phase_zone_, graph->blocks()->length()) {
+ zone_(graph->isolate()),
+ block_side_effects_(graph->blocks()->length(), zone()),
+ loop_side_effects_(graph->blocks()->length(), zone()),
+ visited_on_paths_(zone(), graph->blocks()->length()) {
ASSERT(!AllowHandleAllocation::IsAllowed());
block_side_effects_.AddBlock(GVNFlagSet(), graph_->blocks()->length(),
- phase_zone_);
+ zone());
loop_side_effects_.AddBlock(GVNFlagSet(), graph_->blocks()->length(),
- phase_zone_);
+ zone());
}
bool HGlobalValueNumberer::Analyze() {
@@ -758,9 +757,9 @@ class GvnBasicBlockState: public ZoneObject {
// GvnBasicBlockState instances.
void HGlobalValueNumberer::AnalyzeGraph() {
HBasicBlock* entry_block = graph_->entry_block();
- HValueMap* entry_map = new(phase_zone()) HValueMap(phase_zone());
+ HValueMap* entry_map = new(zone()) HValueMap(zone());
GvnBasicBlockState* current =
- GvnBasicBlockState::CreateEntry(phase_zone(), entry_block, entry_map);
+ GvnBasicBlockState::CreateEntry(zone(), entry_block, entry_map);
while (current != NULL) {
HBasicBlock* block = current->block();
@@ -802,7 +801,7 @@ void HGlobalValueNumberer::AnalyzeGraph() {
if (instr->HasSideEffects()) removed_side_effects_ = true;
instr->DeleteAndReplaceWith(other);
} else {
- map->Add(instr, phase_zone());
+ map->Add(instr, zone());
}
}
if (instr->IsLinked() &&
@@ -828,7 +827,7 @@ void HGlobalValueNumberer::AnalyzeGraph() {
HBasicBlock* dominator_block;
GvnBasicBlockState* next =
- current->next_in_dominator_tree_traversal(phase_zone(),
+ current->next_in_dominator_tree_traversal(zone(),
&dominator_block);
if (next != NULL) {
« no previous file with comments | « src/hydrogen-gvn.h ('k') | src/isolate.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698