https://codereview.chromium.org/23201016/diff/1/src/runtime.cc
File src/runtime.cc (right):
https://codereview.chromium.org/23201016/diff/1/src/runtime.cc#newcode8341
src/runtime.cc:8341: Handle<Code>
optimized_code(deoptimizer->compiled_code());
On 2013/08/21 15:33:17, Toon Verwaest wrote:
Could we also rename compiled_code() in the deoptimizer to
optimized_code() to
make it obvious?
Will have to do that rename in a later CL; wanted to keep this change
small.
https://codereview.chromium.org/23201016/diff/1/src/runtime.cc#newcode8361
src/runtime.cc:8361: // the code to be reused a few times before
unlinking it from the function.
On 2013/08/21 15:33:17, Toon Verwaest wrote:
I'm not fully sure how this would work yet. Is it worthwhile putting
this
comment in?
Removed.
https://codereview.chromium.org/23201016/diff/1/test/mjsunit/compiler/regress-shared-deopt.js
File test/mjsunit/compiler/regress-shared-deopt.js (right):
https://codereview.chromium.org/23201016/diff/1/test/mjsunit/compiler/regress-shared-deopt.js#newcode38
test/mjsunit/compiler/regress-shared-deopt.js:38: function test() {
On 2013/08/21 15:33:17, Toon Verwaest wrote:
2-space indent.
Done.
https://codereview.chromium.org/23201016/
--
--
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/groups/opt_out.
