[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] Re: [Xen-devel] [PATCH 2/2] x86emul: consolidate string insn address increments
On 09/12/16 15:22, Jan Beulich wrote: > Move the looking at EFLAGS.DF into the macro, rendering all call sites > more readable. > > Signed-off-by: Jan Beulich <jbeulich@xxxxxxxx> The net change is ok; it is certainly cleaner to read in the body of x86_emulate(). However, the naming of register_address_increment() was previously ok, as it was obvious at the calling point that a negative increment was possible. This subtly is now hidden. How about reg_addr_adjust() or reg_addr_adjust_dir() as an alternative name? This retains the property that it is obvious that the direction flag is followed in the calculation. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@xxxxxxxxxxxxx https://lists.xen.org/xen-devel
|
![]() |
Lists.xenproject.org is hosted with RackSpace, monitoring our |