[PATCH] D91533: [lib/Object] - Generalize the RelocationResolver API.
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 18 17:31:48 PST 2020
MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.
================
Comment at: lld/ELF/DWARF.cpp:89
template <class ELFT> struct LLDRelocationResolver<Elf_Rel_Impl<ELFT, false>> {
- // For Rel, the addend A is supplied by the caller.
- static uint64_t resolve(object::RelocationRef /*Ref*/, uint64_t s,
- uint64_t a) {
- return s + a;
+ // For Rel, the addend is stored in the object data and is supplied by the
+ // caller.
----------------
"stored in the field to be relocated" => "extracted from the relocated location"
http://www.sco.com/developers/gabi/latest/ch4.reloc.html has a similar wording.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91533/new/
https://reviews.llvm.org/D91533
More information about the llvm-commits
mailing list