public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug libstdc++/97930] New: pair is not a structural type
@ 2020-11-20 18:03 stinkingmadgod at gmail dot com
  2020-11-20 20:06 ` [Bug libstdc++/97930] " redi at gcc dot gnu.org
                   ` (7 more replies)
  0 siblings, 8 replies; 9+ messages in thread
From: stinkingmadgod at gmail dot com @ 2020-11-20 18:03 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

            Bug ID: 97930
           Summary: pair is not a structural type
           Product: gcc
           Version: 11.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libstdc++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: stinkingmadgod at gmail dot com
  Target Milestone: ---

template<std::pair<int, int>>
struct S;

This is valid in C++20, but fails due to libstdc++ using a private base
__pair_base, rendering pair non-structural.

[pairs.pairs]
pair<T, U> is a structural type if T and U are both structural types.

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
@ 2020-11-20 20:06 ` redi at gcc dot gnu.org
  2020-11-20 20:07 ` redi at gcc dot gnu.org
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2020-11-20 20:06 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Ugh, we need that base class to make assignment operators work correctly.

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
  2020-11-20 20:06 ` [Bug libstdc++/97930] " redi at gcc dot gnu.org
@ 2020-11-20 20:07 ` redi at gcc dot gnu.org
  2021-04-07 10:29 ` redi at gcc dot gnu.org
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2020-11-20 20:07 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
     Ever confirmed|0                           |1
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2020-11-20

--- Comment #2 from Jonathan Wakely <redi at gcc dot gnu.org> ---
I suppose making it public won't hurt.

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
  2020-11-20 20:06 ` [Bug libstdc++/97930] " redi at gcc dot gnu.org
  2020-11-20 20:07 ` redi at gcc dot gnu.org
@ 2021-04-07 10:29 ` redi at gcc dot gnu.org
  2021-04-28 17:17 ` cvs-commit at gcc dot gnu.org
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2021-04-07 10:29 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|---                         |12.0

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
                   ` (2 preceding siblings ...)
  2021-04-07 10:29 ` redi at gcc dot gnu.org
@ 2021-04-28 17:17 ` cvs-commit at gcc dot gnu.org
  2021-04-28 17:20 ` redi at gcc dot gnu.org
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: cvs-commit at gcc dot gnu.org @ 2021-04-28 17:17 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

--- Comment #3 from CVS Commits <cvs-commit at gcc dot gnu.org> ---
The master branch has been updated by Jonathan Wakely <redi@gcc.gnu.org>:

https://gcc.gnu.org/g:c8767ee9f9355a63bfeb8318df32bc39c5b0f3ad

commit r12-225-gc8767ee9f9355a63bfeb8318df32bc39c5b0f3ad
Author: Jonathan Wakely <jwakely@redhat.com>
Date:   Wed Apr 28 18:14:05 2021 +0100

    libstdc++: Add testcase for std::pair as a structural type [PR 97930]

    This PR was fixed by r12-221-ge1543e694dadf1ea70eb72325219bc0cdc914a35
    (for compilers that support C++20 Concepts) so this adds the testcase.

    libstdc++-v3/ChangeLog:

            PR libstdc++/97930
            * testsuite/20_util/pair/requirements/structural.cc: New test.

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
                   ` (3 preceding siblings ...)
  2021-04-28 17:17 ` cvs-commit at gcc dot gnu.org
@ 2021-04-28 17:20 ` redi at gcc dot gnu.org
  2021-07-28  7:04 ` rguenth at gcc dot gnu.org
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2021-04-28 17:20 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|12.0                        |11.2

--- Comment #4 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Fixed on trunk by using concepts for std::pair in C++20 mode.

For GCC 11 (and maybe 10) just changing the base to public is probably OK.

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
                   ` (4 preceding siblings ...)
  2021-04-28 17:20 ` redi at gcc dot gnu.org
@ 2021-07-28  7:04 ` rguenth at gcc dot gnu.org
  2022-09-29 18:06 ` redi at gcc dot gnu.org
  2023-05-29 10:03 ` jakub at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: rguenth at gcc dot gnu.org @ 2021-07-28  7:04 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|11.2                        |---

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
                   ` (5 preceding siblings ...)
  2021-07-28  7:04 ` rguenth at gcc dot gnu.org
@ 2022-09-29 18:06 ` redi at gcc dot gnu.org
  2023-05-29 10:03 ` jakub at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: redi at gcc dot gnu.org @ 2022-09-29 18:06 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

Jonathan Wakely <redi at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |rejects-valid
      Known to work|                            |12.1.0
             Blocks|                            |88322
      Known to fail|                            |11.3.0
   Target Milestone|---                         |11.4


Referenced Bugs:

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=88322
[Bug 88322] Implement C++20 library features.

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

* [Bug libstdc++/97930] pair is not a structural type
  2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
                   ` (6 preceding siblings ...)
  2022-09-29 18:06 ` redi at gcc dot gnu.org
@ 2023-05-29 10:03 ` jakub at gcc dot gnu.org
  7 siblings, 0 replies; 9+ messages in thread
From: jakub at gcc dot gnu.org @ 2023-05-29 10:03 UTC (permalink / raw)
  To: gcc-bugs

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=97930

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|11.4                        |11.5

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 11.4 is being released, retargeting bugs to GCC 11.5.

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

end of thread, other threads:[~2023-05-29 10:03 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-11-20 18:03 [Bug libstdc++/97930] New: pair is not a structural type stinkingmadgod at gmail dot com
2020-11-20 20:06 ` [Bug libstdc++/97930] " redi at gcc dot gnu.org
2020-11-20 20:07 ` redi at gcc dot gnu.org
2021-04-07 10:29 ` redi at gcc dot gnu.org
2021-04-28 17:17 ` cvs-commit at gcc dot gnu.org
2021-04-28 17:20 ` redi at gcc dot gnu.org
2021-07-28  7:04 ` rguenth at gcc dot gnu.org
2022-09-29 18:06 ` redi at gcc dot gnu.org
2023-05-29 10:03 ` jakub at gcc dot gnu.org

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).