[PATCH] D21018: [ELF] - Basic versioned symbols support implemented.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 10 11:01:35 PDT 2016


ruiu added a comment.

Ahh, I got the reason why it didn't appear in my inbox. It went to a spam folder. I'm sorry about that. If you have sent me mails and did not get responses, please re-send.


================
Comment at: ELF/SymbolListFile.cpp:106-108
@@ +105,5 @@
+  while (!atEOF() && !Error) {
+    Config->SymbolVersions.push_back({next(), {}});
+    Version &V = Config->SymbolVersions.back();
+
+    expect("{");
----------------
Then how about defining a constructor that takes a StringRef?


http://reviews.llvm.org/D21018





More information about the llvm-commits mailing list