• Ng Zhi An's avatar
    [wasm-simd][arm64][arm] Fix v128.const · 7c10560d
    Ng Zhi An authored
    There is a sign-extension bug happening when packing 2 32-bit ints into
    a 64-bit int. We are OR-ing int32_t with a uint64_t, so an integral
    conversion converts int32_t to uint64_t, which is a sign extension, and
    this gives unexpected results for a negative value:
    
        0x80000000 | uint64_t{0} -> 0xffffffff80000000
    
    What we want is 0x0000000080000000.
    
    Created a helper function to do this work of combining two uint32_t
    into one uint64_t. The use of this function will also ensure that
    if callers passed a int32_t, it would first be converted to a
    uint32_t, and will not have this sign extension bug.
    
    Sneaked a small regression test into the existing v128.const cctest,
    and also cleanup the loop to reset `expected` array to 0.
    
    Bug: chromium:1104033
    Change-Id: Icaca4c5ba42077dd4463697b9220cdbca9974b5e
    Reviewed-on: https://chromium-review.googlesource.com/c/v8/v8/+/2293044
    Commit-Queue: Zhi An Ng <zhin@chromium.org>
    Reviewed-by: 's avatarClemens Backes <clemensb@chromium.org>
    Reviewed-by: 's avatarBill Budge <bbudge@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#68850}
    7c10560d
Name
Last commit
Last update
..
OWNERS Loading commit data...
test-c-wasm-entry.cc Loading commit data...
test-compilation-cache.cc Loading commit data...
test-gc.cc Loading commit data...
test-grow-memory.cc Loading commit data...
test-jump-table-assembler.cc Loading commit data...
test-liftoff-inspection.cc Loading commit data...
test-run-wasm-64.cc Loading commit data...
test-run-wasm-asmjs.cc Loading commit data...
test-run-wasm-atomics.cc Loading commit data...
test-run-wasm-atomics64.cc Loading commit data...
test-run-wasm-bulk-memory.cc Loading commit data...
test-run-wasm-exceptions.cc Loading commit data...
test-run-wasm-interpreter.cc Loading commit data...
test-run-wasm-js.cc Loading commit data...
test-run-wasm-module.cc Loading commit data...
test-run-wasm-sign-extension.cc Loading commit data...
test-run-wasm-simd-liftoff.cc Loading commit data...
test-run-wasm-simd-scalar-lowering.cc Loading commit data...
test-run-wasm-simd.cc Loading commit data...
test-run-wasm.cc Loading commit data...
test-streaming-compilation.cc Loading commit data...
test-wasm-breakpoints.cc Loading commit data...
test-wasm-codegen.cc Loading commit data...
test-wasm-debug-evaluate.cc Loading commit data...
test-wasm-import-wrapper-cache.cc Loading commit data...
test-wasm-serialization.cc Loading commit data...
test-wasm-shared-engine.cc Loading commit data...
test-wasm-stack.cc Loading commit data...
test-wasm-trap-position.cc Loading commit data...
wasm-atomics-utils.h Loading commit data...
wasm-run-utils.cc Loading commit data...
wasm-run-utils.h Loading commit data...