Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v10.x] deps: V8: backport f27ac28 #28061

Closed
wants to merge 1 commit into from

Commits on Jun 4, 2019

  1. deps: V8: backport f27ac28

    Original commit message:
    
        [turbofan] Pin pure unreachable values to effect chain (in rep selection)
    
        Currently, if we lower to a pure computation that is unreachable because
        of some runtime check, we just rename it with DeadValue. This is
        problematic if the pure computation gets later eliminated - that allows
        the DeadValue node float above the check that makes it dead. As we
        conservatively lower DeadValues to debug-break (i.e., crash), we
        might induce crash where we should not.
    
        With this CL, whenever we lower an impossible effectful node (i.e., with
        Type::None) to a pure node in simplified lowering, we insert an
        Unreachable node there (pinned to the effect chain) and mark the
        impossible node dead (and make it depend on the Unreachable node).
    
        Bug: chromium:910838
        Change-Id: I218991c79b9e283a9dd5beb4d3f0c4664be76cb2
        Reviewed-on: https://chromium-review.googlesource.com/c/1365274
        Reviewed-by: Benedikt Meurer <bmeurer@chromium.org>
        Commit-Queue: Jaroslav Sevcik <jarin@chromium.org>
        Cr-Commit-Position: refs/heads/master@{#58066}
    
    Refs: v8/v8@f27ac28
    targos committed Jun 4, 2019
    Configuration menu
    Copy the full SHA
    36ac348 View commit details
    Browse the repository at this point in the history