• titzer's avatar
    [turbofan] Handle dead diamonds in scheduling and add a test. · 45d75bca
    titzer authored
    The background here is that graphs generated from WASM are not trimmed.
    That means there can be some floating control diamonds that are not
    reachable from end. An assertion in the scheduler for phis from floating
    diamonds checks that the use edge in this situation is the control edge,
    but in general, any edge could cause this.
    
    Scheduling still works without this assertion. The longer term fix
    is to either trim the graphs (more compile time overhead for WASM)
    or improve the scheduler's handling of dead code in the graph. Currently
    it does not schedule dead code but the potential use positions of
    dead code are used in the computation of the common dominator of uses. We could
    recognize dead nodes in PrepareUses() and check in GetBlockForUse()
    as per TODO.
    
    R=bradnelson@chromium.org, mstarzinger@chromium.org
    BUG=
    
    Review URL: https://codereview.chromium.org/1846933002
    
    Cr-Commit-Position: refs/heads/master@{#35245}
    45d75bca
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
fuzzer Loading commit data...
intl Loading commit data...
js-perf-test Loading commit data...
memory Loading commit data...
message Loading commit data...
mjsunit Loading commit data...
mozilla Loading commit data...
preparser Loading commit data...
promises-aplus Loading commit data...
simdjs Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
webkit Loading commit data...
bot_default.gyp Loading commit data...
bot_default.isolate Loading commit data...
default.gyp Loading commit data...
default.isolate Loading commit data...
ignition.gyp Loading commit data...
ignition.isolate Loading commit data...
optimize_for_size.gyp Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.gyp Loading commit data...
perf.isolate Loading commit data...