[PATCH] D54868: [PPC][PPC64] PPC_REL14 and PPC64_REL14 relocations

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 3 07:00:39 PST 2018


sfertile accepted this revision.
sfertile added a comment.

Other then my one comment it LGTM.



================
Comment at: ELF/Arch/PPC64.cpp:743
+    return isInt<26>(Offset);
+  error("Unexpected relocation type used in branch: " + Twine(Type));
+  return false;
----------------
inBranchRange is either called from the end of `PPC64::needsThunk` in which case the type must be either REL14 or REL24, or from the thunk creation framework which means that `PPC64::needsThunk` must have been true (and the type must have been REL14/REL24) . Because of that I think an llvm_unreachable is more appropriate than an error message.


Repository:
  rLLD LLVM Linker

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54868/new/

https://reviews.llvm.org/D54868





More information about the llvm-commits mailing list