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


Hello Mike,

Thank you very much for reviewing the patch.

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

I tried adding one.
I took an existing test case sim/testsuite/sim/arm/hello.ms and used the 
instructions 'movw' and 'movt' to load the address of the string to be printed.

Test case seems to be working fine.
Please find attached an updated patch with the test case added.

Is this an acceptable test case?

The ChangeLog updates would be:

sim/arm/ChangeLog:
=======================
2013-05-07  Jayant Sonar  <jayant.sonar@kpitcummins.com>
        Kaushik Phatak <Kaushik.Phatak@kpitcummins.com>

        * armemu.c (ARMul_Emulate32): Emulate instructions MOVW and MOVT.

sim/testsuite/sim/arm/ChangeLog:
==============================
2013-05-07  Jayant Sonar  <jayant.sonar@kpitcummins.com>
        Kaushik Phatak <Kaushik.Phatak@kpitcummins.com>

        * movw-movt.ms: New file: Test movw & movt instructions.

Regards,
Jayant

Attachment: movw-movt-insns.txt
Description: movw-movt-insns.txt


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