[PATCH] D40451: [OpenMP] Add function attribute for triggering shared memory lowering in the LLVM backend

Hal Finkel via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 14 15:52:33 PST 2017


hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp:937
     if (!CapturedVars.empty()) {
+      // There's somehting to share, add the attribute
+      CGF.CurFn->addFnAttr("has-nvptx-shared-depot");
----------------
somehting -> something, add a period at the end of the comment.


Repository:
  rL LLVM

https://reviews.llvm.org/D40451





More information about the cfe-commits mailing list