[PATCH] D94907: [llvm-nm][ELF] - Make -D display symbol versions.

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 18 04:31:31 PST 2021


grimar added inline comments.


================
Comment at: llvm/test/tools/llvm-nm/dynamic.test:75
+# VERSIONED-SYMS-NEXT: U version4sym at v4
+# VERSIONED-SYMS-NEXT: U version5sym at v5hidden
+
----------------
Note: I think we should print `@@` for default versions and `@` for hidden versions.
It is not what GNU nm does, but it would be more consistent with other tools that print versioned symbols.
I am going to post a follow-up patch for this.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94907/new/

https://reviews.llvm.org/D94907



More information about the llvm-commits mailing list