[PATCH] D129635: [OpenMP] Update the default version of OpenMP to 5.1

animeshk-amd added a comment. In D129635#4442580 , @h-vetinari wrote: > In D129635#4440613 , @animeshk-amd > wrote: > >> In the multi-company community meeting, the agreement was to move to the 5.1 >> version a

[PATCH] D129635: [OpenMP] Update the default version of OpenMP to 5.1

animeshk-amd added a comment. In D129635#4438970 , @h-vetinari wrote: > Does https://clang.llvm.org/docs/OpenMPSupport.html need an update? It still > says "Clang fully supports OpenMP 4.5" (with many 5.0/5.1 features marked as > "worked on" / "unclaim

[PATCH] D129635: [OpenMP] Update the default version of OpenMP to 5.1

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG0c6f2f629cc0: [OpenMP] Update the default version of OpenMP to 5.1 (authored by animeshk-amd). Repository: rG LLVM Github Monorepo CHANGES SINCE

[PATCH] D129635: [OpenMP] Update the default version of OpenMP to 5.1

animeshk-amd updated this revision to Diff 531189. animeshk-amd added a comment. Herald added subscribers: jplehr, sunshaoce, pcwang-thead. Rebase and fixing the merge conflicts Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129635/new/ https://revi

[PATCH] D144544: [OpenMP] Improve LIT tests on composite target constructs

animeshk-amd added a comment. In D144544#4144563 , @jdoerfert wrote: > I guess this is fine. We should try to run V&V tests, this is really just > checking if we generate (some) code. Thanks for accepting. Right. By running V&V tests, do you mean to su

[PATCH] D144544: [OpenMP] Improve LIT tests on composite target constructs

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG265ea1c7459d: [OpenMP] Improve LIT tests on composite target constructs (authored by animeshk-amd). Repository: rG LLVM Github Monorepo CHANGES S

[PATCH] D144544: [OpenMP] Improve LIT tests on composite target constructs

animeshk-amd updated this revision to Diff 499513. animeshk-amd added a comment. Removed a pragma line which was not required. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D144544/new/ https://reviews.llvm.org/D144544 Files: clang/test/OpenMP/ta

[PATCH] D144544: [OpenMP] Improve LIT tests on composite target constructs

animeshk-amd created this revision. animeshk-amd added reviewers: saiislam, JonChesterfield. Herald added subscribers: guansong, yaxunl. Herald added a project: All. animeshk-amd requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1.

[PATCH] D134974: [OpenMP] Add map clause to the LIT test on use_device_addr clause

This revision was automatically updated to reflect the committed changes. Closed by commit rG0f8e7b43291c: [OpenMP] Add map clause to the LIT test on use_device_addr clause (authored by animeshk-amd). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134

[PATCH] D134974: [OpenMP] Add map clause to the LIT test on use_device_addr clause

animeshk-amd updated this revision to Diff 473193. animeshk-amd added a comment. The main branch dropped the use of flag `-no-opaque-pointers` in this test which generated a different IR than before. This revision gets rid of the conflicts with the main branch caused by above. Repository: rG

[PATCH] D134974: [OpenMP] Add map clause to the LIT test on use_device_addr clause

animeshk-amd added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D134974/new/ https://reviews.llvm.org/D134974 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.or

[PATCH] D132855: [OpenMP] Extend the lit test for uses_allocators in target region

This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG06da9b94ae37: [OpenMP] Extend the lit test for uses_allocators in target region (authored by animeshk-amd). Repository: rG LLVM Github Monorepo C

[PATCH] D132855: [OpenMP] Extend the lit test for uses_allocators in target region

animeshk-amd updated this revision to Diff 467757. animeshk-amd added a comment. Another patch that landed after this patch introduced merge conflicts in the target_map_codegen_10.cpp with regards to the use of the `-no-opaque-pointers`. Those merge conflicts are fixed in the revision. Reposit

[PATCH] D132855: [OpenMP] Extend the lit test for uses_allocators in target region

animeshk-amd added inline comments. Comment at: clang/test/OpenMP/target_map_codegen_10.cpp:19 +// RUN: %clang_cc1 -no-opaque-pointers -DCK11 -fopenmp -fopenmp-targets=i386-pc-linux-gnu -x c++ -std=c++11 -triple i386-unknown-unknown -emit-pch -o %t %s +// RUN: %clang_cc1 -no-op

