[libcxx-commits] [libcxx] [libc++] Reenable codecvt in the dylib. (PR #73679)
via libcxx-commits
libcxx-commits at lists.llvm.org
Tue Nov 28 13:47:03 PST 2023
zeroomega wrote:
> > ```
> > [...]
> > ```
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> >
> > Ah, it is still related, see the test failure message.
> > @petrhosek and I discussed about the reason why this Fuchsia unrelated breakage wasn't caught by the upstream libcxx bots. It is likely due to the toolchain used when building libcxx and runtimes.
> > In our case, we always use a relatively recent clang we built to build the ToT clang. The ToT clang will then be used to build the runtime.
> > In upstream llvm bots, for runtime builds, it usually uses a last stable release clang to build the ToT runtime directly to reduce build time. In this case, the bug like this won't be caught until there is a new llvm stable release. I think for libcxx, at least there should be a few bots configured in a way that bots need to use ToT clang to build libcxx and corresponding runtimes for the host platform. Otherwise, breakage like this will be hide in dark.
>
> We do have a bot that runs all of our tests against a just-built Clang using the bootstrapping build, which should catch that.
Do you have a link to that specific bots so we can take a look at the build steps and configurations?
https://github.com/llvm/llvm-project/pull/73679
More information about the libcxx-commits
mailing list