On 05/25/18 18:48, Michael S. Tsirkin wrote: > Sort alphabetically. Will help us see if anything is missing (e.g. tile > is not there now). > > Signed-off-by: Michael S. Tsirkin <m...@redhat.com> > --- > arch_init.c | 16 ++++++++-------- > 1 file changed, 8 insertions(+), 8 deletions(-) > > diff --git a/arch_init.c b/arch_init.c > index 9597218..f4f3f61 100644 > --- a/arch_init.c > +++ b/arch_init.c > @@ -52,14 +52,14 @@ int graphic_depth = 32; > #define QEMU_ARCH QEMU_ARCH_ARM > #elif defined(TARGET_CRIS) > #define QEMU_ARCH QEMU_ARCH_CRIS > -#elif defined(TARGET_I386) > -#define QEMU_ARCH QEMU_ARCH_I386 > #elif defined(TARGET_HPPA) > #define QEMU_ARCH QEMU_ARCH_HPPA > -#elif defined(TARGET_M68K) > -#define QEMU_ARCH QEMU_ARCH_M68K > +#elif defined(TARGET_I386) > +#define QEMU_ARCH QEMU_ARCH_I386 > #elif defined(TARGET_LM32) > #define QEMU_ARCH QEMU_ARCH_LM32 > +#elif defined(TARGET_M68K) > +#define QEMU_ARCH QEMU_ARCH_M68K > #elif defined(TARGET_MICROBLAZE) > #define QEMU_ARCH QEMU_ARCH_MICROBLAZE > #elif defined(TARGET_MIPS) > @@ -80,12 +80,12 @@ int graphic_depth = 32; > #define QEMU_ARCH QEMU_ARCH_SH4 > #elif defined(TARGET_SPARC) > #define QEMU_ARCH QEMU_ARCH_SPARC > -#elif defined(TARGET_XTENSA) > -#define QEMU_ARCH QEMU_ARCH_XTENSA > -#elif defined(TARGET_UNICORE32) > -#define QEMU_ARCH QEMU_ARCH_UNICORE32 > #elif defined(TARGET_TRICORE) > #define QEMU_ARCH QEMU_ARCH_TRICORE > +#elif defined(TARGET_UNICORE32) > +#define QEMU_ARCH QEMU_ARCH_UNICORE32 > +#elif defined(TARGET_XTENSA) > +#define QEMU_ARCH QEMU_ARCH_XTENSA > #endif > > const uint32_t arch_type = QEMU_ARCH; >
I sorted the list from scratch, using some regexes to temporarily move the #define's to the same lines as the #[el]if's. My results were identical to those of this patch. Reviewed-by: Laszlo Ersek <ler...@redhat.com> Thanks Laszlo