Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(1173)

Issue 4128004: [binutils] Special processing of R_X86_64_32 relocations (Closed)

Created:
10 years, 1 month ago by eaeltsin
Modified:
9 years, 7 months ago
CC:
native-client-reviews_googlegroups.com
Visibility:
Public.

Description

[binutils] Special processing of R_X86_64_32 relocations R_X86_64_32 relocations in NaCl64 play the same role as R_X86_64_64 in native x86_64. They should be rewritten in case relocated symbol is hidden and will not exist in statically linked binary. R_X86_64_64 are rewritten into R_X86_64_RELATIVE using the absolute value of the symbol. Unfortunately we don't have 32-bit alternative, thus we change relocation to use section instead of symbol. Usually this rewrite produces buggy relocations (why? there is corresponding comment in bfd, but it does not explain much), but in our special case we just create valid relocations. BUG=http://code.google.com/p/nativeclient/issues/detail?id=1095 TEST=

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+10 lines, -2 lines) Patch
M binutils/bfd/elf64-x86-64.c View 1 chunk +10 lines, -2 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
eaeltsin
10 years, 1 month ago (2010-10-26 10:39:23 UTC) #1
Mark Seaborn
On 2010/10/26 10:39:23, Evgeny Eltsin wrote: > BUG= > TEST= There's no point in putting ...
10 years, 1 month ago (2010-10-26 11:13:30 UTC) #2
eaeltsin
BUG=http://code.google.com/p/nativeclient/issues/detail?id=1095 TEST= and BUG= are from git-cl. Next time I'll simply erase these lines. I ...
10 years, 1 month ago (2010-10-26 11:55:25 UTC) #3
pasko-google - do not use
10 years, 1 month ago (2010-10-26 15:14:28 UTC) #4
LGTM

Powered by Google App Engine
This is Rietveld 408576698