* LTO apparently does not support _FloatNx types
@ 2023-01-12 16:12 Paul Iannetta
2023-01-12 16:35 ` Richard Biener
0 siblings, 1 reply; 4+ messages in thread
From: Paul Iannetta @ 2023-01-12 16:12 UTC (permalink / raw)
To: gcc
Hi,
I was investigating an ICE (in our yet to be upstreamed back-end which
has native support for float16), on "gcc.dg/torture/float16-complex.c"
when compiled with lto:
./gcc/build/gcc/xgcc -B./gcc/build/gcc/ ./gcc/gcc/testsuite/gcc.dg/torture/float16-complex.c \
-O2 -flto -fno-use-linker-plugin -flto-partition=none -lm -o ./float16-complex.exe
I narrowed it down to the fact that lto-lang does not support _FloatNx
types, the function "lto_type_for_mode" (in gcc/lto/lto-lang.c) and
"c_common_type_for_mode" (in gcc/c-family/c-common.c) are exactly the
same except that "lto_type_for_mode" does not support _FloatNx.
Is this intentional or an oversight?
Cheers,
Paul
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: LTO apparently does not support _FloatNx types
2023-01-12 16:12 LTO apparently does not support _FloatNx types Paul Iannetta
@ 2023-01-12 16:35 ` Richard Biener
2023-01-13 7:15 ` Richard Biener
0 siblings, 1 reply; 4+ messages in thread
From: Richard Biener @ 2023-01-12 16:35 UTC (permalink / raw)
To: Paul Iannetta; +Cc: gcc
> Am 12.01.2023 um 17:18 schrieb Paul Iannetta via Gcc <gcc@gcc.gnu.org>:
>
> Hi,
>
> I was investigating an ICE (in our yet to be upstreamed back-end which
> has native support for float16), on "gcc.dg/torture/float16-complex.c"
> when compiled with lto:
>
> ./gcc/build/gcc/xgcc -B./gcc/build/gcc/ ./gcc/gcc/testsuite/gcc.dg/torture/float16-complex.c \
> -O2 -flto -fno-use-linker-plugin -flto-partition=none -lm -o ./float16-complex.exe
>
> I narrowed it down to the fact that lto-lang does not support _FloatNx
> types, the function "lto_type_for_mode" (in gcc/lto/lto-lang.c) and
> "c_common_type_for_mode" (in gcc/c-family/c-common.c) are exactly the
> same except that "lto_type_for_mode" does not support _FloatNx.
>
> Is this intentional or an oversight?
It’s probably an oversight.
> Cheers,
> Paul
>
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: LTO apparently does not support _FloatNx types
2023-01-12 16:35 ` Richard Biener
@ 2023-01-13 7:15 ` Richard Biener
2023-01-13 8:15 ` Paul Iannetta
0 siblings, 1 reply; 4+ messages in thread
From: Richard Biener @ 2023-01-13 7:15 UTC (permalink / raw)
To: Paul Iannetta; +Cc: gcc
On Thu, Jan 12, 2023 at 5:35 PM Richard Biener
<richard.guenther@gmail.com> wrote:
>
>
>
> > Am 12.01.2023 um 17:18 schrieb Paul Iannetta via Gcc <gcc@gcc.gnu.org>:
> >
> > Hi,
> >
> > I was investigating an ICE (in our yet to be upstreamed back-end which
> > has native support for float16), on "gcc.dg/torture/float16-complex.c"
> > when compiled with lto:
> >
> > ./gcc/build/gcc/xgcc -B./gcc/build/gcc/ ./gcc/gcc/testsuite/gcc.dg/torture/float16-complex.c \
> > -O2 -flto -fno-use-linker-plugin -flto-partition=none -lm -o ./float16-complex.exe
> >
> > I narrowed it down to the fact that lto-lang does not support _FloatNx
> > types, the function "lto_type_for_mode" (in gcc/lto/lto-lang.c) and
> > "c_common_type_for_mode" (in gcc/c-family/c-common.c) are exactly the
> > same except that "lto_type_for_mode" does not support _FloatNx.
> >
> > Is this intentional or an oversight?
>
> It’s probably an oversight.
I'm testing a patch to sync them up.
Richard.
>
> > Cheers,
> > Paul
> >
> >
> >
> >
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: LTO apparently does not support _FloatNx types
2023-01-13 7:15 ` Richard Biener
@ 2023-01-13 8:15 ` Paul Iannetta
0 siblings, 0 replies; 4+ messages in thread
From: Paul Iannetta @ 2023-01-13 8:15 UTC (permalink / raw)
To: Richard Biener; +Cc: gcc
On Fri, Jan 13, 2023 at 08:15:34AM +0100, Richard Biener wrote:
> On Thu, Jan 12, 2023 at 5:35 PM Richard Biener
> <richard.guenther@gmail.com> wrote:
> >
> >
> >
> > > Am 12.01.2023 um 17:18 schrieb Paul Iannetta via Gcc <gcc@gcc.gnu.org>:
> > >
> > > Hi,
> > >
> > > I was investigating an ICE (in our yet to be upstreamed back-end which
> > > has native support for float16), on "gcc.dg/torture/float16-complex.c"
> > > when compiled with lto:
> > >
> > > ./gcc/build/gcc/xgcc -B./gcc/build/gcc/ ./gcc/gcc/testsuite/gcc.dg/torture/float16-complex.c \
> > > -O2 -flto -fno-use-linker-plugin -flto-partition=none -lm -o ./float16-complex.exe
> > >
> > > I narrowed it down to the fact that lto-lang does not support _FloatNx
> > > types, the function "lto_type_for_mode" (in gcc/lto/lto-lang.c) and
> > > "c_common_type_for_mode" (in gcc/c-family/c-common.c) are exactly the
> > > same except that "lto_type_for_mode" does not support _FloatNx.
> > >
> > > Is this intentional or an oversight?
> >
> > It’s probably an oversight.
>
> I'm testing a patch to sync them up.
>
> Richard.
>
Wonderful, thanks!
Paul
> >
> > > Cheers,
> > > Paul
> > >
> > >
> > >
> > >
>
>
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-01-13 8:15 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-01-12 16:12 LTO apparently does not support _FloatNx types Paul Iannetta
2023-01-12 16:35 ` Richard Biener
2023-01-13 7:15 ` Richard Biener
2023-01-13 8:15 ` Paul Iannetta
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).