diff options
| author | Alex Dzyoba <alex@dzyoba.com> | 2016-08-04 13:42:21 +0200 |
|---|---|---|
| committer | Ben Gamari <ben@smart-cactus.org> | 2016-08-05 10:40:45 +0200 |
| commit | e3e2e49a8f6952e1c8a19321c729c17b294d8c92 (patch) | |
| tree | 154d6713a76098b6553516d26c8297213ad1133d /includes | |
| parent | f09d65474ed042360999cb88221d65b07bfb4b5f (diff) | |
| download | haskell-e3e2e49a8f6952e1c8a19321c729c17b294d8c92.tar.gz | |
codeGen: Remove binutils<2.17 hack, fixes T11758
There was a complication on the x86_64 platform, where pointers were 64
bits, but the tools didn't support 64-bit relative relocations. This
was true before binutils 2.17, which nowadays is quite standart (even
CentOs 5 is shipped with 2.17).
Hacks were removed from x86 genSwitch and asm pretty printer. Also
[x86-64-relative] note was dropped from
includes/rts/storage/InfoTables.h as it's not referenced anywhere now.
Reviewers: austin, simonmar, rwbarton, erikd, bgamari
Reviewed By: simonmar, erikd, bgamari
Subscribers: thomie
Differential Revision: https://phabricator.haskell.org/D2426
Diffstat (limited to 'includes')
| -rw-r--r-- | includes/rts/storage/InfoTables.h | 11 |
1 files changed, 0 insertions, 11 deletions
diff --git a/includes/rts/storage/InfoTables.h b/includes/rts/storage/InfoTables.h index fb14ac5821..e6bd15c887 100644 --- a/includes/rts/storage/InfoTables.h +++ b/includes/rts/storage/InfoTables.h @@ -15,17 +15,6 @@ Several pointer fields in info tables are expressed as offsets relative to the info pointer, so that we can generate position-independent code. - - Note [x86-64-relative] - There is a complication on the x86_64 platform, where pointers are - 64 bits, but the tools don't support 64-bit relative relocations. - However, the default memory model (small) ensures that all symbols - have values in the lower 2Gb of the address space, so offsets all - fit in 32 bits. Hence we can use 32-bit offset fields. - - Somewhere between binutils-2.16.1 and binutils-2.16.91.0.6, - support for 64-bit PC-relative relocations was added, so maybe this - hackery can go away sometime. ------------------------------------------------------------------------- */ #if x86_64_TARGET_ARCH |