LGTM with comments
https://codereview.chromium.org/15959021/diff/1/ChangeLog
File ChangeLog (right):
https://codereview.chromium.org/15959021/diff/1/ChangeLog#newcode3
ChangeLog:3: Add support for //# sourceURL similar to deprecated //@
sourceURL one.
past tense please
https://codereview.chromium.org/15959021/diff/1/ChangeLog#newcode6
ChangeLog:6: Make sure IfBuilder::Return clears the current block.
past tense please
https://codereview.chromium.org/15959021/diff/1/ChangeLog#newcode9
ChangeLog:9: Fix two CPU profiler tests on ARM and MIPS simulators
No need to mention this
https://codereview.chromium.org/15959021/diff/1/ChangeLog#newcode15
ChangeLog:15: Fix counting of scanned bytes in incremental marking step
for large
Past tense please.
Since this affects the same bug as the previous entry, maybe combine
them as "Fixed idle incremental GC for large objects"?
https://codereview.chromium.org/15959021/
--
--
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.
