On Mon, Mar 10, 2014 at 01:15:28AM +0100, Andreas Färber wrote: > Most targets were using offsetof(CPUFooState, breakpoints) to determine > how much of CPUFooState to clear on reset. Use the next field after > CPU_COMMON instead, if any, or sizeof(CPUFooState) otherwise.
Would it not be easier and more readable to add a field replacing 'breakpoints' just for this purpose, at least for the time being? I'm guessing CPU_COMMON_TLB will be getting cleaned up too at some point so is it really worth avoiding the tiny amount of bloat this imposes? -- Cheers, Stuart