On Thu, Mar 26, 2020 at 10:46:48AM +0100, Martin Liška wrote:
> 2020-03-26 Martin Liska
>
> * gcc.target/i386/pr81213.c: Do not scan assembler
> and add one missing PR entry.
Ok.
> ---
> gcc/testsuite/gcc.target/i386/pr81213.c | 5 +
> 1 file changed, 1 insertion(+), 4 deleti
On 3/26/20 10:23 AM, Jakub Jelinek wrote:
On Mon, Mar 23, 2020 at 04:09:52PM +0100, Martin Liška wrote:
2020-03-17 Martin Liska
PR target/93274 PR lto/94271
* gcc.target/i386/pr81213-2.c: New test.
* gcc.target/i386/pr81213.c: Add additional source.
* gcc.dg/l
On Mon, Mar 23, 2020 at 04:09:52PM +0100, Martin Liška wrote:
> 2020-03-17 Martin Liska
>
> PR target/93274 PR lto/94271
> * gcc.target/i386/pr81213-2.c: New test.
> * gcc.target/i386/pr81213.c: Add additional source.
> * gcc.dg/lto/pr94271_0.c: New test.
> * gcc.d
On Mon, 2020-03-23 at 16:09 +0100, Martin Liška wrote:
> From 1431a34f70128bdce59c94dad1d10f91673f63eb Mon Sep 17 00:00:00 2001
> From: Martin Liska
> Date: Thu, 16 Jan 2020 10:38:41 +0100
> Subject: [PATCH] Make target_clones resolver fn static if possible.
>
> gcc/ChangeLo
come
privatized eventually in LTO.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Martin
>From 1431a34f70128bdce59c94dad1d10f91673f63eb Mon Sep 17 00:00:00 2001
From: Martin Liska
Date: Thu, 16 Jan 2020 10:38:41 +0100
Subject: [PATCH] Make target_
>From 6d2113da263d7a6a0bb0adcf22edb074646f1996 Mon Sep 17 00:00:00 2001
From: Martin Liska
Date: Thu, 16 Jan 2020 10:38:41 +0100
Subject: [PATCH] Make target_clones resolver fn static if possible.
gcc/ChangeLog:
2020-03-17 Martin Liska
PR target/93274
* config/i386/i386-features.c (make_resolver_func): Drop
On 1/26/20 6:35 PM, Jeff Law wrote:
On Tue, 2020-01-21 at 13:48 +0100, Martin Liška wrote:
From a3faaced989869867671ceadd89b56fabde225ff Mon Sep 17 00:00:00 2001
From: Martin Liska
Date: Thu, 16 Jan 2020 10:38:41 +0100
Subject: [PATCH] Make target_clones resolver fn static.
gcc/ChangeLog
On 1/23/20 2:52 PM, Alexander Monakov wrote:
On Thu, 23 Jan 2020, Martin Liška wrote:
So this doesn't help review including two different target changes. Making
ifunc dispatchers of public functions non-public looks like an unrelated
thing
to the bug (sorry if I mis-suggested). So I feel com
On Tue, 2020-01-21 at 13:48 +0100, Martin Liška wrote:
> From a3faaced989869867671ceadd89b56fabde225ff Mon Sep 17 00:00:00 2001
> From: Martin Liska
> Date: Thu, 16 Jan 2020 10:38:41 +0100
> Subject: [PATCH] Make target_clones resolver fn static.
>
> gcc/ChangeLog:
>
>
On Thu, 23 Jan 2020, Martin Liška wrote:
> > So this doesn't help review including two different target changes. Making
> > ifunc dispatchers of public functions non-public looks like an unrelated
> > thing
> > to the bug (sorry if I mis-suggested). So I feel comfortable approving the
> > earli
On 1/23/20 2:09 PM, Richard Biener wrote:
On Tue, Jan 21, 2020 at 1:48 PM Martin Liška wrote:
On 1/20/20 3:52 PM, Richard Biener wrote:
On Mon, Jan 20, 2020 at 3:46 PM H.J. Lu wrote:
On Mon, Jan 20, 2020 at 6:41 AM Alexander Monakov wrote:
On Mon, 20 Jan 2020, H.J. Lu wrote:
Bare IF
On Tue, Jan 21, 2020 at 1:48 PM Martin Liška wrote:
>
> On 1/20/20 3:52 PM, Richard Biener wrote:
> > On Mon, Jan 20, 2020 at 3:46 PM H.J. Lu wrote:
> >>
> >> On Mon, Jan 20, 2020 at 6:41 AM Alexander Monakov
> >> wrote:
> >>>
> >>>
> >>>
> >>> On Mon, 20 Jan 2020, H.J. Lu wrote:
> >>>
> >
00:00 2001
From: Martin Liska
Date: Thu, 16 Jan 2020 10:38:41 +0100
Subject: [PATCH] Make target_clones resolver fn static.
gcc/ChangeLog:
2020-01-17 Martin Liska
PR target/93274
* config/i386/i386-features.c (make_resolver_func):
Align the code with ppc64 target implementaion.
We d
On Mon, Jan 20, 2020 at 3:46 PM H.J. Lu wrote:
>
> On Mon, Jan 20, 2020 at 6:41 AM Alexander Monakov wrote:
> >
> >
> >
> > On Mon, 20 Jan 2020, H.J. Lu wrote:
> >
> > > > Bare IFUNC's don't seem to have this restriction. Why do we want to
> > > > constrain target clones this way?
> > > >
> > >
>
On Mon, Jan 20, 2020 at 6:41 AM Alexander Monakov wrote:
>
>
>
> On Mon, 20 Jan 2020, H.J. Lu wrote:
>
> > > Bare IFUNC's don't seem to have this restriction. Why do we want to
> > > constrain target clones this way?
> > >
> >
> > foo's resolver acts as foo. It should have the same visibility as
On Mon, 20 Jan 2020, H.J. Lu wrote:
> > Bare IFUNC's don't seem to have this restriction. Why do we want to
> > constrain target clones this way?
> >
>
> foo's resolver acts as foo. It should have the same visibility as foo.
What do you mean by that? From the implementation standpoint, there
On Mon, Jan 20, 2020 at 6:16 AM Alexander Monakov wrote:
>
>
>
> On Mon, 20 Jan 2020, H.J. Lu wrote:
> > For,
> >
> > ---
> > __attribute__((target_clones("avx","default")))
> > int
> > foo ()
> > {
> > return -2;
> > }
> >
> >
> > foo's resolver must be global. For
> >
> > ---
> > __attri
On Mon, 20 Jan 2020, H.J. Lu wrote:
> For,
>
> ---
> __attribute__((target_clones("avx","default")))
> int
> foo ()
> {
> return -2;
> }
>
>
> foo's resolver must be global. For
>
> ---
> __attribute__((target_clones("avx","default")))
> static int
> foo ()
> {
> return -2;
> }
> --
On Mon, Jan 20, 2020 at 5:36 AM Alexander Monakov wrote:
>
>
>
> On Mon, 20 Jan 2020, H.J. Lu wrote:
> > We can that only if function is static:
> >
> [ship asm]
> >
> > In this case, foo must be global.
>
> H.J., can you rephrase more clearly? Your response seems contradictory and
> does not help
On Mon, 20 Jan 2020, H.J. Lu wrote:
> We can that only if function is static:
>
[ship asm]
>
> In this case, foo must be global.
H.J., can you rephrase more clearly? Your response seems contradictory and
does not help to explain the matter.
Alexander
On Mon, Jan 20, 2020 at 2:25 AM Richard Biener
wrote:
>
> On Fri, Jan 17, 2020 at 10:25 AM Martin Liška wrote:
> >
> > Hi.
> >
> > The patch removes need to have a gnu_indirect_function global
> > symbol. That aligns the code with what ppc64 target does.
> >
> > Patch can bootstrap on x86_64-linu
On Fri, Jan 17, 2020 at 10:25 AM Martin Liška wrote:
>
> Hi.
>
> The patch removes need to have a gnu_indirect_function global
> symbol. That aligns the code with what ppc64 target does.
>
> Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
>
> Ready to be installed?
Did you
Hi.
The patch removes need to have a gnu_indirect_function global
symbol. That aligns the code with what ppc64 target does.
Patch can bootstrap on x86_64-linux-gnu and survives regression tests.
Ready to be installed?
Thanks,
Martin
gcc/ChangeLog:
2020-01-17 Martin Liska
PR target
23 matches
Mail list logo
