[PATCH v3] math: Exclude tgmath3-macro-tests for Clang

[H.J. Lu]

On Fri, Jan 3, 2025 at 10:04 AM H.J. Lu <hjl.tools@gmail.com> wrote:
>
> On Fri, Jan 3, 2025 at 1:20 AM Joseph Myers <josmyers@redhat.com> wrote:
> >
> > On Sun, 22 Dec 2024, H.J. Lu wrote:
> >
> > > > >  tgmath3-macro-tests = $(addprefix test-tgmath3-,$(tgmath3-macros))
> > > > > +ifneq ($(have-test-clang),yes)
> > > > > +# NB: tgmath3-macro-tests won't compile with Clang's <tgmath.h>.
> > > > >  tests-tgmath += $(tgmath3-macro-tests)
> > > > > +endif
> > > > >  tests += $(tests-tgmath)
> > > > >
> > > > >  generated += $(addsuffix .c,$(tgmath3-macro-tests))
> > > >
> > > > OK with a bug reference / quote of error message in the commit message.
> > >
> > > The llvm bug is
> > >
> > > https://github.com/llvm/llvm-project/issues/97335
> >
> > That's specifically about _FloatN / _FloatNx support (which I suppose does
> > cover adding that support to <tgmath.h> once Clang has the language
> > support).
> >
> > There's also the question of all the new macros added to <tgmath.h> in
> > C23, for new C23 functions (from a quick look, Clang doesn't have those in
> > its <tgmath.h> either).  I think those need a separate bug filed and
> > referenced in the comment.
>
> I opened:
>
> https://github.com/llvm/llvm-project/issues/121536
>

A patch has been submitted.

-- 
H.J.