• bmeurer's avatar
    [turbofan] Right hand side of shifts needs ToUint32. · 5f288c20
    bmeurer authored
    Currently we lower shifts directly to machine operators, and add an
    appropriate Word32And to implement the & 0x1F operation on the right
    hand side required by the specification. However for Word32And we assume
    Int32 in simplified lowering, which is basically changes the right hand
    side bit interpretation for the shifts from Uint32 to Int32, which is
    obviously wrong. So now we represent that explicitly by proper
    simplified operators for the shifts, which are lowered to machine in
    simplified lowering.
    
    R=jarin@chromium.org
    
    Review URL: https://codereview.chromium.org/1213803008
    
    Cr-Commit-Position: refs/heads/master@{#29465}
    5f288c20
simplified-operator.cc 10.3 KB