summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPatrick Rudolph <siro@das-labor.org>2019-02-15 14:45:58 +0100
committerPatrick Rudolph <siro@das-labor.org>2019-03-03 16:46:30 +0000
commit62b4b4496190c69b3e0f8562a3c560913af119bf (patch)
tree4b791af06ecfd57de8a79e7ac0319e332f8b7a42
parent3a42c885100d38db2f95fa3bd3ae96ae8e501de6 (diff)
downloadcoreboot-62b4b4496190c69b3e0f8562a3c560913af119bf.tar.xz
rmodule: Don't emit reloc for R_X86_64_PC64
Relocations for PC relative instructions must not emitted. As PC64 are unlikely with current code, it never was an issue. Change-Id: Ife472a287ff15b1c04a516e25ff13221441fd122 Signed-off-by: Patrick Rudolph <siro@das-labor.org> Reviewed-on: https://review.coreboot.org/c/31469 Reviewed-by: Aaron Durbin <adurbin@chromium.org> Reviewed-by: Furquan Shaikh <furquan@google.com> Tested-by: build bot (Jenkins) <no-reply@coreboot.org>
-rw-r--r--util/cbfstool/rmodule.c1
1 files changed, 0 insertions, 1 deletions
diff --git a/util/cbfstool/rmodule.c b/util/cbfstool/rmodule.c
index f270e3ec8f..817bc604f0 100644
--- a/util/cbfstool/rmodule.c
+++ b/util/cbfstool/rmodule.c
@@ -66,7 +66,6 @@ static int should_emit_amd64(Elf64_Rela *rel)
/* Only emit absolute relocations */
return (type == R_AMD64_64 ||
- type == R_AMD64_PC64 ||
type == R_AMD64_32S ||
type == R_AMD64_32);
}