[v8-dev] Re: Enable map dependency to in-flight compilation info. (issue 16542003)

2013-06-07 Thread yangguo
Committed patchset #3 manually as r15005 (presubmit successful). https://chromiumcodereview.appspot.com/16542003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-dev --- You received this message because you are subscribed to the Google Groups "v8-dev" group.

[v8-dev] Re: Enable map dependency to in-flight compilation info. (issue 16542003)

2013-06-07 Thread ulan
LGTM https://codereview.chromium.org/16542003/diff/6001/src/compiler.cc File src/compiler.cc (right): https://codereview.chromium.org/16542003/diff/6001/src/compiler.cc#newcode146 src/compiler.cc:146: for (int j = 0; j < group_maps->length(); j++) { This has O(n^2) complexity. It is probably OK

[v8-dev] Re: Enable map dependency to in-flight compilation info. (issue 16542003)

2013-06-07 Thread yangguo
On 2013/06/06 11:49:44, Yang wrote: I've uploaded a second patch set to remove the changes to transition maps and initial object/array maps to put into a second CL. https://codereview.chromium.org/16542003/ -- -- v8-dev mailing list v8-dev@googlegroups.com http://groups.google.com/group/v8-d