[PATCH] D28753: [OpenMP] Codegen support for 'target parallel' on the host.

Arpith Jacob via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 18 04:50:27 PST 2017

arpith-jacob added inline comments.

Comment at: lib/Sema/SemaOpenMP.cpp:1933-1937
+  StmtResult SR = S;
+  int ThisCaptureLevel =
+      getOpenMPCaptureLevels(DSAStack->getCurrentDirective());
+  while (--ThisCaptureLevel >= 0)
+    SR = ActOnCapturedRegionEnd(SR.get());
ABataev wrote:
> Could you move this code to `CaptureRegionUnwinderRAII`?
Hi Alexey, this code will be expanded in the next patch.  Have a look at:
This is the code to codegen preinits.  I think that code doesn't belong to the RAII.  That's why I haven't moved this code to CaptureRegionUnwinderRAII.

Let me know if you disagree.


More information about the cfe-commits mailing list