Re: [Qemu-devel] [PATCH v1 1/1] target/arm: Fix vector operation segfault

2019-05-14 Thread Alistair Francis
On Mon, May 13, 2019 at 5:19 PM Richard Henderson wrote: > > On 5/13/19 5:08 PM, Alistair Francis wrote: > > We hit the second switch case as > > NEON_3R_VQADD and NEON_3R_VQSUB don't return from the function in the > > first switch case. > > That's the bug, not here in this second switch. Ah

Re: [Qemu-devel] [PATCH v1 1/1] target/arm: Fix vector operation segfault

2019-05-13 Thread Richard Henderson
On 5/13/19 5:08 PM, Alistair Francis wrote: > We hit the second switch case as > NEON_3R_VQADD and NEON_3R_VQSUB don't return from the function in the > first switch case. That's the bug, not here in this second switch. r~

[Qemu-devel] [PATCH v1 1/1] target/arm: Fix vector operation segfault

2019-05-13 Thread Alistair Francis
Commit 89e68b575 "target/arm: Use vector operations for saturation" causes this abort() when booting QEMU ARM with a Cortex-A15: 0 0x74c2382f in raise () at /usr/lib/libc.so.6 1 0x74c0e672 in abort () at /usr/lib/libc.so.6 2 0x559c1839 in disas_neon_data_insn (insn=,