• danno's avatar
    [mips] Fix code generation bug in handling of immediates · abd9610e
    danno authored
    When testing turning --verify-csa off to generate better code for builtins, mips
    started failing mksnapshot due to an assert in instruction-selection-mips.cc's
    VisitBinop routine, which creates a buffer to hold InstructionOperand of size 4
    that would be overflowed. This fix is somewhat speculative, assuming that either
    the LHS or the RHS operand of a mips binary op can be an immediate (as opposed
    to the current code which seems to have a code path where both the left and
    right operands are added, leading to the buffer overflow). With this fix, the
    assert doesn't fire and all of the mips tests run through successfully in debug
    mode.
    
    R=ishell@chromium.org
    TBR=dusan.m.milosavljevic@gmail.com
    
    Review-Url: https://codereview.chromium.org/2647283009
    Cr-Commit-Position: refs/heads/master@{#42701}
    abd9610e
Name
Last commit
Last update
benchmarks Loading commit data...
build_overrides Loading commit data...
docs Loading commit data...
gni Loading commit data...
gypfiles 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 Loading commit data...
tools Loading commit data...
.clang-format Loading commit data...
.gitignore Loading commit data...
.gn Loading commit data...
.ycm_extra_conf.py Loading commit data...
AUTHORS Loading commit data...
BUILD.gn Loading commit data...
CODE_OF_CONDUCT.md Loading commit data...
ChangeLog Loading commit data...
DEPS Loading commit data...
LICENSE Loading commit data...
LICENSE.fdlibm 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...
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...