Description[turbofan] Try hard(er) to use smi representation for float64 values.
Previously ChangeLowering would always box float64 values when going to
tagged representation, but that introduces a lot of deoptimizer loops
and polymorphism into TurboFan, which is unfortunate and unnecessary.
This adds some logic to ChangeFloat64ToTagged to try harder to create a
Smi when going from Float64 to Tagged, instead of always allocating a
HeapNumber. This might need some additional tweaking, but at least it
makes it possible to start comparing TurboFan and Crankshaft for some
regular JavaScript.
R=jarin@chromium.org
Committed: https://crrev.com/7709e41aec9de8b42ddebb197a02c1017d6fe87a
Cr-Commit-Position: refs/heads/master@{#31609}
Patch Set 1 #Patch Set 2 : Remove unittest for now. Will need new ones, once we know what we want. #
Total comments: 2
Messages
Total messages: 8 (1 generated)
|