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

Side by Side Diff: src/compiler.cc

Issue 2636903002: Assert that context creation doesn't throw (Closed)
Patch Set: Created 3 years, 11 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 unified diff | Download patch
OLDNEW
1 // Copyright 2012 the V8 project authors. All rights reserved. 1 // Copyright 2012 the V8 project authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "src/compiler.h" 5 #include "src/compiler.h"
6 6
7 #include <algorithm> 7 #include <algorithm>
8 #include <memory> 8 #include <memory>
9 9
10 #include "src/asmjs/asm-js.h" 10 #include "src/asmjs/asm-js.h"
(...skipping 1596 matching lines...) Expand 10 before | Expand all | Expand 10 after
1607 "V8.CompileSerialize"); 1607 "V8.CompileSerialize");
1608 *cached_data = CodeSerializer::Serialize(isolate, result, source); 1608 *cached_data = CodeSerializer::Serialize(isolate, result, source);
1609 if (FLAG_profile_deserialization) { 1609 if (FLAG_profile_deserialization) {
1610 PrintF("[Compiling and serializing took %0.3f ms]\n", 1610 PrintF("[Compiling and serializing took %0.3f ms]\n",
1611 timer.Elapsed().InMillisecondsF()); 1611 timer.Elapsed().InMillisecondsF());
1612 } 1612 }
1613 } 1613 }
1614 } 1614 }
1615 1615
1616 if (result.is_null()) { 1616 if (result.is_null()) {
1617 isolate->ReportPendingMessages(); 1617 if (natives != EXTENSION_CODE) isolate->ReportPendingMessages();
1618 } else { 1618 } else {
1619 isolate->debug()->OnAfterCompile(script); 1619 isolate->debug()->OnAfterCompile(script);
1620 } 1620 }
1621 } else if (result->ic_age() != isolate->heap()->global_ic_age()) { 1621 } else if (result->ic_age() != isolate->heap()->global_ic_age()) {
1622 result->ResetForNewContext(isolate->heap()->global_ic_age()); 1622 result->ResetForNewContext(isolate->heap()->global_ic_age());
1623 } 1623 }
1624 return result; 1624 return result;
1625 } 1625 }
1626 1626
1627 Handle<SharedFunctionInfo> Compiler::GetSharedFunctionInfoForStreamedScript( 1627 Handle<SharedFunctionInfo> Compiler::GetSharedFunctionInfoForStreamedScript(
(...skipping 134 matching lines...) Expand 10 before | Expand all | Expand 10 after
1762 } 1762 }
1763 1763
1764 if (shared->is_compiled()) { 1764 if (shared->is_compiled()) {
1765 // TODO(mvstanton): pass pretenure flag to EnsureLiterals. 1765 // TODO(mvstanton): pass pretenure flag to EnsureLiterals.
1766 JSFunction::EnsureLiterals(function); 1766 JSFunction::EnsureLiterals(function);
1767 } 1767 }
1768 } 1768 }
1769 1769
1770 } // namespace internal 1770 } // namespace internal
1771 } // namespace v8 1771 } // namespace v8
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698