lgtm
Please check that --prof and tick-processors are not affected by this
change.
https://codereview.chromium.org/16901014/diff/1/src/log.cc
File src/log.cc (right):
https://codereview.chromium.org/16901014/diff/1/src/log.cc#newcode922
src/log.cc:922: ASSERT(symbol);
No need in this assert.
https://codereview.chromium.org/16901014/diff/1/src/log.cc#newcode1199
src/log.cc:1199: AppendCodeCreateHeader(&msg, REG_EXP_TAG, code, -2);
I wonder if we could have a constant with descriptive name for -2.
https://codereview.chromium.org/16901014/
--
--
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.
