• mythria's avatar
    [Interpreter] Adds a placeholder merge node when visiting jumps. · d0e01b0b
    mythria authored
    This is to fix a bug in the bytecode graph builder. This cl adds a new merge
    node before we copy the environment on conditional/unconditional jumps. Since
    these environments could be merged later, we add a place holder merge so that
    the control dependencies are correctly merged. If we do not have a merge node
    we may incorrectly merge the dependencies into the previous block.
    For ex: test-run-variables/ContextStoreVariables in cctests.
    
    BUG=v8:4280
    LOG=N
    
    Review URL: https://codereview.chromium.org/1641143002
    
    Cr-Commit-Position: refs/heads/master@{#33591}
    d0e01b0b
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...