Reviewers: Yang,
Message:
Note that the platform ports are still missing and one Promises test (i.e.
try-throw-reject-in-constructor) is still failing, but I would like to get a
first round of comments if possible. Thanks a bunch!
Description:
Switch full-codegen from StackHandlers to handler table.
This switches full-codegen to no longer push and pop StackHandler
markers onto the operand stack, but relies on a range-based handler
tables instead. We only use StackHandlers in JSEntryStubs to mark the
transition from C to JS code.
Note that this makes deoptimization and OSR from within any try-block
work out of the box, makes the non-exception paths faster and should
overall be neutral on the memory footprint (pros).
On the other hand it makes the exceptions paths slower and actually
throwing and exception more expensive (cons).
[email protected]
TEST=cctest/test-run-jsexceptions/DeoptTry
Please review this at https://codereview.chromium.org/1010883002/
Base URL:
https://chromium.googlesource.com/v8/v8.git@local_cleanup-isolate-dead-code
Affected files (+337, -350 lines):
M src/compiler/ast-graph-builder.cc
M src/compiler/code-generator.cc
M src/debug.cc
M src/frames.h
M src/frames.cc
M src/frames-inl.h
M src/full-codegen.h
M src/full-codegen.cc
M src/ia32/code-stubs-ia32.cc
M src/ia32/full-codegen-ia32.cc
M src/ia32/macro-assembler-ia32.h
M src/ia32/macro-assembler-ia32.cc
M src/isolate.cc
M src/objects.h
M src/objects.cc
M src/objects-debug.cc
M src/objects-inl.h
M src/runtime/runtime.h
M src/runtime/runtime-generator.cc
M test/cctest/compiler/test-run-jsexceptions.cc
--
--
v8-dev mailing list
[email protected]
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 [email protected].
For more options, visit https://groups.google.com/d/optout.
