Interesting idea, but what's the purpose?

http://codereview.chromium.org/546125/diff/4001/4003
File src/runtime.cc (right):

http://codereview.chromium.org/546125/diff/4001/4003#newcode7904
src/runtime.cc:7904: SharedFunctionInfo** buffer, int buffer_size) {
Vectors (from utils.h) are preferred instead of arrays because they do
bounds checking in debug mode.

http://codereview.chromium.org/546125/diff/4001/4003#newcode7952
src/runtime.cc:7952: // No GC must occur here
FYI, usually AssertNoAllocation auto variable is used to ensure that no
heap allocations take place => GC can't occur.

http://codereview.chromium.org/546125/diff/4001/4003#newcode7960
src/runtime.cc:7960:
Then, this allocation should appear in a separate block.

http://codereview.chromium.org/546125

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to