[PATCH] D132855: [OpenMP] Extend the lit test for uses_allocators in target region

animeshk-amd updated this revision to Diff 464932. animeshk-amd added a comment. Rebased the patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132855/new/ https://reviews.llvm.org/D132855 Files: clang/test/OpenMP/target_map_codegen_10.cpp c

[PATCH] D134974: [OpenMP] Add map clause to the LIT test on use_device_addr clause

animeshk-amd created this revision. animeshk-amd added reviewers: saiislam, JonChesterfield. Herald added subscribers: guansong, yaxunl. Herald added a project: All. animeshk-amd requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1.

[PATCH] D132855: [OpenMP] Extend the lit test for uses_allocators in target region

animeshk-amd updated this revision to Diff 461827. animeshk-amd added a comment. Updated the LIT test for omp_thread_mem_alloc allocator Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132855/new/ https://reviews.llvm.org/D132855 Files: clang/test

[PATCH] D132643: [OpenMP] Extend lit test for parallel for simd construct

This revision was automatically updated to reflect the committed changes. Closed by commit rG7f57b646d150: [OpenMP] Extend lit test for parallel for simd construct (authored by animeshk-amd). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132643/new/

[PATCH] D132643: [OpenMP] Extend lit test for parallel for simd construct

animeshk-amd added a comment. ping Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132643/new/ https://reviews.llvm.org/D132643 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.or

[PATCH] D131763: [OpenMP] Add lit test for metadirective device arch inspired from sollve

animeshk-amd added a comment. Thanks @saiislam. Since I don't have the commit access yet, I request you to kindly commit this on my behalf. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D131763/new/ https://reviews.llvm.org/D131763 ___

[PATCH] D132855: [OpenMP] Extend the lit test for uses_allocators in target region

animeshk-amd updated this revision to Diff 457219. animeshk-amd added a comment. Extended the LIT test for `defaultmap` clause in the target region with respect to the default OpenMP version. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D132855/new/

[PATCH] D132855: [OpenMP] Extend the lit test for uses_allocators in target region

animeshk-amd created this revision. animeshk-amd added reviewers: saiislam, JonChesterfield. Herald added subscribers: guansong, yaxunl. Herald added a project: All. animeshk-amd requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1.

[PATCH] D132643: [OpenMP] Extend lit test for parallel for simd construct

animeshk-amd created this revision. animeshk-amd added reviewers: saiislam, JonChesterfield. Herald added subscribers: guansong, yaxunl. Herald added a project: All. animeshk-amd requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1.

[PATCH] D131763: [OpenMP] Add lit test for metadirective device arch inspired from sollve

animeshk-amd updated this revision to Diff 455161. animeshk-amd added a comment. The second function was testing the `implementation` trait set for the `vendor` trait with respect to the `metadirective` directive. After Johannes's comment, on further searching it turns out that this trait and tra

[PATCH] D131763: [OpenMP] Add lit test for metadirective device arch inspired from sollve

animeshk-amd created this revision. animeshk-amd added reviewers: saiislam, JonChesterfield. Herald added subscribers: guansong, yaxunl. Herald added a project: All. animeshk-amd requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1.

[PATCH] D129635: [OpenMP] Update the default version of OpenMP to 5.1

animeshk-amd updated this revision to Diff 447998. animeshk-amd edited the summary of this revision. animeshk-amd added a comment. Herald added subscribers: openmp-commits, steakhal, arphaman. Herald added a project: OpenMP. [OpenMP] Update more LIT tests to support OpenMP version upgrade Some mo

[PATCH] D129635: [OpenMP] Update the default version of OpenMP to 5.1

animeshk-amd updated this revision to Diff 446359. animeshk-amd added a comment. Herald added a subscriber: zzheng. [OpenMP] Update remaining tests for OpenMP version update to 5.1 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D129635/new/ https://r

[PATCH] D129635: [OpenMP] Update the default version of OpenMP to 5.1

animeshk-amd created this revision. animeshk-amd added reviewers: saiislam, JonChesterfield. Herald added subscribers: guansong, yaxunl. Herald added a project: All. animeshk-amd requested review of this revision. Herald added a reviewer: jdoerfert. Herald added subscribers: cfe-commits, sstefan1.