[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH 00/16] minially change gen_intermediate_code_int

From: Peter Maydell
Subject: Re: [Qemu-devel] [PATCH 00/16] minially change gen_intermediate_code_internal function
Date: Thu, 11 Apr 2013 09:20:24 +0100

On 11 April 2013 04:41, li guang <address@hidden> wrote:
> here are instructions in the finial qemu-system-x86
> before and after my chanage, you can easily find a
> redundant 'addl' was removed

Yes, but:
 * the removed instruction is not inside the loop
 * the entire loop is not in a speed critical path
 * removing an addl will save a cycle or two
 * the specific syntax used for this loop is far from
   the hardest barrier to understanding what the search_pc
   code is actually doing

Is it really worth the code churn?

It would be more interesting to find a better interface
between the common parts of QEMU and the target code
so that there's less repetition in these target functions
in the first place.

-- PMM

reply via email to

[Prev in Thread] Current Thread [Next in Thread]