• bmeurer's avatar
    [turbofan] Introduce simplified NumberBitwise{Or,Xor,And} operators. · d08f9045
    bmeurer authored
    Currently we still (mis)used some machine operators in typed lowering
    (namely Word32Or, Word32Xor and Word32And). But these operators are
    "polymorphic" in the signedness of their inputs and output, hence the
    representation selection (and thereby simplified lowering) was unable to
    figure out whether a bitwise operation that was seen would produce an
    unsigned or a signed result. If such nodes also have frame state uses,
    the only safe choice was float64, which was not only a lot less ideal,
    but also the main cause of the for-in related deoptimizer loops.
    
    Adding dedicated NumberBitwiseOr, NumberBitwiseAnd and NumberBitwiseXor
    simplified operators not only gives us precise (and correct) typing for
    the bitwise operations, but also allows us to actually verify the graph
    properly after typed lowering.
    
    Drive-by-fix: Remove the double-to-smi magic from the Deoptimizer, which
    is responsible for various deopt-loops in TurboFan, and is no longer
    needed with the addition of the NumberBitwise operators.
    
    R=jarin@chromium.org
    
    Review URL: https://codereview.chromium.org/1422213002
    
    Cr-Commit-Position: refs/heads/master@{#31594}
    d08f9045
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
simdjs Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
webkit Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...