[PATCH] D59609: [clang][OpenMP] Fix build when using libgomp

Jordan Rupprecht via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Mar 20 13:21:50 PDT 2019


rupprecht created this revision.
rupprecht added a reviewer: lebedev.ri.
Herald added subscribers: cfe-commits, jdoerfert, guansong.
Herald added a project: clang.

rL356570 <https://reviews.llvm.org/rL356570> introduced a test which only passes with the default openmp library, libomp, and fails with other openmp libraries, such as libgomp. Explicitly choose libomp.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D59609

Files:
  clang/unittests/AST/OMPStructuredBlockTest.cpp


Index: clang/unittests/AST/OMPStructuredBlockTest.cpp
===================================================================
--- clang/unittests/AST/OMPStructuredBlockTest.cpp
+++ clang/unittests/AST/OMPStructuredBlockTest.cpp
@@ -55,7 +55,7 @@
                       StringRef ExpectedPrinted,
                       PolicyAdjusterType PolicyAdjuster = None) {
   std::vector<std::string> Args = {
-      "-fopenmp",
+      "-fopenmp=libomp",
   };
   return PrintedStmtMatches(Code, Args, NodeMatch, ExpectedPrinted,
                             PolicyAdjuster);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D59609.191567.patch
Type: text/x-patch
Size: 562 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190320/02fbba9d/attachment.bin>


More information about the cfe-commits mailing list