On 12/23/2010 01:00 AM, Yao Qi wrote: > OK. I have to try the second approach, which is 1) exposing displaced > stepping state to tdep, and 2) take displaced stepping state into > account when determining the mode. > After talked with Ulrich, I realize that it is *not* a good idea to expose displaced stepping outside of infrun, and my patch is a little bit too intrusive. > 2010-12-23 Yao Qi > > * arm-tdep.c: (arm_pc_is_thumb): Adjust MEMADDR if it is within > copy area of displaced stepping. > * infrun.c (struct displaced_step_inferior_state): Move to ... > Expose get_displaced_stepping_state. > * inferior.h: ... here. > Declare get_displaced_stepping_state. This time, instead of exposing displaced_step_inferior_state to tdep, we return displaced_step_closure, which is defined by each tdep, instance to tdep appropriately. OK to mainline? -- Yao Qi