Commit Graph

6 Commits

Author SHA1 Message Date
Tom Deakin
74f705cac9 Update OpenMP for Issue #80 2020-12-07 10:41:48 +00:00
Tom Deakin
73f122089d [OpenMP 4.5] Remove superfluous map clauses
Fixes #41.
OpenMP 4.5 does not require the map clauses on the target region
if the data has been previously defined using unstructured data
enter/exit clauses. Removing this clauses works fine with the
Clang compiler, however we noticed issues with the Cray compiler.
The issue is that the Cray compiler does not block the target
region "kernel calls" and so the timing is incorrect. This was
not noticed before due to the presence of the map clauses.
For now, we have had to add an update from clause of a scalar
value to ensure that the kenel blocks. It is hoped that we can
remove this in due course. But in the vein of showing how the
models work we want to keep the minimum required correct code
(which is without the map clause) but need the code to also
work correctly.
2018-02-07 15:05:06 +00:00
Tom Deakin
8b83802fb1 [OpenMP 4.5] Swap map and reduction clause on dot kernel
This fixes #42. On the Clang compiler, the map clause should appear
before the reduction clause to ensure that the evaluation order
of the clauses copies the reduction value back correctly.
This makes no difference with the Cray compiler.
2018-02-07 14:16:39 +00:00
James Price
703eb945a2 [OpenMP] Align memory (2MB by default) 2017-03-13 17:17:20 +00:00
Tom Deakin
d75b0004de [OMP] Update deconstructor to only call target region if building for GPU 2016-12-13 11:45:30 +00:00
Tom Deakin
e6615944f4 Use a compiler switch to select OpenMP directives (target or parallel for) 2016-12-09 12:24:08 +00:00