[PATCH] h8300 target breakpoint doesn't work on Simulator

Joel Brobecker brobecker@adacore.com
Sat Mar 5 03:27:00 GMT 2011


> > -  /*static unsigned char breakpoint[] = { 0x7A, 0xFF }; *//* ??? */
> > -  static unsigned char breakpoint[] = { 0x01, 0x80 };	/* Sleep */
> > +  static unsigned char breakpoint[] = { 0x7A, 0xFF };	/* bpt (only
> > simulator) */
> 
> this sounds like you're fixing one system (sim) at the cost of
> breaking others (everyone else).  this func is used by all h8300
> targets ... not just sim.  so are you sure this is what you want ?
> 
> try looking at the Blackfin tdep to see how we handle sim-specific bp's.

I was going to say exactly the very same thing.

-- 
Joel



More information about the Gdb-patches mailing list