[v3] target/arm: Use tcg_constant_*

[PATCH v3 0/5] target/arm: Use tcg_constant_*

Philippe Mathieu-Daudé posted 5 patches 4 years, 3 months ago

Diff against v1 v2
Download series mbox

Test checkpatch passed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/20211029231834.2476117-1-f4bug@amsat.org

target/arm/translate-a32.h |  3 +++
target/arm/translate-sve.c | 17 ++++++++---------
target/arm/translate.c     | 27 ++++++++-------------------
3 files changed, 19 insertions(+), 28 deletions(-)

Expand all Fold all

[PATCH v3 0/5] target/arm: Use tcg_constant_*

Posted by Philippe Mathieu-Daudé 4 years, 3 months ago

Missing review: patches #2 & #5 (new)

Introduce store_cpu_field_constant() helper to avoid using temporary
when the value is constant (and read-only).

Since v2:
- Simplify store_cpu_field_constant(), do not modify store_cpu_field()
- Added another use in gen_rev16()

Philippe Mathieu-Daudé (5):
  target/arm: Use tcg_constant_i32() in op_smlad()
  target/arm: Introduce store_cpu_field_constant() helper
  target/arm: Use the constant variant of store_cpu_field() when
    possible
  target/arm: Use tcg_constant_i64() in do_sat_addsub_64()
  target/arm: Use tcg_constant_i32() in gen_rev16()

 target/arm/translate-a32.h |  3 +++
 target/arm/translate-sve.c | 17 ++++++++---------
 target/arm/translate.c     | 27 ++++++++-------------------
 3 files changed, 19 insertions(+), 28 deletions(-)

-- 
2.31.1

Re: [PATCH v3 0/5] target/arm: Use tcg_constant_*

Posted by Richard Henderson 4 years, 3 months ago

On 10/29/21 7:18 PM, Philippe Mathieu-Daudé wrote:
> Missing review: patches #2 & #5 (new)
> 
> Introduce store_cpu_field_constant() helper to avoid using temporary
> when the value is constant (and read-only).
> 
> Since v2:
> - Simplify store_cpu_field_constant(), do not modify store_cpu_field()
> - Added another use in gen_rev16()
> 
> Philippe Mathieu-Daudé (5):
>    target/arm: Use tcg_constant_i32() in op_smlad()
>    target/arm: Introduce store_cpu_field_constant() helper
>    target/arm: Use the constant variant of store_cpu_field() when
>      possible
>    target/arm: Use tcg_constant_i64() in do_sat_addsub_64()
>    target/arm: Use tcg_constant_i32() in gen_rev16()
> 
>   target/arm/translate-a32.h |  3 +++
>   target/arm/translate-sve.c | 17 ++++++++---------
>   target/arm/translate.c     | 27 ++++++++-------------------
>   3 files changed, 19 insertions(+), 28 deletions(-)

Thanks, applied to target-arm.next.


r~