[libcxx-commits] [PATCH] D143158: [libcxx][test] Cover RISC-V in string.capacity test

[Nikolas Klauser via Phabricator via libcxx-commits]

philnik added a comment.

In D143158#4115378 <https://reviews.llvm.org/D143158#4115378>, @asb wrote:

> The runner I'm adding in D143172 <https://reviews.llvm.org/D143172> runs the libc++ tests on RISC-V, which should avoid this becoming dead.

We generally require a runner in the libc++ precommit-CI itself. See https://libcxx.llvm.org/AddingNewCIJobs.html#addingnewcijobs for more information. Is the post-commit CI documented somewhere? If that's the case we should probably add a sentence about libc++ requiring a pre-commit CI runner. It seems that most people aren't aware of this. D139147 <https://reviews.llvm.org/D139147> might me interesting in case you want to cross-compile the code and run it in an emulator.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143158