[libcxx-commits] [PATCH] D111359: [libc++][CI] Add AIX pipeline config

Louis Dionne via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Oct 18 13:00:56 PDT 2021


ldionne requested changes to this revision.
ldionne added a comment.
This revision now requires changes to proceed.

This is awesome! I see that you currently have a single `aix` builder, and it takes about 45 minutes to run the tests. I can imagine that becoming a bottleneck -- do you think there is any way we could increase the capacity on aix builders?

Also, you could add `AIX` to the set of supported platforms in `libcxx/docs/index.rst` in this commit, since you are adding a CI job for it.

Finally, I assume there's a commitment to trying to fix the remaining failures on that platform?



================
Comment at: libcxx/utils/ci/buildkite-pipeline.yml:693
+    env:
+        OBJECT_MODE: "32"
+    agents:
----------------
Is it possible to handle that by using a different target triple instead? That would be consistent with what we try to do elsewhere.


================
Comment at: libcxx/utils/ci/buildkite-pipeline.yml:694-695
+        OBJECT_MODE: "32"
+    agents:
+        os: aix
+    retry:
----------------
Would it be possible to assign those builders to the `libcxx-builders` queue?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111359



More information about the libcxx-commits mailing list