[PATCH] D91426: [PowerPC] Fix issue where binary uses a .got but is missing a .TOC.
Stefan Pintilie via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Feb 1 12:31:25 PST 2021
stefanp added inline comments.
================
Comment at: lld/test/ELF/ppc64-ld-got-dtprel.s:28
+## lo(i at got@dtprel) = &.got[1] - (.got+0x8000) & 0xffff = -32760
+## hi(j at got@dtprel) = (&.got[2] - (.got+0x8000) >> 16) & 0xffff = -1
+## j at got@dtprel = &.got[2] - (.got+0x8000) = -32752
----------------
NeHuang wrote:
> is the computation proper here? To me, &.got[2] is different from &.got[1]
You are correct the comment there is wrong.
It should probably be:
```
## hi(j at got@dtprel) = (&.got[2] - .got >> 16) & 0xffff = -1
```
I will fix that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D91426/new/
https://reviews.llvm.org/D91426
More information about the llvm-commits
mailing list