On 23 December 2013 12:50, Paolo Bonzini <pbonz...@redhat.com> wrote: > Il 23/12/2013 13:37, Peter Maydell ha scritto: >> At a minimum, if we take this approach we should add TODO comments >> to the effect that the NULL terminator and the if() can be removed >> when the first real AArch64 CPU is added. >> >> I think I'd rather put the if (!info->name) continue into the function >> which is doing the looping over the array. > > Or just change the termination condition from a check on the array size > to one on info->name.
That would take it out of line with the equivalent 32 bit ARM code (and also moxie and openrisc for what little that's worth) and be fractionally more tedious to revert later. thanks -- PMM
