• Clemens Hammacher's avatar
    Revert "[turbofan] eagerly prune None types and deadness from the graph" · 324e0a7a
    Clemens Hammacher authored
    This reverts commit e1cdda25.
    
    Reason for revert: Fails 'constructor-inlining' on GC-Stress bot: https://build.chromium.org/p/client.v8/builders/V8%20Linux64%20GC%20Stress%20-%20custom%20snapshot/builds/15270
    
    Original change's description:
    > [turbofan] eagerly prune None types and deadness from the graph
    > 
    > In addition to using the {Dead} node to prune dead control nodes and nodes that 
    > depend on them, we introduce a {DeadValue} node representing an impossible value 
    > that can occur at any position in the graph. The extended {DeadCodeElimination}
    > prunes {DeadValue} and its uses, inserting a crashing {Unreachable} node into
    > the effect chain when possible. The remaining uses of {DeadValue} are handled
    > in {EffectControlLinearizer}, where we always have access to the effect chain.
    > In addition to explicitly introduced {DeadValue} nodes, we consider any value use
    > of a node with type {None} as dead.
    > 
    > Bug: chromium:741225
    > Change-Id: Icc4b636d1d018c452ba1a2fa7cd3e00e522f1655
    > Reviewed-on: https://chromium-review.googlesource.com/641250
    > Commit-Queue: Tobias Tebbi <tebbi@chromium.org>
    > Reviewed-by: Jaroslav Sevcik <jarin@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#48208}
    
    TBR=jarin@chromium.org,tebbi@chromium.org
    
    Change-Id: I9c175d47e2ee4b11a36ed90421202f2354610398
    No-Presubmit: true
    No-Tree-Checks: true
    No-Try: true
    Bug: chromium:741225
    Reviewed-on: https://chromium-review.googlesource.com/690080Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#48210}
    324e0a7a
instruction-selector.h 18.9 KB