On Thu, Feb 23, 2017 at 12:32:44PM +0530, Nikunj A Dadhania wrote:
> David Gibson <da...@gibson.dropbear.id.au> writes:
>
> > -static void gen_read_xer(TCGv dst)
> >> +static void gen_read_xer(DisasContext *ctx, TCGv dst)
> >> {
> >> TCGv t0 = tcg_temp_new();
> >> TCGv t1 = tcg_temp_new();
> >> @@ -3715,15 +3719,30 @@ static void gen_read_xer(TCGv dst)
> >> tcg_gen_or_tl(t0, t0, t1);
> >> tcg_gen_or_tl(dst, dst, t2);
> >> tcg_gen_or_tl(dst, dst, t0);
> >> + if (is_isa300(ctx)) {
> >> + tcg_gen_shli_tl(t0, cpu_ov32, XER_OV32);
> >> + tcg_gen_or_tl(dst, dst, t0);
> >> + tcg_gen_shli_tl(t0, cpu_ca32, XER_CA32);
> >> + tcg_gen_or_tl(dst, dst, t0);
> >
> > Could you use 2 deposits here, instead of 2 shifts and 2 ors?
>
> I checked the implementation of tcg_gen_deposit_i64, resultant will have much
> more than 2 shifts + 2 ors.
Ok, fair enough.
--
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
