[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