Get rid of the function sorting in for polymorphic calls.

Get rid of the function sorting in for polymorphic calls.

The idea of this code was to sort functions according to
ticks spend executing them, but now these ticks are always
zero and therefore we fall back to sorting by AST length (or
even worse by source length) all the time, which is a bad,
arbitrary measure.

R=verwaest@chromium.org

Committed: https://code.google.com/p/v8/source/detail?r=19267

[Benedikt Meurer, 2014-02-11 10:22:27]

PTAL

[Toon Verwaest, 2014-02-11 10:32:48]

https://codereview.chromium.org/159653003/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/159653003/diff/1/src/hydrogen.cc#newcode6785
src/hydrogen.cc:6785: for (int i = 0; i < types->length(); ++i) {
Please put back the MaxCallPolymorphism; which will force you to put back the
type()->Is(Type::String()) thing below to keep the limits of both loops in sync.

[Benedikt Meurer, 2014-02-11 10:34:54]

Simplified, fixed patch.

https://codereview.chromium.org/159653003/diff/1/src/hydrogen.cc
File src/hydrogen.cc (right):

https://codereview.chromium.org/159653003/diff/1/src/hydrogen.cc#newcode6785
src/hydrogen.cc:6785: for (int i = 0; i < types->length(); ++i) {
On 2014/02/11 10:32:48, Toon Verwaest wrote:
> Please put back the MaxCallPolymorphism; which will force you to put back the
> type()->Is(Type::String()) thing below to keep the limits of both loops in
sync.

Done.

[Toon Verwaest, 2014-02-11 10:46:04]

lgtm

[Benedikt Meurer, 2014-02-11 10:48:43]

Committed patchset #2 manually as r19267 (presubmit successful).