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
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
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
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