On Tue, Nov 22, 2016 at 05:16:00PM +0530, Nikunj A Dadhania wrote:
> From: Bharata B Rao <bhar...@linux.vnet.ibm.com>
> 
> xscmpoqp - VSX Scalar Compare Ordered Quad-Precision
> xscmpuqp - VSX Scalar Compare Unordered Quad-Precision
> 
> Signed-off-by: Bharata B Rao <bhar...@linux.vnet.ibm.com>
> Signed-off-by: Nikunj A Dadhania <nik...@linux.vnet.ibm.com>
> ---
>  target-ppc/fpu_helper.c             | 52 
> +++++++++++++++++++++++++++++++++++++
>  target-ppc/helper.h                 |  2 ++
>  target-ppc/translate/vsx-impl.inc.c |  2 ++
>  target-ppc/translate/vsx-ops.inc.c  |  2 ++
>  4 files changed, 58 insertions(+)
> 
> diff --git a/target-ppc/fpu_helper.c b/target-ppc/fpu_helper.c
> index b1c5a07..28c1fea 100644
> --- a/target-ppc/fpu_helper.c
> +++ b/target-ppc/fpu_helper.c
> @@ -2518,6 +2518,58 @@ void helper_##op(CPUPPCState *env, uint32_t opcode)    
>                   \
>  VSX_SCALAR_CMP(xscmpodp, 1)
>  VSX_SCALAR_CMP(xscmpudp, 0)
>  
> +#define VSX_SCALAR_CMPQ(op, ordered)                                    \
> +void helper_##op(CPUPPCState *env, uint32_t opcode)                     \
> +{                                                                       \
> +    ppc_vsr_t xa, xb;                                                   \
> +    uint32_t cc = 0;                                                    \
> +    bool vxsnan_flag = false, vxvc_flag = false;                        \
> +    float128 a, b;                                                      \
> +                                                                        \
> +    helper_reset_fpstatus(env);                                         \
> +    getVSR(rA(opcode) + 32, &xa, env);                                  \
> +    getVSR(rB(opcode) + 32, &xb, env);                                  \
> +                                                                        \
> +    a = make_float128(xa.VsrD(0), xa.VsrD(1));                          \
> +    b = make_float128(xb.VsrD(0), xb.VsrD(1));                          \
> +                                                                        \
> +    if (float128_is_signaling_nan(a, &env->fp_status) ||                \
> +        float128_is_signaling_nan(b, &env->fp_status)) {                \
> +        vxsnan_flag = true;                                             \
> +        cc = 1;                                                         \
> +        if (fpscr_ve == 0 && ordered) {                                 \
> +            vxvc_flag = true;                                           \
> +        }                                                               \
> +    } else if (ordered && (float128_is_quiet_nan(a, &env->fp_status)    \
> +                           || float128_is_quiet_nan(b, &env->fp_status))) { \
> +        cc = 1;                                                         \

Please use symbolic constants for the CC bits.

> +        vxvc_flag = true;                                               \
> +    }                                                                   \
> +    if (vxsnan_flag) {                                                  \
> +        float_invalid_op_excp(env, POWERPC_EXCP_FP_VXSNAN, 0);          \
> +    }                                                                   \
> +    if (vxvc_flag) {                                                    \
> +        float_invalid_op_excp(env, POWERPC_EXCP_FP_VXVC, 0);            \
> +    }                                                                   \
> +                                                                        \
> +    if (float128_lt(a, b, &env->fp_status)) {                           \
> +        cc |= 8;                                                        \
> +    } else if (!float128_le(a, b, &env->fp_status)) {                   \
> +        cc |= 4;                                                        \
> +    } else {                                                            \
> +        cc |= 2;                                                        \
> +    }                                                                   \
> +                                                                        \
> +    env->fpscr &= ~(0x0F << FPSCR_FPRF);                                \
> +    env->fpscr |= cc << FPSCR_FPRF;                                     \
> +    env->crf[BF(opcode)] = cc;                                          \
> +                                                                        \
> +    float_check_status(env);                                            \
> +}
> +
> +VSX_SCALAR_CMPQ(xscmpoqp, 1)
> +VSX_SCALAR_CMPQ(xscmpuqp, 0)
> +
>  /* VSX_MAX_MIN - VSX floating point maximum/minimum
>   *   name  - instruction mnemonic
>   *   op    - operation (max or min)
> diff --git a/target-ppc/helper.h b/target-ppc/helper.h
> index ba42015..3b26678 100644
> --- a/target-ppc/helper.h
> +++ b/target-ppc/helper.h
> @@ -408,6 +408,8 @@ DEF_HELPER_2(xscmpexpdp, void, env, i32)
>  DEF_HELPER_2(xscmpexpqp, void, env, i32)
>  DEF_HELPER_2(xscmpodp, void, env, i32)
>  DEF_HELPER_2(xscmpudp, void, env, i32)
> +DEF_HELPER_2(xscmpoqp, void, env, i32)
> +DEF_HELPER_2(xscmpuqp, void, env, i32)
>  DEF_HELPER_2(xsmaxdp, void, env, i32)
>  DEF_HELPER_2(xsmindp, void, env, i32)
>  DEF_HELPER_2(xscvdpsp, void, env, i32)
> diff --git a/target-ppc/translate/vsx-impl.inc.c 
> b/target-ppc/translate/vsx-impl.inc.c
> index 5206258..ed9588e 100644
> --- a/target-ppc/translate/vsx-impl.inc.c
> +++ b/target-ppc/translate/vsx-impl.inc.c
> @@ -628,6 +628,8 @@ GEN_VSX_HELPER_2(xscmpexpdp, 0x0C, 0x07, 0, PPC2_ISA300)
>  GEN_VSX_HELPER_2(xscmpexpqp, 0x04, 0x05, 0, PPC2_ISA300)
>  GEN_VSX_HELPER_2(xscmpodp, 0x0C, 0x05, 0, PPC2_VSX)
>  GEN_VSX_HELPER_2(xscmpudp, 0x0C, 0x04, 0, PPC2_VSX)
> +GEN_VSX_HELPER_2(xscmpoqp, 0x04, 0x04, 0, PPC2_VSX)
> +GEN_VSX_HELPER_2(xscmpuqp, 0x04, 0x14, 0, PPC2_VSX)
>  GEN_VSX_HELPER_2(xsmaxdp, 0x00, 0x14, 0, PPC2_VSX)
>  GEN_VSX_HELPER_2(xsmindp, 0x00, 0x15, 0, PPC2_VSX)
>  GEN_VSX_HELPER_2(xscvdpsp, 0x12, 0x10, 0, PPC2_VSX)
> diff --git a/target-ppc/translate/vsx-ops.inc.c 
> b/target-ppc/translate/vsx-ops.inc.c
> index 2468ee9..7f09527 100644
> --- a/target-ppc/translate/vsx-ops.inc.c
> +++ b/target-ppc/translate/vsx-ops.inc.c
> @@ -126,6 +126,8 @@ GEN_XX3FORM(xscmpexpdp, 0x0C, 0x07, PPC2_ISA300),
>  GEN_VSX_XFORM_300(xscmpexpqp, 0x04, 0x05, 0x00600001),
>  GEN_XX2IFORM(xscmpodp,  0x0C, 0x05, PPC2_VSX),
>  GEN_XX2IFORM(xscmpudp,  0x0C, 0x04, PPC2_VSX),
> +GEN_VSX_XFORM_300(xscmpoqp, 0x04, 0x04, 0x00600001),
> +GEN_VSX_XFORM_300(xscmpuqp, 0x04, 0x14, 0x00600001),
>  GEN_XX3FORM(xsmaxdp, 0x00, 0x14, PPC2_VSX),
>  GEN_XX3FORM(xsmindp, 0x00, 0x15, PPC2_VSX),
>  GEN_XX2FORM(xscvdpsp, 0x12, 0x10, PPC2_VSX),

-- 
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: signature.asc
Description: PGP signature

Reply via email to