Re: [PATCH] c++: Fix mangling of _Float16 template args [PR121801]

[Matthias Kretz]

Andrew Pinski [Friday, 5 September 2025, 22:50:30 CEST]:
> On Fri, Sep 5, 2025 at 4:06 AM Matthias Kretz <m.kr...@gsi.de> wrote:
> > diff --git a/gcc/testsuite/g++.dg/pr121801_float16_cst_mangling.C b/gcc/
> > testsuite/g++.dg/pr121801_float16_cst_mangling.C
> > new file mode 100644
> > index 00000000000..030822ff1c8
> > --- /dev/null
> > +++ b/gcc/testsuite/g++.dg/pr121801_float16_cst_mangling.C
> > @@ -0,0 +1,12 @@
> > +// PR c++/121801
> > +// { dg-do compile { target c++20 } }
> 
> Since you are using _Float16 and not  all target supports _Float16,
> this should also use float16 as the target marker so:
> // { dg-do compile { target {c++20 && float16 }  } }

Thanks. I assume doing this ^ is enough so that I don't have to any of what 
Jakub wrote? (dg-add-options float16, #ifdef __FLT16_MANT_DIG__, and { target 
float16 } on dg-final)

-- 
──────────────────────────────────────────────────────────────────────────
 Dr. Matthias Kretz                           https://mattkretz.github.io
 GSI Helmholtz Center for Heavy Ion Research               https://gsi.de
 std::simd
──────────────────────────────────────────────────────────────────────────