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

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 15 10:53:02 PST 2016


ruiu added inline comments.


================
Comment at: ELF/LinkerScript.cpp:1810
 
+void ScriptParser::readSymbols(std::vector<SymbolVersion> &V, bool Globals) {
+  for (;;) {
----------------
I don't think you need `Globals`. You could just stop reading until `local:` in any context.


https://reviews.llvm.org/D26604





More information about the llvm-commits mailing list