[Openmp-commits] [PATCH] D69473: [nfc][libomptarget] Decrease coupling between files

Jon Chesterfield via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Sat Oct 26 22:08:12 PDT 2019


JonChesterfield created this revision.
JonChesterfield added reviewers: ABataev, jdoerfert, grokos.
Herald added a project: OpenMP.
Herald added a subscriber: openmp-commits.

[nfc][libomptarget] Decrease coupling between files

debug.h used the symbol omptarget_device_environment so implicitly required
an include of omptarget-nvptx.h to compile. Similarly interface.h uses size_t.

Moving this declaration to a new header means cancel, critical can now build
without omptarget-nvptx.h. After this change, debug.h, cancel.cu, critical.cu
could move under a common source directory.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D69473

Files:
  openmp/libomptarget/deviceRTLs/interface.h
  openmp/libomptarget/deviceRTLs/nvptx/src/cancel.cu
  openmp/libomptarget/deviceRTLs/nvptx/src/critical.cu
  openmp/libomptarget/deviceRTLs/nvptx/src/debug.h
  openmp/libomptarget/deviceRTLs/nvptx/src/device_environment.h
  openmp/libomptarget/deviceRTLs/nvptx/src/omp_data.cu
  openmp/libomptarget/deviceRTLs/nvptx/src/omptarget-nvptx.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D69473.226564.patch
Type: text/x-patch
Size: 4726 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/openmp-commits/attachments/20191027/72b6c5c0/attachment.bin>


More information about the Openmp-commits mailing list