[v8-dev] Initialize CompilationInfo::osr_expr_stack_height correctly. (issue 889383002 by titzer@chromium.org)

[titzer]

Reviewers: Michael Starzinger,

Description:
Initialize CompilationInfo::osr_expr_stack_height correctly.
R=mstarzin...@chromium.org
BUG=

Please review this at https://codereview.chromium.org/889383002/

Base URL: https://chromium.googlesource.com/v8/v8.git@master

Affected files (+12, -6 lines):
  M src/compiler.cc


Index: src/compiler.cc
diff --git a/src/compiler.cc b/src/compiler.cc
index  
d31ca9e11bed6d7cec3ca8dd54eaf6e9e2790577..f792f226b713945a20aa490d1a405d78f0142763  
100644
--- a/src/compiler.cc
+++ b/src/compiler.cc
@@ -57,7 +57,8 @@ CompilationInfo::CompilationInfo(Handle<Script> script,  
Zone* zone)
       optimization_id_(-1),
       ast_value_factory_(NULL),
       ast_value_factory_owned_(false),
-      aborted_due_to_dependency_change_(false) {
+      aborted_due_to_dependency_change_(false),
+      osr_expr_stack_height_(0) {
   Initialize(script->GetIsolate(), BASE, zone);
 }

@@ -71,7 +72,8 @@ CompilationInfo::CompilationInfo(Isolate* isolate, Zone*  
zone)
       optimization_id_(-1),
       ast_value_factory_(NULL),
       ast_value_factory_owned_(false),
-      aborted_due_to_dependency_change_(false) {
+      aborted_due_to_dependency_change_(false),
+      osr_expr_stack_height_(0) {
   Initialize(isolate, STUB, zone);
 }

@@ -87,7 +89,8 @@  
CompilationInfo::CompilationInfo(Handle<SharedFunctionInfo> shared_info,
       optimization_id_(-1),
       ast_value_factory_(NULL),
       ast_value_factory_owned_(false),
-      aborted_due_to_dependency_change_(false) {
+      aborted_due_to_dependency_change_(false),
+      osr_expr_stack_height_(0) {
   Initialize(script_->GetIsolate(), BASE, zone);
 }

@@ -104,7 +107,8 @@ CompilationInfo::CompilationInfo(Handle<JSFunction>  
closure, Zone* zone)
       optimization_id_(-1),
       ast_value_factory_(NULL),
       ast_value_factory_owned_(false),
-      aborted_due_to_dependency_change_(false) {
+      aborted_due_to_dependency_change_(false),
+      osr_expr_stack_height_(0) {
   Initialize(script_->GetIsolate(), BASE, zone);
 }

@@ -118,7 +122,8 @@ CompilationInfo::CompilationInfo(HydrogenCodeStub*  
stub, Isolate* isolate,
       optimization_id_(-1),
       ast_value_factory_(NULL),
       ast_value_factory_owned_(false),
-      aborted_due_to_dependency_change_(false) {
+      aborted_due_to_dependency_change_(false),
+      osr_expr_stack_height_(0) {
   Initialize(isolate, STUB, zone);
   code_stub_ = stub;
 }
@@ -136,7 +141,8 @@ CompilationInfo::CompilationInfo(
       optimization_id_(-1),
       ast_value_factory_(NULL),
       ast_value_factory_owned_(false),
-      aborted_due_to_dependency_change_(false) {
+      aborted_due_to_dependency_change_(false),
+      osr_expr_stack_height_(0) {
   Initialize(isolate, BASE, zone);
 }



--
--
v8-dev mailing list
v8-dev@googlegroups.com
http://groups.google.com/group/v8-dev
--- 
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to v8-dev+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.