From: David Gibson <david@gibson.dropbear.id.au>
To: Richard Henderson <richard.henderson@linaro.org>
Cc: qemu-devel@nongnu.org, mark.cave-ayland@ilande.co.uk,
qemu-ppc@nongnu.org
Subject: Re: [Qemu-devel] [PATCH 15/34] target/ppc: merge ppc_vsr_t and ppc_avr_t union types
Date: Wed, 19 Dec 2018 17:21:53 +1100 [thread overview]
Message-ID: <20181219062153.GK30570@umbus.fritz.box> (raw)
In-Reply-To: <20181218063911.2112-16-richard.henderson@linaro.org>
[-- Attachment #1: Type: text/plain, Size: 11075 bytes --]
On Mon, Dec 17, 2018 at 10:38:52PM -0800, Richard Henderson wrote:
> From: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
>
> Since the VSX registers are actually a superset of the VMX registers then they
> can be represented by the same type. Merge ppc_avr_t into ppc_vsr_t and change
> ppc_avr_t to be a simple typedef alias.
>
> Note that due to a difference in the naming of the float32 member between
> ppc_avr_t and ppc_vsr_t, references to the ppc_avr_t f member must be replaced
> with f32 instead.
>
> Signed-off-by: Mark Cave-Ayland <mark.cave-ayland@ilande.co.uk>
Acked-by: David Gibson <david@gibson.dropbear.id.au>
> Message-Id: <20181217122405.18732-7-mark.cave-ayland@ilande.co.uk>
> ---
> target/ppc/cpu.h | 17 +++++++------
> target/ppc/internal.h | 11 --------
> target/ppc/int_helper.c | 56 +++++++++++++++++++++--------------------
> 3 files changed, 39 insertions(+), 45 deletions(-)
>
> diff --git a/target/ppc/cpu.h b/target/ppc/cpu.h
> index ab68abe8a2..5445d4c3c1 100644
> --- a/target/ppc/cpu.h
> +++ b/target/ppc/cpu.h
> @@ -230,7 +230,6 @@ typedef struct opc_handler_t opc_handler_t;
> /* Types used to describe some PowerPC registers etc. */
> typedef struct DisasContext DisasContext;
> typedef struct ppc_spr_t ppc_spr_t;
> -typedef union ppc_avr_t ppc_avr_t;
> typedef union ppc_tlb_t ppc_tlb_t;
> typedef struct ppc_hash_pte64 ppc_hash_pte64_t;
>
> @@ -254,22 +253,26 @@ struct ppc_spr_t {
> #endif
> };
>
> -/* Altivec registers (128 bits) */
> -union ppc_avr_t {
> - float32 f[4];
> +/* VSX/Altivec registers (128 bits) */
> +typedef union _ppc_vsr_t {
> uint8_t u8[16];
> uint16_t u16[8];
> uint32_t u32[4];
> + uint64_t u64[2];
> int8_t s8[16];
> int16_t s16[8];
> int32_t s32[4];
> - uint64_t u64[2];
> int64_t s64[2];
> + float32 f32[4];
> + float64 f64[2];
> + float128 f128;
> #ifdef CONFIG_INT128
> __uint128_t u128;
> #endif
> - Int128 s128;
> -};
> + Int128 s128;
> +} ppc_vsr_t;
> +
> +typedef ppc_vsr_t ppc_avr_t;
>
> #if !defined(CONFIG_USER_ONLY)
> /* Software TLB cache */
> diff --git a/target/ppc/internal.h b/target/ppc/internal.h
> index a9bcadff42..b4b1f7b3db 100644
> --- a/target/ppc/internal.h
> +++ b/target/ppc/internal.h
> @@ -204,17 +204,6 @@ EXTRACT_HELPER(IMM8, 11, 8);
> EXTRACT_HELPER(DCMX, 16, 7);
> EXTRACT_HELPER_SPLIT_3(DCMX_XV, 5, 16, 0, 1, 2, 5, 1, 6, 6);
>
> -typedef union _ppc_vsr_t {
> - uint8_t u8[16];
> - uint16_t u16[8];
> - uint32_t u32[4];
> - uint64_t u64[2];
> - float32 f32[4];
> - float64 f64[2];
> - float128 f128;
> - Int128 s128;
> -} ppc_vsr_t;
> -
> #if defined(HOST_WORDS_BIGENDIAN)
> #define VsrB(i) u8[i]
> #define VsrH(i) u16[i]
> diff --git a/target/ppc/int_helper.c b/target/ppc/int_helper.c
> index fcac90a4a9..9d715be25c 100644
> --- a/target/ppc/int_helper.c
> +++ b/target/ppc/int_helper.c
> @@ -548,8 +548,8 @@ VARITH_DO(muluwm, *, u32)
> { \
> int i; \
> \
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) { \
> - r->f[i] = func(a->f[i], b->f[i], &env->vec_status); \
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) { \
> + r->f32[i] = func(a->f32[i], b->f32[i], &env->vec_status); \
> } \
> }
> VARITHFP(addfp, float32_add)
> @@ -563,9 +563,9 @@ VARITHFP(maxfp, float32_max)
> ppc_avr_t *b, ppc_avr_t *c) \
> { \
> int i; \
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) { \
> - r->f[i] = float32_muladd(a->f[i], c->f[i], b->f[i], \
> - type, &env->vec_status); \
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) { \
> + r->f32[i] = float32_muladd(a->f32[i], c->f32[i], b->f32[i], \
> + type, &env->vec_status); \
> } \
> }
> VARITHFPFMA(maddfp, 0);
> @@ -670,9 +670,9 @@ VABSDU(w, u32)
> { \
> int i; \
> \
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) { \
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) { \
> float32 t = cvt(b->element[i], &env->vec_status); \
> - r->f[i] = float32_scalbn(t, -uim, &env->vec_status); \
> + r->f32[i] = float32_scalbn(t, -uim, &env->vec_status); \
> } \
> }
> VCF(ux, uint32_to_float32, u32)
> @@ -782,9 +782,9 @@ VCMPNE(w, u32, uint32_t, 0)
> uint32_t none = 0; \
> int i; \
> \
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) { \
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) { \
> uint32_t result; \
> - int rel = float32_compare_quiet(a->f[i], b->f[i], \
> + int rel = float32_compare_quiet(a->f32[i], b->f32[i], \
> &env->vec_status); \
> if (rel == float_relation_unordered) { \
> result = 0; \
> @@ -816,14 +816,16 @@ static inline void vcmpbfp_internal(CPUPPCState *env, ppc_avr_t *r,
> int i;
> int all_in = 0;
>
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) {
> - int le_rel = float32_compare_quiet(a->f[i], b->f[i], &env->vec_status);
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) {
> + int le_rel = float32_compare_quiet(a->f32[i], b->f32[i],
> + &env->vec_status);
> if (le_rel == float_relation_unordered) {
> r->u32[i] = 0xc0000000;
> all_in = 1;
> } else {
> - float32 bneg = float32_chs(b->f[i]);
> - int ge_rel = float32_compare_quiet(a->f[i], bneg, &env->vec_status);
> + float32 bneg = float32_chs(b->f32[i]);
> + int ge_rel = float32_compare_quiet(a->f32[i], bneg,
> + &env->vec_status);
> int le = le_rel != float_relation_greater;
> int ge = ge_rel != float_relation_less;
>
> @@ -856,11 +858,11 @@ void helper_vcmpbfp_dot(CPUPPCState *env, ppc_avr_t *r, ppc_avr_t *a,
> float_status s = env->vec_status; \
> \
> set_float_rounding_mode(float_round_to_zero, &s); \
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) { \
> - if (float32_is_any_nan(b->f[i])) { \
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) { \
> + if (float32_is_any_nan(b->f32[i])) { \
> r->element[i] = 0; \
> } else { \
> - float64 t = float32_to_float64(b->f[i], &s); \
> + float64 t = float32_to_float64(b->f32[i], &s); \
> int64_t j; \
> \
> t = float64_scalbn(t, uim, &s); \
> @@ -1661,8 +1663,8 @@ void helper_vrefp(CPUPPCState *env, ppc_avr_t *r, ppc_avr_t *b)
> {
> int i;
>
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) {
> - r->f[i] = float32_div(float32_one, b->f[i], &env->vec_status);
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) {
> + r->f32[i] = float32_div(float32_one, b->f32[i], &env->vec_status);
> }
> }
>
> @@ -1674,8 +1676,8 @@ void helper_vrefp(CPUPPCState *env, ppc_avr_t *r, ppc_avr_t *b)
> float_status s = env->vec_status; \
> \
> set_float_rounding_mode(rounding, &s); \
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) { \
> - r->f[i] = float32_round_to_int (b->f[i], &s); \
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) { \
> + r->f32[i] = float32_round_to_int (b->f32[i], &s); \
> } \
> }
> VRFI(n, float_round_nearest_even)
> @@ -1705,10 +1707,10 @@ void helper_vrsqrtefp(CPUPPCState *env, ppc_avr_t *r, ppc_avr_t *b)
> {
> int i;
>
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) {
> - float32 t = float32_sqrt(b->f[i], &env->vec_status);
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) {
> + float32 t = float32_sqrt(b->f32[i], &env->vec_status);
>
> - r->f[i] = float32_div(float32_one, t, &env->vec_status);
> + r->f32[i] = float32_div(float32_one, t, &env->vec_status);
> }
> }
>
> @@ -1751,8 +1753,8 @@ void helper_vexptefp(CPUPPCState *env, ppc_avr_t *r, ppc_avr_t *b)
> {
> int i;
>
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) {
> - r->f[i] = float32_exp2(b->f[i], &env->vec_status);
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) {
> + r->f32[i] = float32_exp2(b->f32[i], &env->vec_status);
> }
> }
>
> @@ -1760,8 +1762,8 @@ void helper_vlogefp(CPUPPCState *env, ppc_avr_t *r, ppc_avr_t *b)
> {
> int i;
>
> - for (i = 0; i < ARRAY_SIZE(r->f); i++) {
> - r->f[i] = float32_log2(b->f[i], &env->vec_status);
> + for (i = 0; i < ARRAY_SIZE(r->f32); i++) {
> + r->f32[i] = float32_log2(b->f32[i], &env->vec_status);
> }
> }
>
--
David Gibson | I'll have my music baroque, and my code
david AT gibson.dropbear.id.au | minimalist, thank you. NOT _the_ _other_
| _way_ _around_!
http://www.ozlabs.org/~dgibson
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
next prev parent reply other threads:[~2018-12-19 7:49 UTC|newest]
Thread overview: 75+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-18 6:38 [Qemu-devel] [PATCH 00/34] tcg, target/ppc vector improvements Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 01/34] tcg: Add logical simplifications during gvec expand Richard Henderson
2018-12-19 5:36 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 02/34] target/arm: Rely on optimization within tcg_gen_gvec_or Richard Henderson
2018-12-19 5:37 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 03/34] tcg: Add gvec expanders for nand, nor, eqv Richard Henderson
2018-12-19 5:39 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 04/34] tcg: Add write_aofs to GVecGen4 Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 05/34] tcg: Add opcodes for vector saturated arithmetic Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 06/34] tcg/i386: Implement vector saturating arithmetic Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 07/34] tcg: Add opcodes for vector minmax arithmetic Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 08/34] tcg/i386: Implement " Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 09/34] target/arm: Use vector minmax expanders for aarch64 Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 10/34] target/arm: Use vector minmax expanders for aarch32 Richard Henderson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 11/34] target/ppc: introduce get_fpr() and set_fpr() helpers for FP register access Richard Henderson
2018-12-19 6:15 ` David Gibson
2018-12-19 12:29 ` Mark Cave-Ayland
2018-12-20 16:52 ` Mark Cave-Ayland
2018-12-18 6:38 ` [Qemu-devel] [PATCH 12/34] target/ppc: introduce get_avr64() and set_avr64() helpers for VMX " Richard Henderson
2018-12-19 6:15 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 13/34] target/ppc: introduce get_cpu_vsr{l, h}() and set_cpu_vsr{l, h}() helpers for VSR " Richard Henderson
2018-12-19 6:17 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 14/34] target/ppc: switch FPR, VMX and VSX helpers to access data directly from cpu_env Richard Henderson
2018-12-19 6:20 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 15/34] target/ppc: merge ppc_vsr_t and ppc_avr_t union types Richard Henderson
2018-12-19 6:21 ` David Gibson [this message]
2018-12-18 6:38 ` [Qemu-devel] [PATCH 16/34] target/ppc: move FP and VMX registers into aligned vsr register array Richard Henderson
2018-12-19 6:27 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 17/34] target/ppc: convert VMX logical instructions to use vector operations Richard Henderson
2018-12-19 6:29 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 18/34] target/ppc: convert vaddu[b, h, w, d] and vsubu[b, h, w, d] over " Richard Henderson
2018-12-19 6:29 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 19/34] target/ppc: convert vspltis[bhw] " Richard Henderson
2018-12-19 6:31 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 20/34] target/ppc: convert vsplt[bhw] " Richard Henderson
2018-12-19 6:32 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 21/34] target/ppc: nand, nor, eqv are now generic " Richard Henderson
2018-12-19 6:32 ` David Gibson
2018-12-18 6:38 ` [Qemu-devel] [PATCH 22/34] target/ppc: convert VSX logical operations to " Richard Henderson
2018-12-19 6:33 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 23/34] target/ppc: convert xxspltib " Richard Henderson
2018-12-19 6:34 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 24/34] target/ppc: convert xxspltw " Richard Henderson
2018-12-19 6:35 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 25/34] target/ppc: convert xxsel " Richard Henderson
2018-12-19 6:35 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 26/34] target/ppc: Pass integer to helper_mtvscr Richard Henderson
2018-12-19 6:37 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 27/34] target/ppc: Use helper_mtvscr for reset and gdb Richard Henderson
2018-12-19 6:38 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 28/34] target/ppc: Remove vscr_nj and vscr_sat Richard Henderson
2018-12-19 6:38 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 29/34] target/ppc: Add helper_mfvscr Richard Henderson
2018-12-19 6:39 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 30/34] target/ppc: Use mtvscr/mfvscr for vmstate Richard Henderson
2018-12-19 6:40 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 31/34] target/ppc: Add set_vscr_sat Richard Henderson
2018-12-19 6:40 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 32/34] target/ppc: Split out VSCR_SAT to a vector field Richard Henderson
2018-12-19 6:41 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 33/34] target/ppc: convert vadd*s and vsub*s to vector operations Richard Henderson
2018-12-19 6:42 ` David Gibson
2018-12-18 6:39 ` [Qemu-devel] [PATCH 34/34] target/ppc: convert vmin* and vmax* " Richard Henderson
2018-12-19 6:42 ` David Gibson
2018-12-18 9:49 ` [Qemu-devel] [PATCH 00/34] tcg, target/ppc vector improvements Mark Cave-Ayland
2018-12-18 14:51 ` Mark Cave-Ayland
2018-12-18 15:07 ` Richard Henderson
2018-12-18 15:22 ` Mark Cave-Ayland
2018-12-18 15:05 ` Mark Cave-Ayland
2018-12-18 15:17 ` Richard Henderson
2018-12-18 15:26 ` Mark Cave-Ayland
2018-12-18 16:16 ` Richard Henderson
2019-01-03 14:58 ` Mark Cave-Ayland
2019-01-03 18:31 ` Mark Cave-Ayland
2019-01-04 22:33 ` Richard Henderson
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20181219062153.GK30570@umbus.fritz.box \
--to=david@gibson.dropbear.id.au \
--cc=mark.cave-ayland@ilande.co.uk \
--cc=qemu-devel@nongnu.org \
--cc=qemu-ppc@nongnu.org \
--cc=richard.henderson@linaro.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).