[PATCH] D42486: [ELF][MIPS] Ignore incorrect version definition index for _gp_disp symbol
George Rimar via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Jan 24 23:02:45 PST 2018
grimar added inline comments.
================
Comment at: ELF/InputFiles.cpp:860
+ if (Config->EMachine == EM_MIPS) {
+ // MIPS BFD linker puts _gp_disp symbol into DSO files and incorrectly
+ // assigns VER_NDX_LOCAL to this section global symbol.
----------------
I suggest to add "FIXME: " at the begining of comment,
saying that this hack can be removed once fixed in BFD and settle for a while.
It's consistent with other hacks/workarounds we have in code.
Repository:
rLLD LLVM Linker
https://reviews.llvm.org/D42486
More information about the llvm-commits
mailing list