Re: [PATCH] MIPS bfd: abi 64 ld -r

[Thiemo Seufer <ica2_ts at csv dot ica dot uni-stuttgart dot de>]

Alexandre Oliva wrote:
> On Sep  3, 2002, Thiemo Seufer <ica2_ts@csv.ica.uni-stuttgart.de> wrote:
> 
> > It's wrong to say all RELA relocations will use partial_inplace
> > (especially if none of them do, AFAIK).
> 
> So, did you test it on IRIX?  I did, and I found that not only did the
> IRIX assembler place the addends in place, but also the linker not
> surprisingly picked them up from there, unlike our linker, that
> insisted on ignoring the in-place addend.  Unfortunately, I can't find
> the exact testcase I was using at that time, but I wouldn't be
> surprised if it was the same `main(){}' testcase that has exposed
> other problems fixed in this patch Stan posted.

I did test some slightly more complex testcase on IRIX, and I haven't
seen such a case.

> > If you are talking about R_MIPS_HI16 and R_MIPS_LO16, the IRIX tools
> > actually use REL relocations for them. That's why the backend has
> > also REL support.
> 
> I don't think those are the ones I'm talking about.  The very fact
> that I had to tweak one of the RELA relocations to fix the problem
> speaks for itself.  Whether I went too far in my generalization is
> fair to question, but not the fact that, for at least one of the RELA
> relocations, the IRIX tools use in-place addends.

I would like to see the exact case so I can try to reproduce it here.


Thiemo