Am 23.04.2013 10:16, schrieb liguang: > remove macros EAX, EBX, ECX, EDX, EBP, ESP, ESI, EDI, EIP, DF > as suggested by Richard Henderson <r...@twiddle.net> > > Li Guang (12) > target-i386/helper: remove EAX macro > target-i386/helper: remove EBX macro > target-i386/helper: remove ECX macro > target-i386/helper: remove EDX macro > target-i386/helper: remove EBP macro > target-i386/helper: remove ESP macro > target-i386/helper: remove ESI macro > target-i386/helper: remove EDI macro > target-i386/helper: remove EIP macro > target-i386/helper: remove DF macro > target-i386/helper: remove redundant env->eip assignment > target-i386: fix over 80 chars warnings
Hard Freeze is over, so more time to look into refactorings: There's one thing to be aware of here, macros would make it easier to transition from CPUX86State to X86CPU fields. However I am guessing that all these registers are accessed by TCG code via offsets from cpu_env - please verify that. If so, Reviewed-by: Andreas Färber <afaer...@suse.de> However, it would be nice if you could fix the \ alignment in patch 06/12 or in the cleanup patch 12/12. Regards, Andreas -- SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 Nürnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imendörffer; HRB 16746 AG Nürnberg