• Clemens Hammacher's avatar
    Revert "[wasm][liftoff] Optimize one-armed ifs" · 0536ee43
    Clemens Hammacher authored
    This reverts commit c2aaf0a6.
    
    Reason for revert: Benchmarks fail, and ClusterFuzz is not happy (issue 911406, issue 911271)
    
    Original change's description:
    > [wasm][liftoff] Optimize one-armed ifs
    > 
    > Do not implement one-armed ifs by emulating an empty else branch. In
    > Liftoff, we can generate better code and save compile time by handling
    > this specially. If the merge point at the end of the if is not reached
    > by the if-branch, we do not need to generate any merge code.
    > 
    > R=​titzer@chromium.org
    > 
    > Bug: v8:6600, v8:8423
    > Change-Id: Ie8ea69dd7491f225605a8e1b986d275d869aa90b
    > Reviewed-on: https://chromium-review.googlesource.com/c/1356508
    > Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    > Reviewed-by: Ben Titzer <titzer@chromium.org>
    > Cr-Commit-Position: refs/heads/master@{#57968}
    
    TBR=titzer@chromium.org,clemensh@chromium.org
    
    # Not skipping CQ checks because original CL landed > 1 day ago.
    
    Bug: v8:6600, v8:8423
    Change-Id: I5cb3b069f40e34f34da4013e666f6ff293752567
    Reviewed-on: https://chromium-review.googlesource.com/c/1360633Reviewed-by: 's avatarClemens Hammacher <clemensh@chromium.org>
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#58012}
    0536ee43
liftoff-compiler.cc 81 KB