Re: [PATCH 1/2] target/riscv/vector_helper.c: skip set tail when vta is zero

2023-05-07 Thread Alistair Francis
On Fri, Apr 28, 2023 at 6:58 AM Daniel Henrique Barboza wrote: > > The function is a no-op if 'vta' is zero but we're still doing a lot of > stuff in this function regardless. vext_set_elems_1s() will ignore every > single time (since vta is zero) and we just wasted time. > > Skip it altogether in

Re: [PATCH 1/2] target/riscv/vector_helper.c: skip set tail when vta is zero

2023-05-07 Thread Alistair Francis
On Fri, Apr 28, 2023 at 6:58 AM Daniel Henrique Barboza wrote: > > The function is a no-op if 'vta' is zero but we're still doing a lot of > stuff in this function regardless. vext_set_elems_1s() will ignore every > single time (since vta is zero) and we just wasted time. > > Skip it altogether in

Re: [PATCH 1/2] target/riscv/vector_helper.c: skip set tail when vta is zero

2023-04-27 Thread Weiwei Li
On 2023/4/28 04:57, Daniel Henrique Barboza wrote: The function is a no-op if 'vta' is zero but we're still doing a lot of stuff in this function regardless. vext_set_elems_1s() will ignore every single time (since vta is zero) and we just wasted time. Skip it altogether in this case. Aside fr

[PATCH 1/2] target/riscv/vector_helper.c: skip set tail when vta is zero

2023-04-27 Thread Daniel Henrique Barboza
The function is a no-op if 'vta' is zero but we're still doing a lot of stuff in this function regardless. vext_set_elems_1s() will ignore every single time (since vta is zero) and we just wasted time. Skip it altogether in this case. Aside from the code simplification there's a noticeable emulati