[PATCH] D80311: [lld][test] Expand testing for dynamic-list and export-dynamic

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri May 22 13:57:36 PDT 2020


MaskRay added inline comments.


================
Comment at: lld/test/ELF/export-symbol-types.s:4
+
+## Not exported:
+## - local symbols
----------------
I think the test should be named `dynamic-list-*` instead, instead of `export-symbol`

--dynamic-list work for both executables and shared libraries.
For an executable, export some symbols.
For a shared library, specify preemptible symbols.

We should check both executable and so cases.


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

https://reviews.llvm.org/D80311





More information about the llvm-commits mailing list