On 5 January 2015 at 18:15, Peter Maydell <peter.mayd...@linaro.org> wrote: > On 18 December 2014 at 00:26, David Morrison <dmorri...@invlim.com> wrote: >> --- a/target-openrisc/cpu.h >> +++ b/target-openrisc/cpu.h >> @@ -20,6 +20,7 @@ >> #ifndef CPU_OPENRISC_H >> #define CPU_OPENRISC_H >> >> +#define TARGET_HAS_ICE >> #define TARGET_LONG_BITS 32 >> #define ELF_MACHINE EM_OPENRISC > > This looks like a correct change
...actually, on second thought, we should do this the other way round and just get rid of TARGET_HAS_ICE altogether. The only two targets which don't define it are openrisc and unicore32, and both of those actually do have the breakpoint handling code, so failing to define it was just a bug. I'll cook up a patch... -- PMM
