Note that a gold bug has been fixed.
authorRafael Espindola <rafael.espindola@gmail.com>
Mon, 6 Oct 2014 12:33:27 +0000 (12:33 +0000)
committerRafael Espindola <rafael.espindola@gmail.com>
Mon, 6 Oct 2014 12:33:27 +0000 (12:33 +0000)
We should be able to stop working around it at some point in the future.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@219115 91177308-0d34-0410-b5e6-96231b3b80d8

lib/MC/ELFObjectWriter.cpp

index efc9cbdd89adcb9c9967680236fa284333ea554d..371e145bddc7bcba54b22afb6836619768de1346 100644 (file)
@@ -770,8 +770,9 @@ bool ELFObjectWriter::shouldRelocateWithSymbol(const MCAssembler &Asm,
   }
 
   // Most TLS relocations use a got, so they need the symbol. Even those that
-  // are just an offset (@tpoff), require a symbol in some linkers (gold,
-  // but not bfd ld).
+  // are just an offset (@tpoff), require a symbol in gold versions before
+  // 5efeedf61e4fe720fd3e9a08e6c91c10abb66d42 (2014-09-26) which fixed
+  // http://sourceware.org/PR16773.
   if (Flags & ELF::SHF_TLS)
     return true;