This is the mail archive of the binutils@sourceware.org mailing list for the binutils 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] |
On 05/18/2011 02:51 PM, Joseph S. Myers wrote: > On Wed, 18 May 2011, Bernd Schmidt wrote: > >>> Also, since the >>> relocations are L16 and H16 (with, correspondingly, no overflow checking >>> possible) are you sure you want to allow the BFD_RELOC_C6000_ABS_S16 case? >> >> TI's examples use mvk, and that requires allowing BFD_RELOC_C6000_ABS_S16. > > Thanks. I think there should however be testcases covering the > BFD_RELOC_C6000_ABS_L16 case - at present all the tests seem to use mvk > and mvkh, so only covering the other two cases. > > The code and tests in this patch version are OK as far as they go, but in > addition to the tests needed mentioned above I think there should also be > a linker test covering nonzero results of the H16 relocation (maybe cases > where the result is negative). I'm attaching a diff of the modified tests. In the gas test I've changed some insns to mvkl; the .d file is unchanged since mvk and mvkl are the same instruction. The linker test now has a negative result test. Bernd
Attachment:
newtcs.diff
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |