Re: [PATCH][nvptx] Handle memmodel for atomic ops

2021-05-17 Thread Tobias Burnus
Hi Tom, short version works :-) now (and I don't know why it didn't). Long version: On 17.05.21 18:58, Tom de Vries wrote: I have: ... @ %r25 atom.global.exch.b32 %r22,[atomic_lock],1; @ %r25 membar.sys; ... I tried with the PowerPC cross build - running on PowerPC + Quadro GV100 did work for

Re: [PATCH][nvptx] Handle memmodel for atomic ops

2021-05-17 Thread Tom de Vries
On 5/17/21 6:47 PM, Tobias Burnus wrote: > On 17.05.21 17:49, Tom de Vries wrote: >> [ Tobias, can you test this on volta ? ] > > Unfortunately, it does not seem to help. On a non-Volta system, it still > works (run time 0.3s) but on a Volta system it fails after 1.5s (abort). > > Looking (with a

Re: [PATCH][nvptx] Handle memmodel for atomic ops

2021-05-17 Thread Tobias Burnus
On 17.05.21 17:49, Tom de Vries wrote: [ Tobias, can you test this on volta ? ] Unfortunately, it does not seem to help. On a non-Volta system, it still works (run time 0.3s) but on a Volta system it fails after 1.5s (abort). Looking (with an editor) at nvptx-none/lib/mgomp/libgomp.a, I still

[PATCH][nvptx] Handle memmodel for atomic ops

2021-05-17 Thread Tom de Vries
Hi, [ Tobias, can you test this on volta ? ] The atomic ops in nvptx.md have memmodel arguments, which are currently ignored. Handle these, fixing test-case fails libgomp.c-c++-common/reduction-{5,6}.c on volta. Tested libgomp on x86_64-linux with nvptx accelerator. Any comments? Thanks, - To