[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