[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

saiislam added a comment. In D81098#2123559 , @clementval wrote: > In D81098#2112159 , @ABataev wrote: > > > LG > > > Since this revision landed two tests are failing. > > ` libomp.env::kmp_set_dispatch_buf.c` and

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

clementval added a comment. In D81098#2112159 , @ABataev wrote: > LG Since this revision landed two tests are failing. ` libomp.env::kmp_set_dispatch_buf.c` and `libomp.worksharing/for::kmp_set_dispatch_buf.c`. It was also reported on the mailing

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

ABataev accepted this revision. ABataev added a comment. This revision is now accepted and ready to land. LG Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81098/new/ https://reviews.llvm.org/D81098

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

ABataev added a comment. Looks good. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D81098/new/ https://reviews.llvm.org/D81098 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

ABataev added a comment. In D81098#2085786 , @saiislam wrote: > [Work in progress] Requesting early comments. 28 tests are still failing. Looks good in general Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

ABataev added a comment. Need to update few other places. CompilerInvocation.cpp:3082 and CompilerInvocation.cpp:3085, which sets the default version to 45 in the presense of the target option or simd option. Most probably, need to remove this check. It was used to use 45 as the default

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

saiislam updated this revision to Diff 269239. saiislam added a comment. Fixed 25 out of 75 failing test cases by marking all old default cases as omp4 tests, and new tests as omp5. All these tests are now specific to the version of openmp, so that we don't run into similar problem during next

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

saiislam added a comment. Currently, following 75 lit-tests are failing. 1. Clang :: OpenMP/declare_reduction_codegen.c 2. Clang :: OpenMP/declare_reduction_codegen.cpp 3. Clang :: OpenMP/declare_target_messages.cpp 4. Clang :: OpenMP/declare_variant_implementation_vendor_codegen.cpp 5. Clang ::

[PATCH] D81098: [OpenMP] Upgrade default version of OpenMP to 5.0

saiislam created this revision. saiislam added reviewers: gregrodgers, jdoerfert. Herald added subscribers: cfe-commits, sstefan1, guansong, yaxunl. Herald added a project: clang. When -fopenmp option is specified then version 5.0 will be set as default. Repository: rG LLVM Github Monorepo