maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[OpenMP][libomptarget] Add support for declare target to clause under unified memory 

Summary:
This patch adds support for handling variables under the:

```
#pragma omp declare target to()
```

clause when the 

```
#pragma omp requires unified_shared_memory
```

is used.

The address of the host variable is copied into the device pointer just like for the declare target link case.

Reviewers: ABataev, caomhin, grokos, AlexEichenberger

Reviewed By: grokos

Subscribers: jcownie, guansong, jdoerfert, openmp-commits

Tags: #openmp

Differential Revision: https://reviews.llvm.org/D63106

llvm-svn: 363825
This commit is contained in:
Gheorghe-Teodor Bercea 2019-06-19 15:48:10 +00:00
parent 6828bc5614
commit aace6d285d
1 changed files with 12 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
openmp/libomptarget/plugins/cuda/src/rtl.cpp
View File

@ -448,9 +448,18 @@ __tgt_target_table *__tgt_rtl_load_binary(int32_t device_id,
DPxPTR(e - HostBegin), e->name, DPxPTR(cuptr)); DPxPTR(e - HostBegin), e->name, DPxPTR(cuptr));
entry.addr = (void *)cuptr; entry.addr = (void *)cuptr;
if (DeviceInfo.RequiresFlags & OMP_REQ_UNIFIED_SHARED_MEMORY && // Note: In the current implementation declare target variables
e->flags & OMP_DECLARE_TARGET_LINK) { // can either be link or to. This means that once unified
// If unified memory is present any target link variables // memory is activated via the requires directive, the variable
// can be used directly from the host in both cases.
// TODO: when variables types other than to or link are added,
// the below condition should be changed to explicitely
// check for to and link variables types:
// (DeviceInfo.RequiresFlags & OMP_REQ_UNIFIED_SHARED_MEMORY &&
// (e->flags & OMP_DECLARE_TARGET_LINK ||
// e->flags == OMP_DECLARE_TARGET_TO))
if (DeviceInfo.RequiresFlags & OMP_REQ_UNIFIED_SHARED_MEMORY) {
// If unified memory is present any target link or to variables
// can access host addresses directly. There is no longer a // can access host addresses directly. There is no longer a
// need for device copies. // need for device copies.
cuMemcpyHtoD(cuptr, e->addr, sizeof(void *)); cuMemcpyHtoD(cuptr, e->addr, sizeof(void *));