[libcxx-commits] [PATCH] D121164: [libcxx] [ci] Check that Windows static libraries don't contain dllexports

[Louis Dionne via Phabricator via libcxx-commits]

ldionne added a comment.

I really like this direction!



================
Comment at: libcxx/test/libcxx/vendor/clangcl/static-lib-exports.sh.cpp:11-12
+
+// This file checks that the built static library doesn't contain dllexport
+// directives.
+
----------------



================
Comment at: libcxx/test/libcxx/vendor/clangcl/static-lib-exports.sh.cpp:14
+
+// RUN: llvm-readobj --coff-directives "%{lib}/libc++.lib" | not grep -i "export:" > /dev/null
+
----------------
Is `llvm-readobj` always available? If not, it would be nice to avoid the creation of a different `lit` feature for each utility that we need to use. I wonder whether it would be a reasonable requirement to make sure `llvm-readobj` is built in order to run the libc++ test suite. Maybe not?


================
Comment at: libcxx/test/libcxx/vendor/mingw/static-lib-exports.sh.cpp:11-12
+
+// This file checks that the built static library doesn't contain dllexport
+// directives.
+
----------------



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121164