[PATCH] D28564: [ELF] - Explicitly list supported relocations for x64 target.

[George Rimar via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL291751: [ELF] - Explicitly list supported relocations for x64 target. (authored by grimar).

Changed prior to commit:
  https://reviews.llvm.org/D28564?vs=83973&id=84089#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D28564

Files:
  lld/trunk/ELF/Target.cpp
  lld/trunk/test/ELF/invalid/invalid-relocation-x64.s


Index: lld/trunk/test/ELF/invalid/invalid-relocation-x64.s
===================================================================
--- lld/trunk/test/ELF/invalid/invalid-relocation-x64.s
+++ lld/trunk/test/ELF/invalid/invalid-relocation-x64.s
@@ -26,5 +26,5 @@
         Type:            R_X86_64_NONE
 
 # RUN: not ld.lld %p/Inputs/invalid-relocation-x64.elf -o %t2 2>&1 | FileCheck %s
-# CHECK: unrecognized reloc 152
-# CHECK: unrecognized reloc 153
+# CHECK: do not know how to handle relocation 'Unknown' (152)
+# CHECK: do not know how to handle relocation 'Unknown' (153)
Index: lld/trunk/ELF/Target.cpp
===================================================================
--- lld/trunk/ELF/Target.cpp
+++ lld/trunk/ELF/Target.cpp
@@ -631,7 +631,11 @@
 RelExpr X86_64TargetInfo<ELFT>::getRelExpr(uint32_t Type,
                                            const SymbolBody &S) const {
   switch (Type) {
-  default:
+  case R_X86_64_32:
+  case R_X86_64_32S:
+  case R_X86_64_64:
+  case R_X86_64_DTPOFF32:
+  case R_X86_64_DTPOFF64:
     return R_ABS;
   case R_X86_64_TPOFF32:
     return R_TLS;
@@ -657,6 +661,10 @@
     return R_GOT_PC;
   case R_X86_64_NONE:
     return R_HINT;
+  default:
+    error("do not know how to handle relocation '" + toString(Type) + "' (" +
+          Twine(Type) + ")");
+    return R_HINT;
   }
 }
 
@@ -878,7 +886,7 @@
     write64le(Loc, Val);
     break;
   default:
-    error(getErrorLocation(Loc) + "unrecognized reloc " + Twine(Type));
+    llvm_unreachable("unexpected relocation");
   }
 }
 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D28564.84089.patch
Type: text/x-patch
Size: 1537 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170112/234c5bbc/attachment.bin>