• bmeurer's avatar
    [turbofan] Introduce a dedicated CheckBounds operator. · 2267ccb1
    bmeurer authored
    This CheckBounds simplified operator is similar to the HBoundsCheck in
    Crankshaft, and is hooked up to the new type feedback support in the
    SimplifiedLowering. We use it to check the index bounds for keyed
    property accesses.
    
    Note to perf sheriffs: This will tank quite a few benchmarks, as the
    operator makes some redundant branch elimination ineffective for
    certain patterns of keyed accesses. This does require more serious
    redundancy elimination, which we will do in a separate CL. So ignore
    any regressions from this CL, we know there will be a few.
    
    R=jarin@chromium.org
    BUG=v8:4470,v8:5100
    
    Committed: https://crrev.com/85e5567dae66a918500ae94c5568221137a0f5d4
    Review-Url: https://codereview.chromium.org/2035893004
    Cr-Original-Commit-Position: refs/heads/master@{#36947}
    Cr-Commit-Position: refs/heads/master@{#37003}
    2267ccb1
regress-5100.js 1.28 KB