• Thibaud Michaud's avatar
    [regalloc] Fix slot requirement for live ranges defined by a const · 0ee6f90a
    Thibaud Michaud authored
    Live ranges defined by a constant operand normally don't require a spill
    slot since they can just rematerialize the value from the constant. In
    the attached issue however, deoptimization adds an explicit slot
    requirement for a range that is defined by a constant operand. This case
    is not expected in the register allocator and we eventually hit a
    DCHECK.
    
    This fix allocates a new stack slot during the MeetRegisterConstraints
    and adds the missing gap move.
    
    Drive-by: remove dead method LiveRange::NextSlotPosition.
    
    R=sigurds@chromium.org
    CC=​nicohartmann@chromium.org
    
    Bug: chromium:1146880
    Change-Id: I08fbb890f2f3d9574196989cf3e5ef6232433484
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2563689Reviewed-by: 's avatarSigurd Schneider <sigurds@chromium.org>
    Commit-Queue: Thibaud Michaud <thibaudm@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#73510}
    0ee6f90a
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
debugging Loading commit data...
fuzzer Loading commit data...
fuzzilli Loading commit data...
inspector 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...
mkgrokdump Loading commit data...
mozilla Loading commit data...
test262 Loading commit data...
torque Loading commit data...
unittests Loading commit data...
wasm-api-tests Loading commit data...
wasm-js Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
OWNERS Loading commit data...