• 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
simplified-operator.h 6.35 KB