[Bug c++/107632] New: has_facet does not work with -mlong-double-64

[Bug c++/107632] New: has_facet does not work with -mlong-double-64

[krebbel at gcc dot gnu.org]

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

            Bug ID: 107632
           Summary: has_facet does not work with -mlong-double-64
           Product: gcc
           Version: 13.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: krebbel at gcc dot gnu.org
  Target Milestone: ---

#include <locale>
#include <stdio.h>

using namespace std;

int main(int argc, char *argv[]) {

  locale oGlobalLocale;

  if (!has_facet< num_get<char, istreambuf_iterator<char, char_traits<char> > >
>( oGlobalLocale ))
    __builtin_abort ();
}

g++ t.cpp -o t && ./t   -> works as expected
g++ t.cpp -o t -mlong-double-64 && ./t   -> aborts

[Bug libstdc++/107632] has_facet does not work with -mlong-double-64

[redi at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |ASSIGNED
           Keywords|                            |ABI
   Last reconfirmed|                            |2022-11-11
     Ever confirmed|0                           |1
           Assignee|unassigned at gcc dot gnu.org      |redi at gcc dot gnu.org
          Component|c++                         |libstdc++

--- Comment #1 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Almost certainly due to my changes for PR 103755

[Bug libstdc++/107632] [13 Regression] has_facet does not work with -mlong-double-64

[redi at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
            Summary|has_facet does not work     |[13 Regression] has_facet
                   |with -mlong-double-64       |does not work with
                   |                            |-mlong-double-64
           Priority|P3                          |P1
   Target Milestone|---                         |13.0

[Bug libstdc++/107632] [13 Regression] has_facet does not work with -mlong-double-64

[redi at gcc dot gnu.org]

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

--- Comment #2 from Jonathan Wakely <redi at gcc dot gnu.org> ---
I'm kinda tempted to just disable the new optimization on these targets, the
handling of compat facets for different float ABIs is impossible to get right.

[Bug libstdc++/107632] has_facet does not work with -mlong-double-64

[redi at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|                            |12.1.0, 13.0, 6.3.1, 9.1.0
            Summary|[13 Regression] has_facet   |has_facet does not work
                   |does not work with          |with -mlong-double-64
                   |-mlong-double-64            |
   Target Milestone|13.0                        |---

--- Comment #3 from Jonathan Wakely <redi at gcc dot gnu.org> ---
Oh wait, not a regression, at least not caused by me this week.

This has failed since at least gcc 6.

[Bug libstdc++/107632] has_facet does not work with -mlong-double-64

[redi at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Assignee|redi at gcc dot gnu.org            |unassigned at gcc dot gnu.org
             Status|ASSIGNED                    |NEW