• jarin's avatar
    [turbofan] Introduce CheckIf node (deopt without explicit frame state). · f2312019
    jarin authored
    Type feedback introduced DeoptimizeIf node in representation inference
    (for Int32AddWithOverflow); we found the frame state for the deopt by
    walking the effect chain. Unfortunately, the effect chain can hit
    effect merges introduced by simplified lowering (e.g., in LoadBuffer)
    and thus fail the assertion (we refuse to go through effect phis).
    
    This CL postpones assignment of the frame state to the effect-control
    lninearizer, so that we can correctly propagate the frame state to
    the deopt point. The DeoptimizeIf node with unassigned frame state is
    called CheckIf.
    
    BUG=
    
    Review-Url: https://codereview.chromium.org/2050813003
    Cr-Commit-Position: refs/heads/master@{#36839}
    f2312019
node-properties.cc 12.7 KB