• bmeurer's avatar
    [turbofan] Address the useless overflow bit materialization. · 00889cc2
    bmeurer authored
    Add control dependencies to Projection and Int32Add/SubWithOverflow
    operators, to prevent the scheduler from moving the Projection nodes
    into the wrong place. This way the instruction selection can combine
    the Int32Add/SubWithOverflow operations with the DeoptimizeIf and/or
    DeoptimizeUnless nodes. This needs new operators CheckedInt32Add and
    CheckedInt32Sub so that we can delay the actual lowering until the
    effect/control linearizer.
    
    This also makes CheckIf operator obsolete, so we can drop it.
    
    R=jarin@chromium.org
    
    Review-Url: https://codereview.chromium.org/2082993002
    Cr-Commit-Position: refs/heads/master@{#37148}
    00889cc2
common-operator-unittest.cc 14.4 KB