[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers
From: |
Richard Henderson |
Subject: |
Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers |
Date: |
Mon, 5 Sep 2022 18:20:07 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 |
On 9/5/22 17:31, Víctor Colombo wrote:
I have a message bookmarked here ([1]), but I don't know if there is a
previous one with a more in depth scheme.
There may have been a previous, but that's the one I was thinking of.
Anyway, I was also analyzing recently the idea of removing all these
reset_fpstatus() calls from instructions helpers. I think this would
require to actually call it from the end of the (previous) instructions instead of the
beginning? Like adding the call to
do_float_check_status() and float_invalid_op_*() as a focal point to
'hide' the calls to reset_fpstatus().
Well, there would of course be no separate call, but do_float_check_status
would:
int status = get_float_exception_flags(&env->fp_status);
set_float_exception_flags(0, &env->fp_status);
straight away. No extra call overhead, and the steady-state of softfp exception flags
outside of an in-progress fp operation is 0.
However there are also insns
helpers that don't call these auxiliary functions, which I think would
cause the refactor to not be worthy overall.
Anything that can raise a softfp exception and doesn't do something with it, either
immediately within the same helper, or just afterward with helper_float_check_status, is
buggy. With those fixed, helper_reset_fpstatus may be removed entirely.
r~
- [PATCH 0/3] target/ppc: Move fsqrt[s] to decodetree, Víctor Colombo, 2022/09/05
- [PATCH 1/3] target/ppc: Move fsqrt to decodetree, Víctor Colombo, 2022/09/05
- [PATCH 2/3] target/ppc: Move fsqrts to decodetree, Víctor Colombo, 2022/09/05
- [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers, Víctor Colombo, 2022/09/05
- Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers, Richard Henderson, 2022/09/05
- Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers, Víctor Colombo, 2022/09/05
- Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers, Richard Henderson, 2022/09/05
- Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers, Víctor Colombo, 2022/09/05
- Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers,
Richard Henderson <=
- Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers, Víctor Colombo, 2022/09/06
- Re: [PATCH 3/3] target/ppc: Merge fsqrt and fsqrts helpers, Richard Henderson, 2022/09/08
Re: [PATCH 0/3] target/ppc: Move fsqrt[s] to decodetree, Daniel Henrique Barboza, 2022/09/06