This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH/ARM] Adding support for MOVW and MOVT instructions


On Monday 29 April 2013 08:47:44 Jayant R. Sonar wrote:
> This patch adds simulator support for handling the armv7 instructions
> 'movw (immediate)' and 'movt'.
> 
> Compiler frequently use these instructions to load the 32bit addresses of
> global variables, string pointers etc. into the general registers.
> 
> In absence of support of these instructions:
> 1. GDB run simulator fails to print even simple "hello world" string
>    on console.
> 2. Loading of global variable addresses into the registers fail causing
>    arithmetic operation failures.
> 
> Patch has been regression tested for arm-none-eabi (-march=armv7-a).

thanks, looks fine to me (not that i'm an ARM ISA expert)

i don't suppose you could write a small test in sim/testsuite/sim/arm/ ?
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]