• Clemens Hammacher's avatar
    [Liftoff] Fewer pinned registers on store · 56b8ab5d
    Clemens Hammacher authored
    On ia32, we were pinning too many registers, resulting in no unpinned
    byte registers left (we only have three byte registers since {ebx}
    is reserved for the root register).
    It turns out that on most paths, we don't actually need to pin any
    registers, since {Store} is often the last call for an operation (like
    any store or set_global). If registers need to be pinned, only pass
    those that must be kept alive across the {Store}. This allows to
    compute a more narrow set of pinned registers on demand inside {Store}.
    
    Plus minor drive-by changes.
    
    R=titzer@chromium.org
    
    Bug: chromium:894374, chromium:894307, v8:6600
    Change-Id: Ic4d7131784c193dc7a2abf0e504d9973f6d5c5f1
    Reviewed-on: https://chromium-review.googlesource.com/c/1275819
    Commit-Queue: Clemens Hammacher <clemensh@chromium.org>
    Reviewed-by: 's avatarBen Titzer <titzer@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#56587}
    56b8ab5d
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...
torque Loading commit data...
unittests Loading commit data...
wasm-spec-tests Loading commit data...
webkit Loading commit data...
BUILD.gn Loading commit data...