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

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

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

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

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