Author: [email protected]
Date: Thu May 14 04:36:10 2009
New Revision: 1948
Modified:
branches/bleeding_edge/src/arm/codegen-arm.cc
branches/bleeding_edge/src/arm/jump-target-arm.cc
branches/bleeding_edge/src/codegen.cc
branches/bleeding_edge/src/ia32/codegen-ia32.cc
branches/bleeding_edge/src/ia32/jump-target-ia32.cc
Log:
Removed the delete calls to ZoneObjects.
Review URL: http://codereview.chromium.org/115348
Modified: branches/bleeding_edge/src/arm/codegen-arm.cc
==============================================================================
--- branches/bleeding_edge/src/arm/codegen-arm.cc (original)
+++ branches/bleeding_edge/src/arm/codegen-arm.cc Thu May 14 04:36:10 2009
@@ -1529,8 +1529,6 @@
if (node->break_target()->is_linked()) {
node->break_target()->Bind();
}
-
- delete start_frame;
}
@@ -2132,7 +2130,6 @@
}
shadows[i]->other_target()->Jump();
}
- delete shadows[i];
}
exit.Bind();
@@ -2308,7 +2305,6 @@
original->Branch(eq);
}
}
- delete shadows[i];
}
if (has_valid_frame()) {
Modified: branches/bleeding_edge/src/arm/jump-target-arm.cc
==============================================================================
--- branches/bleeding_edge/src/arm/jump-target-arm.cc (original)
+++ branches/bleeding_edge/src/arm/jump-target-arm.cc Thu May 14 04:36:10
2009
@@ -272,7 +272,6 @@
if (other != NULL && other->Equals(cgen_->frame())) {
// Set the reaching frame element to null to avoid
// processing it later, and then bind its entry label.
- delete other;
reaching_frames_[j] = NULL;
__ bind(&merge_labels_[j]);
}
@@ -309,7 +308,6 @@
// them.
for (int i = 0; i < reaching_frames_.length(); i++) {
if (!merge_labels_[i].is_bound()) {
- delete reaching_frames_[i];
reaching_frames_[i] = NULL;
__ bind(&merge_labels_[i]);
}
Modified: branches/bleeding_edge/src/codegen.cc
==============================================================================
--- branches/bleeding_edge/src/codegen.cc (original)
+++ branches/bleeding_edge/src/codegen.cc Thu May 14 04:36:10 2009
@@ -104,7 +104,6 @@
void CodeGenerator::DeleteFrame() {
if (has_valid_frame()) {
frame_->DetachFromCodeGenerator();
- delete frame_;
frame_ = NULL;
}
}
Modified: branches/bleeding_edge/src/ia32/codegen-ia32.cc
==============================================================================
--- branches/bleeding_edge/src/ia32/codegen-ia32.cc (original)
+++ branches/bleeding_edge/src/ia32/codegen-ia32.cc Thu May 14 04:36:10 2009
@@ -2183,8 +2183,6 @@
__ WriteInternalReference(entry_pos, *case_targets[i]);
}
}
-
- delete start_frame;
}
@@ -2978,7 +2976,6 @@
shadows[i]->other_target()->Jump();
}
}
- delete shadows[i];
}
exit.Bind();
@@ -3159,7 +3156,6 @@
original->Branch(equal);
}
}
- delete shadows[i];
}
if (has_valid_frame()) {
Modified: branches/bleeding_edge/src/ia32/jump-target-ia32.cc
==============================================================================
--- branches/bleeding_edge/src/ia32/jump-target-ia32.cc (original)
+++ branches/bleeding_edge/src/ia32/jump-target-ia32.cc Thu May 14 04:36:10
2009
@@ -305,7 +305,6 @@
if (other != NULL && other->Equals(cgen_->frame())) {
// Set the reaching frame element to null to avoid
// processing it later, and then bind its entry label.
- delete other;
reaching_frames_[j] = NULL;
__ bind(&merge_labels_[j]);
}
@@ -342,7 +341,6 @@
// them.
for (int i = 0; i < reaching_frames_.length(); i++) {
if (!merge_labels_[i].is_bound()) {
- delete reaching_frames_[i];
reaching_frames_[i] = NULL;
__ bind(&merge_labels_[i]);
}
--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---
