[PATCH] D22433: Handle versioned symbols efficiently.

Rui Ueyama via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 17 10:31:24 PDT 2016


ruiu added a comment.

Submitted as r275711. Can you run your benchmark again so that we find what was wrong?


================
Comment at: ELF/SymbolTable.cpp:626
@@ +625,3 @@
+    if (V.Name == Verstr)
+      return {Name, IsDefault ? V.Id : (V.Id | VERSYM_HIDDEN)};
+
----------------
grimar wrote:
> This required the next change for me to compile using msvs2015:
> 
> ```
>   for (elf::VersionDefinition &V : Config->VersionDefinitions) {
>     if (V.Name == Verstr)
>       return{ Name, IsDefault ? V.Id : (V.Id | VERSYM_HIDDEN) };
>   }
> ```
Fixed.


Repository:
  rL LLVM

https://reviews.llvm.org/D22433





More information about the llvm-commits mailing list