-
Notifications
You must be signed in to change notification settings - Fork 109
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
[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.
- Loading branch information
Showing
1 changed file
with
30 additions
and
5 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters