This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Add dynamic memory function to omp.h and add documentation
ClosedPublic

Authored by jhuber6 on Apr 6 2022, 12:59 PM.

Details

Summary

This patch adds the llvm_omp_target_dynamic_shared_alloc function to
the omp.h header file so users can access it by default. Also changed
the name to keep it consistent with the other target allocators. Added
some documentation so users know how to use it. Didn't add the interface
for Fortran since there's no way to test it right now.

Diff Detail

Event Timeline

jhuber6 created this revision.Apr 6 2022, 12:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 6 2022, 12:59 PM
jhuber6 requested review of this revision.Apr 6 2022, 12:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptApr 6 2022, 12:59 PM
jdoerfert added inline comments.Apr 7 2022, 8:09 AM
openmp/docs/design/Runtimes.rst
1010

Can we link to the example?

openmp/libomptarget/src/exports
58

We might need to keep the old one for ABI compatibility. Not sure if we require that here though.

jhuber6 updated this revision to Diff 421234.Apr 7 2022, 8:56 AM

Making suggested changes.

This revision is now accepted and ready to land.Apr 7 2022, 10:50 AM
This revision was landed with ongoing or failed builds.Apr 7 2022, 11:23 AM
This revision was automatically updated to reflect the committed changes.