This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Remove implicit data sharing code gen that aims to use device shared memory
ClosedPublic

Authored by gtbercea on Feb 22 2018, 8:37 AM.

Details

Summary

Remove this scheme for now since it will be covered by another more generic scheme using global memory. This code will be worked into an optimization for the generic data sharing scheme. Removing this completely and then adding it via future patches will make all future data sharing patches cleaner.

Diff Detail

Repository
rC Clang

Event Timeline

gtbercea created this revision.Feb 22 2018, 8:37 AM
ABataev added inline comments.Feb 22 2018, 8:48 AM
lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp
552

Pass non-null SourceLocation here as the last argument

gtbercea updated this revision to Diff 136570.Mar 1 2018, 10:23 AM

Add Source location.

gtbercea marked an inline comment as done.Mar 1 2018, 10:24 AM
This revision is now accepted and ready to land.Mar 1 2018, 10:25 AM
This revision was automatically updated to reflect the committed changes.
lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp