[Bug libstdc++/115512] New: _Float32 overloads in cmath cause ICE for default AVR

[Bug libstdc++/115512] New: _Float32 overloads in cmath cause ICE for default AVR

[dv at vollmann dot ch]

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

            Bug ID: 115512
           Summary: _Float32 overloads in cmath cause ICE for default AVR
           Product: gcc
           Version: 15.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: libstdc++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: dv at vollmann dot ch
  Target Milestone: ---
            Target: avr

As reported in <https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115511>, g++
produces an ICE for calling functions with _Float32 for which both float and
double overloads exist.
I got it for __hypot3, but there are probably several others.
I'm wondering if cmath should provide overloads for _Float32 at all if double
is 32 bits.

The problem can be reproduced with a single line source:

#include <cmath>

avr-g++ -std=gnu++23 -o cmath-only.o -c cmath-only.cc

[Bug libstdc++/115512] _Float32 overloads in cmath cause ICE for default AVR

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jakub at gcc dot gnu.org

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Of course cmath needs to provide _Float32 overloads when _Float32 is supported.
I've provided a fix for the other bug and this is IMNSHO a non-bug.

[Bug libstdc++/115512] _Float32 overloads in cmath cause ICE for default AVR

[dv at vollmann dot ch]

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

--- Comment #2 from dv at vollmann dot ch ---
On 6/17/24 10:59, jakub at gcc dot gnu.org wrote:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115512
> 
> Jakub Jelinek <jakub at gcc dot gnu.org> changed:
> 
>             What    |Removed                     |Added
> ----------------------------------------------------------------------------
>                   CC|                            |jakub at gcc dot gnu.org
> 
> --- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Of course cmath needs to provide _Float32 overloads when _Float32 is supported.
> I've provided a fix for the other bug and this is IMNSHO a non-bug.

I agree. I didn't know about _Float32 and thought it's just
a typedef.  But as a real type providing these overloads is fine.

[Bug libstdc++/115512] _Float32 overloads in cmath cause ICE for default AVR

[dv at vollmann dot ch]

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

--- Comment #3 from dv at vollmann dot ch ---
On 6/17/24 10:59, jakub at gcc dot gnu.org wrote:

> --- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
> Of course cmath needs to provide _Float32 overloads when _Float32 is supported.
> I've provided a fix for the other bug and this is IMNSHO a non-bug.

I've now built a libstdc++ with your fix and cmath compiles fine :-)
So this indeed is a non-bug.

Thanks!

[Bug libstdc++/115512] _Float32 overloads in cmath cause ICE for default AVR

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|---                         |INVALID

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
.