public inbox for libstdc++@gcc.gnu.org
 help / color / mirror / Atom feed
* Bug in <complex> header in trunk from malformed "requires" statement, causes compilation errors if included
@ 2022-11-18 18:59 Gavin Ray
  2022-11-18 21:14 ` Jonathan Wakely
  0 siblings, 1 reply; 4+ messages in thread
From: Gavin Ray @ 2022-11-18 18:59 UTC (permalink / raw)
  To: libstdc++

Hello all,

Hit a very confusing bug this morning. I'm using Clang with GCC
upstream libstdc++ from jwakely's repo.
The following program fails to compile:

#include <complex>
int main() { return 0; }

It appears there was an initial commit which added "requires" clauses,
and then a fixup commit later:
- libstdc++: Fix up <complex> for extended floating point types
[PR107649] · gcc-mirror/gcc@0e2c551 (github.com)

But there seems to be one last malformed "requires" statement in trunk:
https://github.com/gcc-mirror/gcc/blob/59cc4da605e5cb8e31e9f1d54ef2b5ba47fc8f88/libstdc%2B%2B-v3/include/std/complex#L2531

(Big thanks to Luke D'Alessandro for figuring this all out)
Thought I'd let you folks know.

Thank you!

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Bug in <complex> header in trunk from malformed "requires" statement, causes compilation errors if included
  2022-11-18 18:59 Bug in <complex> header in trunk from malformed "requires" statement, causes compilation errors if included Gavin Ray
@ 2022-11-18 21:14 ` Jonathan Wakely
  2022-11-19 15:09   ` Jonathan Wakely
  0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Wakely @ 2022-11-18 21:14 UTC (permalink / raw)
  To: Gavin Ray; +Cc: libstdc++

On Fri, 18 Nov 2022 at 18:59, Gavin Ray via Libstdc++
<libstdc++@gcc.gnu.org> wrote:
>
> Hello all,
>
> Hit a very confusing bug this morning. I'm using Clang with GCC
> upstream libstdc++ from jwakely's repo.
> The following program fails to compile:
>
> #include <complex>
> int main() { return 0; }
>
> It appears there was an initial commit which added "requires" clauses,
> and then a fixup commit later:
> - libstdc++: Fix up <complex> for extended floating point types
> [PR107649] · gcc-mirror/gcc@0e2c551 (github.com)

Yes, that was https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107649 but
isn't included in my gcc-latest builds yet.

>
> But there seems to be one last malformed "requires" statement in trunk:
> https://github.com/gcc-mirror/gcc/blob/59cc4da605e5cb8e31e9f1d54ef2b5ba47fc8f88/libstdc%2B%2B-v3/include/std/complex#L2531

Thanks.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Bug in <complex> header in trunk from malformed "requires" statement, causes compilation errors if included
  2022-11-18 21:14 ` Jonathan Wakely
@ 2022-11-19 15:09   ` Jonathan Wakely
  2022-11-19 15:19     ` Gavin Ray
  0 siblings, 1 reply; 4+ messages in thread
From: Jonathan Wakely @ 2022-11-19 15:09 UTC (permalink / raw)
  To: Gavin Ray; +Cc: libstdc++

On Fri, 18 Nov 2022 at 21:14, Jonathan Wakely wrote:
>
> On Fri, 18 Nov 2022 at 18:59, Gavin Ray via Libstdc++
> <libstdc++@gcc.gnu.org> wrote:
> >
> > Hello all,
> >
> > Hit a very confusing bug this morning. I'm using Clang with GCC
> > upstream libstdc++ from jwakely's repo.
> > The following program fails to compile:
> >
> > #include <complex>
> > int main() { return 0; }
> >
> > It appears there was an initial commit which added "requires" clauses,
> > and then a fixup commit later:
> > - libstdc++: Fix up <complex> for extended floating point types
> > [PR107649] · gcc-mirror/gcc@0e2c551 (github.com)
>
> Yes, that was https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107649 but
> isn't included in my gcc-latest builds yet.
>
> >
> > But there seems to be one last malformed "requires" statement in trunk:
> > https://github.com/gcc-mirror/gcc/blob/59cc4da605e5cb8e31e9f1d54ef2b5ba47fc8f88/libstdc%2B%2B-v3/include/std/complex#L2531
>
> Thanks.

That's fixed now, thanks for reporting it.

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: Bug in <complex> header in trunk from malformed "requires" statement, causes compilation errors if included
  2022-11-19 15:09   ` Jonathan Wakely
@ 2022-11-19 15:19     ` Gavin Ray
  0 siblings, 0 replies; 4+ messages in thread
From: Gavin Ray @ 2022-11-19 15:19 UTC (permalink / raw)
  To: Jonathan Wakely; +Cc: libstdc++

No problem, thank you for fixing and for maintaining the gcc-latest repo!

On Sat, Nov 19, 2022 at 10:09 AM Jonathan Wakely <jwakely.gcc@gmail.com> wrote:
>
> On Fri, 18 Nov 2022 at 21:14, Jonathan Wakely wrote:
> >
> > On Fri, 18 Nov 2022 at 18:59, Gavin Ray via Libstdc++
> > <libstdc++@gcc.gnu.org> wrote:
> > >
> > > Hello all,
> > >
> > > Hit a very confusing bug this morning. I'm using Clang with GCC
> > > upstream libstdc++ from jwakely's repo.
> > > The following program fails to compile:
> > >
> > > #include <complex>
> > > int main() { return 0; }
> > >
> > > It appears there was an initial commit which added "requires" clauses,
> > > and then a fixup commit later:
> > > - libstdc++: Fix up <complex> for extended floating point types
> > > [PR107649] · gcc-mirror/gcc@0e2c551 (github.com)
> >
> > Yes, that was https://gcc.gnu.org/bugzilla/show_bug.cgi?id=107649 but
> > isn't included in my gcc-latest builds yet.
> >
> > >
> > > But there seems to be one last malformed "requires" statement in trunk:
> > > https://github.com/gcc-mirror/gcc/blob/59cc4da605e5cb8e31e9f1d54ef2b5ba47fc8f88/libstdc%2B%2B-v3/include/std/complex#L2531
> >
> > Thanks.
>
> That's fixed now, thanks for reporting it.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2022-11-19 15:19 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-11-18 18:59 Bug in <complex> header in trunk from malformed "requires" statement, causes compilation errors if included Gavin Ray
2022-11-18 21:14 ` Jonathan Wakely
2022-11-19 15:09   ` Jonathan Wakely
2022-11-19 15:19     ` Gavin Ray

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).