[PATCH] D11395: [OpenMP] Add capture for threadprivate variables used in copyin clause

Samuel Antao sfantao at us.ibm.com
Thu Jul 23 14:54:56 PDT 2015


sfantao updated the summary for this revision.
sfantao updated this revision to Diff 30526.
sfantao added a comment.

Add logic to capture variables in the copyin clause even if there is no uses in the captured region. We need to capture it anyway because there may be uses that are not visible in compilation unit.

The new logic also prevents the capture declaration from being propagated to innermost scopes.

This patch correctly fixes the issue, but wanted to make this  approach is approved so that I can go ahead and update the regression tests.

Thanks!
Samuel


http://reviews.llvm.org/D11395

Files:
  include/clang/Sema/Sema.h
  lib/CodeGen/CGStmtOpenMP.cpp
  lib/Parse/ParseOpenMP.cpp
  lib/Sema/SemaExpr.cpp
  lib/Sema/SemaOpenMP.cpp
  lib/Sema/TreeTransform.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D11395.30526.patch
Type: text/x-patch
Size: 12170 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150723/c467fa42/attachment.bin>


More information about the cfe-commits mailing list