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]

Re: C6X new PCR relocs


On Thu, 19 May 2011, Bernd Schmidt wrote:

> 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.

Thanks.  These modified tests are OK with the
\ No newline at end of file
fixed by adding a trailing newline to that file.

-- 
Joseph S. Myers
joseph@codesourcery.com


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