llvmorg-github-actions[bot] wrote:
<!--LLVM PR SUMMARY COMMENT--> @llvm/pr-subscribers-clang Author: cmtice <details> <summary>Changes</summary> The test OMPInvariantPredicateBoundOnIntraTileLoop, in AttrTest.cpp uses '-fopenmp' without supplying an explcit value. In some places the default value is 'libomp', which works just fine for this test. But in envorinments where 'libomp' is not the default value (e.g. places that use 'libgomp' by default) the test fails. This fixes that issue by explicitly telling the test to use 'libomp'. --- Full diff: https://github.com/llvm/llvm-project/pull/221098.diff 1 Files Affected: - (modified) clang/unittests/AST/AttrTest.cpp (+1-1) ``````````diff diff --git a/clang/unittests/AST/AttrTest.cpp b/clang/unittests/AST/AttrTest.cpp index 56096054dd267..33cc5c491db93 100644 --- a/clang/unittests/AST/AttrTest.cpp +++ b/clang/unittests/AST/AttrTest.cpp @@ -315,7 +315,7 @@ TEST(Attr, OMPInvariantPredicateBoundOnIntraTileLoop) { body(i); } )cpp", - {"-fopenmp"}); + {"-fopenmp=libomp"}); ASSERT_TRUE(AST); auto GetHint = [&AST](const char *FuncName) { `````````` </details> https://github.com/llvm/llvm-project/pull/221098 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
