LGTM.
http://codereview.chromium.org/853004/diff/1/2 File src/data-flow.cc (right): http://codereview.chromium.org/853004/diff/1/2#newcode1982 src/data-flow.cc:1982: BitVector temp(definition_count); I don't know if it's worth doing, but since ComputeRDOut always overwrites all elements in the BitVector temp, you could reuse one instance for the loop. http://codereview.chromium.org/853004/diff/1/2#newcode2025 src/data-flow.cc:2025: BitVector temp(definition_count); Also could reuse one BitVector temp here. http://codereview.chromium.org/853004 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
