C6X new PCR relocs

Bernd Schmidt bernds@codesourcery.com
Thu May 19 12:21:00 GMT 2011


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
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: newtcs.diff
URL: <https://sourceware.org/pipermail/binutils/attachments/20110519/535e3d35/attachment.ksh>


More information about the Binutils mailing list