[PATCH] D56782: [PPC64] Preserve LocalEntry when linking
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 11 17:49:21 PST 2019
MaskRay added inline comments.
================
Comment at: ELF/Target.h:182
+// GlobalEntryToLocalEntry offset.
+static inline uint8_t getPPC64GlobalEntryToLocalEntryBits(uint8_t StOther) {
+ return StOther & 0xE0;
----------------
MaskRay wrote:
> Perhaps add a comment referencing Arch/PPC64.cpp `getPPC64GlobalEntryToLocalEntryOffset`
>
> The 3 most significant bits of st_other are used by OpenPOWER ABI.
Sorry, the comment should be near line 2003 :)
Repository:
rLLD LLVM Linker
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D56782/new/
https://reviews.llvm.org/D56782
More information about the llvm-commits
mailing list