[libcxx-commits] [PATCH] D112215: [libcxx] [ci] Add CI configurations for MinGW

[Louis Dionne via Phabricator via libcxx-commits]

ldionne requested changes to this revision.
ldionne added inline comments.
This revision now requires changes to proceed.


================
Comment at: libcxx/utils/ci/buildkite-pipeline.yml:509
   # Tests on non-Unix platforms
-  - label: "Windows (DLL)"
-    command: "bash libcxx/utils/ci/run-buildbot windows-dll"
+  - label: "MSVC (DLL)"
+    command: "bash libcxx/utils/ci/run-buildbot msvc-dll"
----------------
I would prefer `Clang-cl` or something like that. For me, MSVC refers to the real MSVC compiler, not the Clang emulation.


================
Comment at: libcxx/utils/ci/run-buildbot:634
+    echo "+++ Running the libc++ tests"
+    ${NINJA} -vC "${BUILD_DIR}" check-cxx
+;;
----------------
We don't run the libc++abi tests?


================
Comment at: libcxx/utils/ci/run-buildbot:651
+    echo "+++ Running the libc++ tests"
+    ${NINJA} -vC "${BUILD_DIR}" check-cxx
+;;
----------------
It seems to me that the tests should have a dependency on libunwind if we are linking them against the just-built libunwind, no? Ideally we'd fix that in a separate patch before landing this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112215