On 09/11/2018 01:28 PM, Emilio G. Cota wrote:
> Signed-off-by: Emilio G. Cota <c...@braap.org>
> ---
>  target/i386/translate.c | 282 ++++++++++++++++++++--------------------
>  1 file changed, 144 insertions(+), 138 deletions(-)

Reviewed-by: Richard Henderson <richard.hender...@linaro.org>

I will note that these tmpN variables ought to be eliminated
completely, in favor of totally local temporary allocation.

But that should be done separately, because while they *ought*
to be local, in some cases it may be hard to see that they are.


r~

Reply via email to