saiislam marked an inline comment as done. saiislam added inline comments.
================ Comment at: clang/test/OpenMP/target_ast_print.cpp:5-11 +// RUN: %clang_cc1 -DOMP45 -verify -fopenmp -ast-print %s | FileCheck %s --check-prefix=OMP45 +// RUN: %clang_cc1 -DOMP45 -fopenmp -x c++ -std=c++11 -emit-pch -o %t %s +// RUN: %clang_cc1 -DOMP45 -fopenmp -std=c++11 -include-pch %t -fsyntax-only -verify %s -ast-print | FileCheck %s --check-prefix=OMP45 -// RUN: %clang_cc1 -DOMP45 -verify -fopenmp-simd -fopenmp-version=45 -ast-print %s | FileCheck %s --check-prefix=OMP45 -// RUN: %clang_cc1 -DOMP45 -fopenmp-simd -fopenmp-version=45 -x c++ -std=c++11 -emit-pch -o %t %s -// RUN: %clang_cc1 -DOMP45 -fopenmp-simd -fopenmp-version=45 -std=c++11 -include-pch %t -fsyntax-only -verify %s -ast-print | FileCheck %s --check-prefix=OMP45 +// RUN: %clang_cc1 -DOMP45 -verify -fopenmp-simd -ast-print %s | FileCheck %s --check-prefix=OMP45 +// RUN: %clang_cc1 -DOMP45 -fopenmp-simd -x c++ -std=c++11 -emit-pch -o %t %s +// RUN: %clang_cc1 -DOMP45 -fopenmp-simd -std=c++11 -include-pch %t -fsyntax-only -verify %s -ast-print | FileCheck %s --check-prefix=OMP45 ---------------- ABataev wrote: > Better to remove -fopenmp-version from the tests for OpenMP 5.0 Yes, that's the plan. D82575 is supposed to remove -fopenmp-version=50 from everywhere. And this patch is supposed to ensure that all tests are testing at least for the default version. Plan is to keep version strings only at places where there is specific divergent behaviors between different versions. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83863/new/ https://reviews.llvm.org/D83863 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
