Re: [Qemu-devel] [PATCH v3 15/35] include/exec: Split target_long def to new header

[Paolo Bonzini <pbonzini@redhat.com>]

On 18/07/2015 14:16, Paolo Bonzini wrote:
>> > +/* target_ulong is the type of a virtual address */
>> > +#if TARGET_LONG_SIZE == 4
>> > +#define target_long int32_t
>> > +#define target_ulong uint32_t
>> > +#define TARGET_FMT_lx "%08x"
>> > +#define TARGET_FMT_ld "%d"
>> > +#define TARGET_FMT_lu "%u"
>> > +#elif TARGET_LONG_SIZE == 8
>> > +#define target_long int64_t
>> > +#define target_ulong uint64_t
>> > +#define TARGET_FMT_lx "%016" PRIx64
>> > +#define TARGET_FMT_ld "%" PRId64
>> > +#define TARGET_FMT_lu "%" PRIu64
>> > +#else
>> > +#error TARGET_LONG_SIZE undefined
>> > +#endif
> Would it be possible, or make sense, to do
> 
> #define target_long arm_target_long
> #define target_ulong arm_target_ulong
> 
> instead?  This makes prototypes nicer when printed in the debugger with
> ptype.  Where could this be done?

Hmm, ISTR that this was more or less what v2 was like, so I guess you
made the change for a reason. :)  So ignore this.

Paolo