On 10/01/2013 04:00 AM, Richard Henderson wrote: > On 09/30/2013 08:57 AM, Fabien Chouteau wrote: >> +extern const MonitorDef arch_monitor_defs[]; > > This is supplied by target-foo/monitor.c, right? > Why in the world is it declared in generic code? >
Yes, why? > Especially if it's only ever accessed via the > cpu->monitor_defs member? > To begin with, I though I'd put in in each target-*/cpu.c, then having it at only one place seemed more clean. I'm open to any suggestion. -- Fabien Chouteau
