[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

2020-09-01 Thread Fady Ghanim via Phabricator via cfe-commits
fghanim added a comment. In D85619#2248195 , @kiranchandramohan wrote: > What is the plan for this patch? Waiting on you to review it ;) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85619/new/ https://re

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

2020-08-31 Thread Kiran Chandramohan via Phabricator via cfe-commits
kiranchandramohan added a comment. What is the plan for this patch? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85619/new/ https://reviews.llvm.org/D85619 ___ cfe-commits mailing list cfe-commits@lists

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

2020-08-13 Thread Fady Ghanim via Phabricator via cfe-commits
fghanim added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85619/new/ https://reviews.llvm.org/D85619 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-b

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

2020-08-10 Thread Fady Ghanim via Phabricator via cfe-commits
fghanim added a comment. In D85619#2208425 , @kiranchandramohan wrote: > Before I start, Why is this change all new code and no modification or > deletion of existing Clang functionality for omp single/copyprivate? Because the OMPBuilder currently is no

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

2020-08-10 Thread Kiran Chandramohan via Phabricator via cfe-commits
kiranchandramohan added a comment. Before I start, Why is this change all new code and no modification or deletion of existing Clang functionality for omp single/copyprivate? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85619/new/ https://reviews

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

2020-08-09 Thread Fady Ghanim via Phabricator via cfe-commits
fghanim added a comment. Feel free to add other reviewers. Thanks. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D85619/new/ https://reviews.llvm.org/D85619 ___ cfe-commits mailing list cfe-commits@lists.

[PATCH] D85619: [clang][OpenMP][OMPBuilder] Use OMPBuilder to CG `omp single`

2020-08-09 Thread Fady Ghanim via Phabricator via cfe-commits
fghanim created this revision. Herald added subscribers: cfe-commits, aaron.ballman, guansong, yaxunl. Herald added a project: clang. fghanim requested review of this revision. Herald added a reviewer: jdoerfert. Herald added a subscriber: sstefan1. ADD support to allow `omp single` to be CG-ed by