• 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...
build Loading commit data...
docs Loading commit data...
include Loading commit data...
infra Loading commit data...
samples Loading commit data...
src Loading commit data...
test Loading commit data...
testing Loading commit data...
third_party/binutils Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.gitignore Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.strongtalk Loading commit data...
LICENSE.v8 Loading commit data...
LICENSE.valgrind Loading commit data...
Makefile Loading commit data...
Makefile.android Loading commit data...
Makefile.nacl Loading commit data...
OWNERS Loading commit data...
PRESUBMIT.py Loading commit data...
README.md Loading commit data...
WATCHLISTS Loading commit data...
codereview.settings Loading commit data...
snapshot_toolchain.gni Loading commit data...