[PATCH] D56782: [PPC64] Preserve LocalEntry when linking

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 11 17:40:41 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;
----------------
Perhaps add a comment referencing Arch/PPC64.cpp `getPPC64GlobalEntryToLocalEntryOffset`

The 3 most significant bits of st_other are used by OpenPOWER ABI.


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