diff options
author | Francis Visoiu Mistrih <francisvm@yahoo.com> | 2019-08-22 16:59:00 +0000 |
---|---|---|
committer | Francis Visoiu Mistrih <francisvm@yahoo.com> | 2019-08-22 16:59:00 +0000 |
commit | 5b5ee61b5fa95327ed5a57b16f22e9e52a459129 (patch) | |
tree | f628ebcf3d8a8ea317d29ca65f77e67f1beeced0 /clang/lib/Basic/SourceManager.cpp | |
parent | 9a0f124f26437504c50fdb78401b73d820ed3991 (diff) | |
download | llvm-5b5ee61b5fa95327ed5a57b16f22e9e52a459129.zip llvm-5b5ee61b5fa95327ed5a57b16f22e9e52a459129.tar.gz llvm-5b5ee61b5fa95327ed5a57b16f22e9e52a459129.tar.bz2 |
[MachO][TLOF] Use hasLocalLinkage to determine if indirect symbol is local
Local symbols in the indirect symbol table contain the value
`INDIRECT_SYMBOL_LOCAL` and the corresponding __pointers entry must
contain the address of the target.
In r349060, I added support for local symbols in the indirect symbol
table, which was checking if the symbol `isDefined` && `!isExternal` to
determine if the symbol is local or not.
It turns out that `isDefined` will return false if the user of the
symbol comes before its definition, and we'll again generate .long 0
which will be the symbol at the adress 0x0.
Instead of doing that, use GlobalValue::hasLocalLinkage() to check if
the symbol is local.
Differential Revision: https://reviews.llvm.org/D66563
llvm-svn: 369671
Diffstat (limited to 'clang/lib/Basic/SourceManager.cpp')
0 files changed, 0 insertions, 0 deletions