• Jaroslav Sevcik's avatar
    [turbofan] Remove int32 narrowing during typed lowering. · c3af691e
    Jaroslav Sevcik authored
    With Int32Add we lose the int/uint distinction, so later, in simplified lowering we can make a wrong decision. E.g., see the attached test case, where we lower NumberAdd -> Int32Add because inputs are Uint32, but during simplified lowering we change the inputs to Int32, so we get a wrong result.
    
    Simplified lowering will lower the NumberAdd operations anyway, so we should lose performance.
    
    BUG=
    R=bmeurer@chromium.org
    
    Review URL: https://codereview.chromium.org/721723004
    
    Cr-Commit-Position: refs/heads/master@{#25368}
    c3af691e
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
test262 Loading commit data...
test262-es6 Loading commit data...
unittests Loading commit data...
webkit Loading commit data...