Re: [Qemu-devel] [PATCH] target-mips: apply workaround for TCG optimizations for MFC1

[Richard Henderson]

On 07/15/2015 09:06 AM, Aurelien Jarno wrote:
> On 2015-07-15 09:31, Paolo Bonzini wrote:
> > Ok, I see your point.  If you put it like this :) the fault definitely
> > lies in the backends.  What I'm proposing would be in a new
> > tcg_reg_alloc_trunc function, and it would require implementing a
> > non-noop trunc.
>
> Why not reusing the existing trunc_shr_i64_i32 op? AFAIU, it has been
> designed exactly for that.

Indeed.

> Actually I think we should implement the following ops as optional but
> *real* TCG ops:
> - trunc_shr_i64_i32
> - extu_i32_i64
> - ext_i32_i64

While we could perhaps gain something from the last two, reliably using the 
first is probably the most important.

I've been unable to reproduce the binary in question.  I'm curious if something 
as simple as this helps.  Alternately, the two hunks might just cancel each 
other out and result in no change.


r~

z
Description: Text document