[PATCH] D26604: [ELF] - Added support for extern "c++" local symbols in version script.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 16 09:31:13 PST 2016


grimar added inline comments.


================
Comment at: ELF/SymbolTable.cpp:721
+    for (SymbolVersion Sym : V.Locals)
+      assignVersion(Sym, VER_NDX_LOCAL, "'local'");
   }
----------------
ruiu wrote:
> Is there any reason to add single quotes? Please remove them because `V.Name` doesn't contain such quotes.
Lets add them for V.Name ? That is consistent with other messages we added recently


https://reviews.llvm.org/D26604





More information about the llvm-commits mailing list