On Wed, Jul 31, 2024 at 09:26:43PM +0100, Jonathan Wakely wrote: > On Wed, 31 Jul 2024 at 19:18, Jonathan Wakely <jwak...@redhat.com> wrote: > > > > On Wed, 31 Jul 2024 at 19:17, Dimitar Dimitrov <dimi...@dinux.eu> wrote: > > > > > > On Wed, Jul 31, 2024 at 05:09:52PM +0100, Jonathan Wakely wrote: > > > > It took a while, but I was finally happy with this v4 patch, so I pushed > > > > it to trunk. Then I noticed silly mistake in the new test, which I'll > > > > fix shortly. > > > > > > > ... > > > > +#if defined _GLIBCXX_HAVE_ICONV > > > > + ::iconv_t _M_cd = (::iconv_t)-1; > > > > + mutable mutex mx; > > > > +#endif > > > > > > Hi Jonathan, > > > > > > This patch caused pru-unknown-elf build to break: > > > > > > /mnt/nvme/dinux/local-workspace/gcc/libstdc++-v3/src/c++20/format.cc:86:11: > > > error: ‘mutex’ does not name a type > > > 86 | mutable mutex mx; > > > | ^~~~~ > > > > > > Should the section also be guarded by _GLIBCXX_HAS_GTHREADS ? > > > > Doh, yes it should. I'll fix that today. > > I've pushed this, which should fix it.
Thank you. Build is healthy now, as confirmed by: https://gcc.gnu.org/pipermail/gcc-testresults/2024-August/821338.html Regards, Dimitar