Re: [PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Richard Sandiford
t; From: Martin Liska > Date: Thu, 21 May 2020 09:19:21 +0200 > Subject: [PATCH] Add outline-atomics to target attribute. > > gcc/ChangeLog: > > 2020-05-21 Martin Liska > > * common/config/aarch64/aarch64-common.c (aarch64_handle_option): >

Re: [PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Martin Liška
ter. Is it also fine for all active branches where you backported the atomics outlining? Martin >From 357a86fbf18ad511a72e6e9c703a6e0b0dd30ddf Mon Sep 17 00:00:00 2001 From: Martin Liska Date: Thu, 21 May 2020 09:19:21 +0200 Subject: [PATCH] Add outline-atomics to target attribute. gcc/ChangeL

Re: [PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Richard Sandiford
Jakub Jelinek via Gcc-patches writes: > On Thu, May 21, 2020 at 10:03:37AM +0200, Martin Liška wrote: >> On 5/21/20 9:56 AM, Jakub Jelinek wrote: >> > Can't it be __attribute__((target ("outline-atomics"))) instead? >> >> Ah sorry, I wan unclear. >> It's support the this _target_ attribute which

Re: [PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Jakub Jelinek via Gcc-patches
On Thu, May 21, 2020 at 10:03:37AM +0200, Martin Liška wrote: > On 5/21/20 9:56 AM, Jakub Jelinek wrote: > > Can't it be __attribute__((target ("outline-atomics"))) instead? > > Ah sorry, I wan unclear. > It's support the this _target_ attribute which you mentioned. Ok, better, will defer review

Re: [PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Martin Liška
On 5/21/20 9:56 AM, Jakub Jelinek wrote: Can't it be __attribute__((target ("outline-atomics"))) instead? Ah sorry, I wan unclear. It's support the this _target_ attribute which you mentioned. Martin

Re: [PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Jakub Jelinek via Gcc-patches
On Thu, May 21, 2020 at 09:23:47AM +0200, Martin Liška wrote: > As mentioned in the ovmf project [1], they would like to have a control > over the -mno-outline-atomics via a function attribute. They struggle > with configure detection and this can help them to disable the outlining. > > Would it b

[PATCH] Add outline-atomics to target attribute.

2020-05-21 Thread Martin Liška
Hi. As mentioned in the ovmf project [1], they would like to have a control over the -mno-outline-atomics via a function attribute. They struggle with configure detection and this can help them to disable the outlining. Would it be possible to add the attribute? Thanks, Martin [1] https://bugzi