On Wed, Nov 23, 2016 at 02:21:45PM -0200, Jose Ricardo Ziviani wrote: > bcdsetsgn.: Decimal set sign. This instruction copies the register > value to the result register but adjust the signal according to > the preferred sign value. > > Signed-off-by: Jose Ricardo Ziviani <jos...@linux.vnet.ibm.com>
Reviewed-by: David Gibson <da...@gibson.dropbear.id.au> > --- > target-ppc/helper.h | 1 + > target-ppc/int_helper.c | 19 +++++++++++++++++++ > target-ppc/translate/vmx-impl.inc.c | 8 ++++++++ > 3 files changed, 28 insertions(+) > > diff --git a/target-ppc/helper.h b/target-ppc/helper.h > index dada48e..cddac8e 100644 > --- a/target-ppc/helper.h > +++ b/target-ppc/helper.h > @@ -385,6 +385,7 @@ DEF_HELPER_3(bcdctz, i32, avr, avr, i32) > DEF_HELPER_3(bcdcfsq, i32, avr, avr, i32) > DEF_HELPER_3(bcdctsq, i32, avr, avr, i32) > DEF_HELPER_4(bcdcpsgn, i32, avr, avr, avr, i32) > +DEF_HELPER_3(bcdsetsgn, i32, avr, avr, i32) > > DEF_HELPER_2(xsadddp, void, env, i32) > DEF_HELPER_2(xssubdp, void, env, i32) > diff --git a/target-ppc/int_helper.c b/target-ppc/int_helper.c > index 833c9d2..db430ef 100644 > --- a/target-ppc/int_helper.c > +++ b/target-ppc/int_helper.c > @@ -2982,6 +2982,25 @@ uint32_t helper_bcdcpsgn(ppc_avr_t *r, ppc_avr_t *a, > ppc_avr_t *b, uint32_t ps) > return bcd_cmp_zero(r); > } > > +uint32_t helper_bcdsetsgn(ppc_avr_t *r, ppc_avr_t *b, uint32_t ps) > +{ > + int i; > + int invalid = 0; > + int sgnb = bcd_get_sgn(b); > + > + *r = *b; > + bcd_put_digit(r, bcd_preferred_sgn(sgnb, ps), 0); > + > + for (i = 1; i < 32; i++) { > + bcd_get_digit(b, i, &invalid); > + if (unlikely(invalid)) { > + return 1 << CRF_SO; > + } > + } > + > + return bcd_cmp_zero(r); > +} > + > void helper_vsbox(ppc_avr_t *r, ppc_avr_t *a) > { > int i; > diff --git a/target-ppc/translate/vmx-impl.inc.c > b/target-ppc/translate/vmx-impl.inc.c > index c14b666..b188e60 100644 > --- a/target-ppc/translate/vmx-impl.inc.c > +++ b/target-ppc/translate/vmx-impl.inc.c > @@ -991,6 +991,7 @@ GEN_BCD2(bcdcfz) > GEN_BCD2(bcdctz) > GEN_BCD2(bcdcfsq) > GEN_BCD2(bcdctsq) > +GEN_BCD2(bcdsetsgn) > GEN_BCD(bcdcpsgn); > > static void gen_xpnd04_1(DisasContext *ctx) > @@ -1014,6 +1015,9 @@ static void gen_xpnd04_1(DisasContext *ctx) > case 7: > gen_bcdcfn(ctx); > break; > + case 31: > + gen_bcdsetsgn(ctx); > + break; > default: > gen_invalid(ctx); > break; > @@ -1038,12 +1042,16 @@ static void gen_xpnd04_2(DisasContext *ctx) > case 7: > gen_bcdcfn(ctx); > break; > + case 31: > + gen_bcdsetsgn(ctx); > + break; > default: > gen_invalid(ctx); > break; > } > } > > + > GEN_VXFORM_DUAL(vsubcuw, PPC_ALTIVEC, PPC_NONE, \ > xpnd04_1, PPC_NONE, PPC2_ISA300) > GEN_VXFORM_DUAL(vsubsws, PPC_ALTIVEC, PPC_NONE, \ -- 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
signature.asc
Description: PGP signature