[PATCH] D28145: [OpenMP] Basic support for a parallel directive in a target region on an NVPTX device.

Arpith Jacob via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Dec 28 15:18:30 PST 2016


arpith-jacob added inline comments.


================
Comment at: lib/CodeGen/CGOpenMPRuntime.cpp:114
   /// \brief Get the name of the capture helper.
-  StringRef getHelperName() const override { return ".omp_outlined."; }
+  StringRef getHelperName() const override { return "__omp_outlined__"; }
 
----------------
On the nvptx device, it is illegal for an identifier to contain a dot ('.') so I've modified it here.  If there is a better way to do this, please let me know.


https://reviews.llvm.org/D28145





More information about the cfe-commits mailing list