[libcxx-commits] [PATCH] D101565: [libcxx] [ci] Add a Windows CI configuration for a statically linked libc++

[Martin Storsjö via Phabricator via libcxx-commits]

mstorsjo added inline comments.


================
Comment at: libcxx/utils/ci/buildkite-pipeline.yml:321
+  - label: "Windows (DLL)"
+    command: "bash libcxx/utils/ci/run-buildbot generic-win"
+    artifact_paths:
----------------
curdeius wrote:
> ?
Oh, sorry, yes you're right - I've missed in resolving git conflicts here after removing the bash path recently.


================
Comment at: libcxx/utils/ci/run-buildbot:494
+    CMAKE_FLAGS=""
+    if [ "${BUILDER}" = "generic-win-static" ]; then
+        CMAKE_FLAGS="$CMAKE_FLAGS -DLIBCXX_ENABLE_SHARED=NO"
----------------
curdeius wrote:
> I'm not a big fan of having this if inside case. I'd rather go for factoring out common flags to a function.
> The current way is harder to reason about IMO.
Ok, I can try to make it into a function.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D101565