• Clemens Hammacher's avatar
    [Liftoff] Fix stack pointer corruption · cc862e69
    Clemens Hammacher authored
    During a C call, a previous value of the stack pointer is stored in a
    platform specific callee saved register. Loading the out argument of the
    C call might overwrite the value in that register, if the destination
    register collides with the platform specific register. Hence, do first
    use that register to restore the previous stack pointer, and only then
    load the out argument.
    Similarly, when pushing arguments to the stack, do first push all
    values and then set the platform specific register in order to avoid
    overwriting an argument value held in that register.
    
    Drive-by: Fix offset computations for parameters pushed to the stack
    for c calls.
    
    R=titzer@chromium.org
    
    Bug: chromium:820802,chromium:820896,chromium:820807,v8:6600
    Change-Id: If4567467b7912454f0bd2cad5927233c98894b03
    Reviewed-on: https://chromium-review.googlesource.com/959064Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#51916}
    cc862e69
Name
Last commit
Last update
..
benchmarks Loading commit data...
cctest Loading commit data...
common Loading commit data...
debugger Loading commit data...
fuzzer 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...
preparser Loading commit data...
test262 Loading commit data...
unittests Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...
bot_default.isolate Loading commit data...
d8_default.isolate Loading commit data...
default.isolate Loading commit data...
optimize_for_size.isolate Loading commit data...
perf.isolate Loading commit data...