Re: [PATCH v2] target/i386: Use assert() to sanity-check b1 in SSE decode

2021-12-09 Thread Peter Maydell
Gave up pinging for i386 maintainers; will take this via target-arm.next. thanks -- PMM On Mon, 15 Nov 2021 at 14:38, Peter Maydell wrote: > > Ping^4. Who is collecting target/i386 patches these days ? > > -- PMM > > On Mon, 1 Nov 2021 at 16:18, Peter Maydell wrote: > > > > Ping^3, now 2

Re: [PATCH v2] target/i386: Use assert() to sanity-check b1 in SSE decode

2021-11-15 Thread Peter Maydell
Ping^4. Who is collecting target/i386 patches these days ? -- PMM On Mon, 1 Nov 2021 at 16:18, Peter Maydell wrote: > > Ping^3, now 2 months after patch posted and reviewed... > > -- PMM > > On Mon, 27 Sept 2021 at 11:03, Peter Maydell wrote: > > > > Ping^2 ! > > > > thanks > > -- PMM > > > >

Re: [PATCH v2] target/i386: Use assert() to sanity-check b1 in SSE decode

2021-11-01 Thread Peter Maydell
Ping^3, now 2 months after patch posted and reviewed... -- PMM On Mon, 27 Sept 2021 at 11:03, Peter Maydell wrote: > > Ping^2 ! > > thanks > -- PMM > > On Mon, 13 Sept 2021 at 13:34, Peter Maydell wrote: > > > > Ping? (this has been reviewed) > > > > thanks > > -- PMM > > > > On Wed, 1 Sept

Re: [PATCH v2] target/i386: Use assert() to sanity-check b1 in SSE decode

2021-09-27 Thread Peter Maydell
Ping^2 ! thanks -- PMM On Mon, 13 Sept 2021 at 13:34, Peter Maydell wrote: > > Ping? (this has been reviewed) > > thanks > -- PMM > > On Wed, 1 Sept 2021 at 15:10, Peter Maydell wrote: > > > > In the SSE decode function gen_sse(), we combine a byte > > 'b' and a value 'b1' which can be [0..3],

Re: [PATCH v2] target/i386: Use assert() to sanity-check b1 in SSE decode

2021-09-13 Thread Peter Maydell
Ping? (this has been reviewed) thanks -- PMM On Wed, 1 Sept 2021 at 15:10, Peter Maydell wrote: > > In the SSE decode function gen_sse(), we combine a byte > 'b' and a value 'b1' which can be [0..3], and switch on them: >b |= (b1 << 8); >switch (b) { >... >default: >

Re: [PATCH v2] target/i386: Use assert() to sanity-check b1 in SSE decode

2021-09-03 Thread Richard Henderson
On 9/1/21 4:10 PM, Peter Maydell wrote: In the SSE decode function gen_sse(), we combine a byte 'b' and a value 'b1' which can be [0..3], and switch on them: b |= (b1 << 8); switch (b) { ... default: unknown_op: gen_unknown_opcode(env, s); return; } In

[PATCH v2] target/i386: Use assert() to sanity-check b1 in SSE decode

2021-09-01 Thread Peter Maydell
In the SSE decode function gen_sse(), we combine a byte 'b' and a value 'b1' which can be [0..3], and switch on them: b |= (b1 << 8); switch (b) { ... default: unknown_op: gen_unknown_opcode(env, s); return; } In three cases inside this switch, we were then also