Re: [PATCH][libatomic] Add nvptx support

2020-09-10 Thread Tobias Burnus
Hi all, hello Tom, On 9/8/20 8:51 AM, Tom de Vries wrote: Add nvptx support to libatomic. I have tried tried this patch together with the latest patch for/of PR97000 for tree-cfgcleanup.c on tree-cfgcleanup.c: Now, I only see the expected failures and the compare_and_swap issue in the testsui

Re: [PATCH][libatomic] Add nvptx support

2020-09-09 Thread Tobias Burnus
Hi Tom, On 9/9/20 3:15 PM, Tom de Vries wrote: And I observe the following fails – which seems to be new and related to your patch (but I have not confirmed it by reverting your libatomic patch). Could you confirm that? It is an indenpend issue; it is newish and I can reproduce it on x86-64-

Re: [PATCH][libatomic] Add nvptx support

2020-09-09 Thread Tom de Vries
On 9/9/20 4:14 PM, Tom de Vries wrote: > On 9/9/20 3:15 PM, Tom de Vries wrote: >> On 9/9/20 2:36 PM, Tobias Burnus wrote: >>> Hi Tom, >>> >>> On 9/8/20 5:05 PM, Tobias Burnus wrote: >>> On 9/8/20 8:51 AM, Tom de Vries wrote: >     PR target/96964 >     * config/nvptx/nvptx.md (define_

Re: [PATCH][libatomic] Add nvptx support

2020-09-09 Thread Tom de Vries
On 9/9/20 3:15 PM, Tom de Vries wrote: > On 9/9/20 2:36 PM, Tobias Burnus wrote: >> Hi Tom, >> >> On 9/8/20 5:05 PM, Tobias Burnus wrote: >> >>> On 9/8/20 8:51 AM, Tom de Vries wrote:     PR target/96964     * config/nvptx/nvptx.md (define_expand "atomic_test_and_set"): New     expan

Re: [PATCH][libatomic] Add nvptx support

2020-09-09 Thread Tom de Vries
On 9/9/20 2:36 PM, Tobias Burnus wrote: > Hi Tom, > > On 9/8/20 5:05 PM, Tobias Burnus wrote: > >> On 9/8/20 8:51 AM, Tom de Vries wrote: >>>     PR target/96964 >>>     * config/nvptx/nvptx.md (define_expand "atomic_test_and_set"): New >>>     expansion. >>>     * sync-builtins.def (BUILT_IN_ATO

Re: [PATCH][libatomic] Add nvptx support

2020-09-09 Thread Tobias Burnus
Hi Tom, On 9/8/20 5:05 PM, Tobias Burnus wrote: On 9/8/20 8:51 AM, Tom de Vries wrote: PR target/96964 * config/nvptx/nvptx.md (define_expand "atomic_test_and_set"): New expansion. * sync-builtins.def (BUILT_IN_ATOMIC_TEST_AND_SET_1): New builtin. I have your patch applied on

Re: [PATCH][libatomic] Add nvptx support

2020-09-08 Thread Tobias Burnus
On 9/8/20 8:51 AM, Tom de Vries wrote: Add nvptx support to libatomic. I tried it on powerpc64le-none-linux-gnu and that solves the __sync_val_compare_and_swap_16 issue, I reported at https://gcc.gnu.org/pipermail/gcc-patches/2020-September/553070.html However, when trying Jakub's example (se