Potentially inefficient reallocation.  Otherwise, LGTM.


http://codereview.chromium.org/6260035/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

http://codereview.chromium.org/6260035/diff/1/src/hydrogen.cc#newcode1991
src/hydrogen.cc:1991: BitVector visited(GetMaximumValueID());
Is this allocation a problem?  Can you reuse one?

http://codereview.chromium.org/6260035/

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

Reply via email to