[PATCH] D13550: [ELF2] Check for TLS mismatch in symbol resolution.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 8 14:48:55 PDT 2015


ruiu accepted this revision.
ruiu added a comment.
This revision is now accepted and ready to land.

LGTM


================
Comment at: ELF/SymbolTable.cpp:178-179
@@ -177,3 +177,4 @@
 template <class ELFT>
-void SymbolTable::reportConflict(const SymbolBody &Old, const SymbolBody &New) {
+void SymbolTable::reportConflict(const SymbolBody &Old, const SymbolBody &New,
+                                 const Twine &Message, bool Warning) {
   typedef typename ELFFile<ELFT>::Elf_Sym Elf_Sym;
----------------
Move Message to the beginning of the parameter list.


http://reviews.llvm.org/D13550





More information about the llvm-commits mailing list