Reviewers: Michael Starzinger,
Message:
mstarzinger PTAL - this is minor clean-up.
picksi - this change attempts to incorporate your comments from
https://codereview.chromium.org/1266713004/.
Thanks!
Description:
[Interpreter] Consistency fixes.
Change minimum BytecodeArray frame size to zero now return value is in
the accumulator.
Fix inconsistent checks in bytecode-array-builder.cc.
Simplify bytecode disassembly by adding Bytecodes::Decode to
disassemble one bytecode and operands.
BUG=v8:4280
LOG=N
Please review this at https://codereview.chromium.org/1259193004/
Base URL: https://chromium.googlesource.com/v8/v8.git@master
Affected files (+69, -41 lines):
M src/interpreter/bytecode-array-builder.cc
M src/interpreter/bytecodes.h
M src/interpreter/bytecodes.cc
M src/objects.cc
M src/objects-inl.h
M test/cctest/interpreter/test-bytecode-array-builder.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.
