[PATCH] D29026: [OpenMP] DSAChecker bug fix for combined directives.

2017-01-23 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/D29026 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c

[PATCH] D29026: [OpenMP] DSAChecker bug fix for combined directives.

2017-01-23 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL292794: [OpenMP] DSAChecker bug fix for combined directives. (authored by arpith). Changed prior to commit: https://reviews.llvm.org/D29026?vs=85382&id=85393#toc Repository: rL LLVM https://reviews.

[PATCH] D29026: [OpenMP] DSAChecker bug fix for combined directives.

2017-01-23 Thread Arpith Jacob via Phabricator via cfe-commits
arpith-jacob created this revision. The DSAChecker code in SemaOpenMP looks at the captured statement associated with an OpenMP directive. A combined directive such as 'target parallel' has nested capture statements, which have to be fully traversed before executing the DSAChecker. This is a pat