• Milad Fa's avatar
    Fix LoadSpillAddress on big endian · ac0cedf1
    Milad Fa authored
    BE machines use a 4 byte bias to spill/fill 32-bit values on
    the stack. This is done so because TF always fills 64-bit values
    even if the spilled value was 32-bits. To make sure this holds between
    LO and TF we have added a 4 byte bias in this CL:
    crrev.com/c/2756712
    
    LoadSpillAddress needs to also take this into account and
    add a bias if the spilled value was 4 bytes.
    
    Change-Id: Ibd2b2071ce1fb11a9c5884611ae8edd1f17cb0c9
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/3891196
    Commit-Queue: Milad Farazmand <mfarazma@redhat.com>
    Reviewed-by: 's avatarThibaud Michaud <thibaudm@chromium.org>
    Cr-Commit-Position: refs/heads/main@{#83163}
    ac0cedf1
liftoff-assembler-arm.h 175 KB