[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-08-02 Thread Chirag Khandelwal via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG77ebfba68b9a: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0. (authored by AMDChirag). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-08-02 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 363625. AMDChirag added a comment. updated commit message Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_print.cp

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-08-02 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 363624. AMDChirag added a comment. updated commit message Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_print.cp

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-08-02 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 363542. AMDChirag added a comment. minor fix in test case Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_print.cp

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-07-28 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 362319. AMDChirag added a comment. Rebased. Verified, this patch is not causing any test failures for me. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: cla

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-07-26 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 361605. AMDChirag added a comment. Minor test fix Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_print.cpp clan

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-07-26 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 361598. AMDChirag added a comment. Removed parent line in summary, it was not longer relevant and caused build failures. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D933

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-07-26 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 361597. AMDChirag added a comment. Rebase to current main branch Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_p

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-07-19 Thread Kiran Chandramohan via Phabricator via cfe-commits
kiranchandramohan accepted this revision. kiranchandramohan added a comment. This revision is now accepted and ready to land. LGTM. Please update the commit message with information about changes in Clang tests as well. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-29 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag added a comment. @jdoerfert Could you please take a look at this patch as well? (especially for the clang part) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 ___

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-28 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag added a comment. @kiranchandramohan The tests passed now! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 ___ cfe-commits mailing list cfe-commits@lists

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-28 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 354839. AMDChirag added a comment. Rebased Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_print.cpp clang/test/

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-22 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag added a comment. @kiranchandramohan It failed again. Apparently, it automatically attempted to add a patch D93105 which is already merged. I removed that patch as dependency before (edit related revisions), but it seems to have been added again. Am I

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-22 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag added a comment. Oh great! That triggered it! Thank you! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 ___ cfe-commits mailing list cfe-commits@lists.

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-22 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 353579. AMDChirag added a comment. Rebased Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_print.cpp clang/test/

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-22 Thread Kiran Chandramohan via Phabricator via cfe-commits
kiranchandramohan added a comment. Can you rebase and update the patch? It seems to have built for the latest revision (Harbormaster completed remote builds in B110336 : Diff 353548). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://r

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-22 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag added a comment. In D93373#2832533 , @kiranchandramohan wrote: > It could be because you are not the author. Can you commandeer the patch > (option available in the Add Action menu below)? Just did that. Still the same error: You do not hav

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-22 Thread Kiran Chandramohan via Phabricator via cfe-commits
kiranchandramohan added a comment. It could be because you are not the author. Can you commandeer the patch (option available in the Add Action menu below)? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-22 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag added a comment. Hi @kiranchandramohan How do I restart the build? It says I don't have permission. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 ___

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-21 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 353548. AMDChirag added a comment. Rebased to main Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D93373/new/ https://reviews.llvm.org/D93373 Files: clang/test/OpenMP/master_taskloop_simd_ast_print.cpp cla

[PATCH] D93373: [Flang][Openmp] Upgrade TASKGROUP construct to 5.0.

2021-06-21 Thread Chirag Khandelwal via Phabricator via cfe-commits
AMDChirag updated this revision to Diff 353379. AMDChirag added a comment. Herald added a reviewer: sscalpone. Herald added a project: clang. Herald added a subscriber: cfe-commits. Fixes the clang test cases Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm