* [PATCH-SH]:Generating proper relocations to link with Renesas SHC linker
@ 2004-03-23 18:20 Asgari J. Jinia
2004-03-23 18:24 ` Kaz Kojima
0 siblings, 1 reply; 2+ messages in thread
From: Asgari J. Jinia @ 2004-03-23 18:20 UTC (permalink / raw)
To: binutils
Hi,
If GNU object file for SH target which contains static functions is linked
with other Renesas SHC objects by SHC linker, it does not call GNU
functions properly. It always calls the first static function when other
static functions are called. This is because GNU assembler does not emit
relocations with addend values in case of static functions.
Following patch emits addend value in relocations for static declarations so
that it will correctly link with Renesas linker.
Regards,
Asgari Jinia
KPIT Cummins
Testcase:
Following is simple test code.
Command Line: sh-elf-gcc test.c -c -mhitachi -o test.o
/* test.c */
static int foo( int a, int b, int c)
{
return a+b;
}
static int bar( int a, int b, int c)
{
return a+b;
}
int dummymain()
{
foo(1,2,3);
bar(1,2,3);
}
If test.o is linked with Renesas SHC linker and called "dummymain", "foo" is
called instead of "bar".
Patch:
gas/ChangeLog:
2004-03-23 Asgari Jinia <asgarij@kpitcummins.com>
* config/tc-sh.c (tc_gen_reloc): addend updated with fixp->fx_offset for
BFD_RELOC_32 and BFD_RELOC_32_GOTOFF type relocations.
==========================================================
--- gas/config/tc-sh.c.old Wed Dec 31 17:05:28 2003
+++ gas/config/tc-sh.c Tue Mar 23 17:26:03 2004
@@ -3957,7 +3957,7 @@ tc_gen_reloc (section, fixp)
else if (fixp->fx_pcrel)
rel->addend = fixp->fx_addnumber;
else if (r_type == BFD_RELOC_32 || r_type == BFD_RELOC_32_GOTOFF)
- rel->addend = fixp->fx_addnumber;
+ rel->addend = fixp->fx_offset;
else
rel->addend = 0;
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH-SH]:Generating proper relocations to link with Renesas SHC linker
2004-03-23 18:20 [PATCH-SH]:Generating proper relocations to link with Renesas SHC linker Asgari J. Jinia
@ 2004-03-23 18:24 ` Kaz Kojima
0 siblings, 0 replies; 2+ messages in thread
From: Kaz Kojima @ 2004-03-23 18:24 UTC (permalink / raw)
To: binutils
"Asgari J. Jinia" <AsgariJ@KPITCummins.com> wrote:
> If GNU object file for SH target which contains static functions is linked
> with other Renesas SHC objects by SHC linker, it does not call GNU
> functions properly. It always calls the first static function when other
> static functions are called. This is because GNU assembler does not emit
> relocations with addend values in case of static functions.
>
> Following patch emits addend value in relocations for static declarations so
> that it will correctly link with Renesas linker.
[snip]
> --- gas/config/tc-sh.c.old Wed Dec 31 17:05:28 2003
> +++ gas/config/tc-sh.c Tue Mar 23 17:26:03 2004
> @@ -3957,7 +3957,7 @@ tc_gen_reloc (section, fixp)
> else if (fixp->fx_pcrel)
> rel->addend = fixp->fx_addnumber;
> else if (r_type == BFD_RELOC_32 || r_type == BFD_RELOC_32_GOTOFF)
> - rel->addend = fixp->fx_addnumber;
> + rel->addend = fixp->fx_offset;
> else
> rel->addend = 0;
With your patch, I've got many failures of ld testsuite for
sh-unknown-linux-gnu and one failure about partial linking
for sh-elf on mainline. It seems that gas with your patch
sets both in-place relocation and addend for R_SH_DIR32.
It would be a radical change of ABI and anyway, your patch
doesn't work together with the current GNU linker.
Regards,
kaz
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2004-03-23 15:28 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2004-03-23 18:20 [PATCH-SH]:Generating proper relocations to link with Renesas SHC linker Asgari J. Jinia
2004-03-23 18:24 ` Kaz Kojima
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).