[PATCH] D34888: [OpenMP] Fix mapping of scalars for combined directives

2017-07-05 Thread Gheorghe-Teodor Bercea via Phabricator via cfe-commits
gtbercea added a comment. Does this also include the fixes in the following revision? https://reviews.llvm.org/D29905 Repository: rL LLVM https://reviews.llvm.org/D34888 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.o

[PATCH] D34888: [OpenMP] Fix mapping of scalars for combined directives

2017-07-01 Thread Jonas Hahnfeld via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL306956: [OpenMP] Fix mapping of scalars for combined directives (authored by Hahnfeld). Changed prior to commit: https://reviews.llvm.org/D34888?vs=104863&id=104986#toc Repository: rL LLVM https://r

[PATCH] D34888: [OpenMP] Fix mapping of scalars for combined directives

2017-06-30 Thread Alexey Bataev via Phabricator via cfe-commits
ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG https://reviews.llvm.org/D34888 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[PATCH] D34888: [OpenMP] Fix mapping of scalars for combined directives

2017-06-30 Thread Jonas Hahnfeld via Phabricator via cfe-commits
Hahnfeld created this revision. Combined directives like 'target parallel' have two captured statements. Sema has to check the right one from the right direction. Previously, Sema::IsOpenMPCapturedByRef would return false for mapped scalars on combined directives. This results in a wrong signat