https://sourceware.org/bugzilla/show_bug.cgi?id=28098
Alan Modra <amodra at gmail dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
Ever confirmed|0 |1
Status|UNCONFIRMED |ASSIGNED
Assignee|ccoutant at gmail dot com |amodra at gmail dot com
Last reconfirmed| |2021-07-18
--- Comment #2 from Alan Modra <amodra at gmail dot com> ---
Thanks for the analysis and patch. I'm going to ask you to do just a little
more work. First, change the patch to skip r_info==0 relocs, which is R_*_NONE
(usually, there are some targets that stupidly define r_type of 0 to be
something other than a no-operation relocations) *and* r_sym of zero. That
will match ld.bfd. Also, put a suitable comment in reloc.h, something like
"..would be skipped, excluding r_info==0 relocs." and perhaps expand on this a
little in the function comment in reloc.cc, mentioning R_*_NONE and r_sym of
zero. Finally, check over all uses of advance() in gold to see that it isn't
used to count relocs for an output array or something like that, which might be
broken by your change. If you get to that point, post the patch to
binut...@sourceware.org and I'll commit it.
--
You are receiving this mail because:
You are on the CC list for the bug.
