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

Reply via email to