On 10/04/2013 07:52 PM, Peter Maydell wrote: >> +#include "monitor/monitor_def.h" >> + >> +const MonitorDef arm_monitor_defs[] = { >> + { "r0", offsetof(CPUARMState, regs[0]) }, >> + { "r1", offsetof(CPUARMState, regs[1]) }, > > These fields are all 32 bits, not target_long, > so they need to be marked as MD_I32. (If you build an > aarch64-softmmu target then it will have target_long be > 64 bit but still support all the 32 bit CPUs, so it does > make a difference.) >
OK, I'll mark them MD_I32. -- Fabien Chouteau