[llvm-dev] [PATCH] D69853: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h
Doerfert, Johannes via llvm-dev
llvm-dev at lists.llvm.org
Tue Dec 10 15:43:13 PST 2019
I think the problem is that only clang has a dependence on the new
library and when you check LLVM without building clang it does not exist
yet. I will commit a LLVM patch today that will cause LLVM to have a
dependence on the new library as well. That should, I hope, fix the
problem.
On 12/10, Hiroshi Yamauchi wrote:
> This seems to be happening to me, too. I think this happens for me on
> "ninja check-llvm" in a clean build. Let me know if you need more info.
>
> On Tue, Dec 10, 2019 at 2:22 PM Doerfert, Johannes via llvm-dev <
> llvm-dev at lists.llvm.org> wrote:
>
> > Hi Jay,
> >
> > I am not sure why this happens.
> > Can you give me some information so I can reproduce it:
> > Is this happens for a clean build?
> > Is Clang build as well?
> >
> > I think the SYSTEM_LIBS macro doesn't contain the new
> > libLLVMFrontendOpenMP but I don't know why.
> >
> > Thanks,
> > Johannes
> >
> >
> > On 12/10, Jay Foad wrote:
> > > Johannes,
> > >
> > > This patch seems to be causing test failures when I just do "ninja
> > > check", without running "ninja" or "ninja all" first.
> > >
> > > $ CC=clang CXX=clang++ cmake -G Ninja ~/git/llvm-project/llvm
> > > -DLLVM_USE_LINKER=lld -DLLVM_PARALLEL_LINK_JOBS=4
> > > -DCMAKE_BUILD_TYPE=Release -DLLVM_TARGETS_TO_BUILD=X86 && ninja check
> > > [...]
> > > FAIL: LLVM :: tools/llvm-config/system-libs.test (32608 of 34640)
> > > ******************** TEST 'LLVM :: tools/llvm-config/system-libs.test'
> > > FAILED ********************
> > > Script:
> > > --
> > > : 'RUN: at line 1'; /home/jayfoad2/tmp/bin/llvm-config --link-static
> > > --system-libs 2>&1 | /home/jayfoad2/tmp/bin/FileCheck
> > >
> > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/system-libs.test
> > > --
> > > Exit Code: 1
> > >
> > > Command Output (stderr):
> > > --
> > >
> > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/system-libs.test:4:8:
> > > error: CHECK: expected string not found in input
> > > CHECK: -l
> > > ^
> > > <stdin>:1:1: note: scanning from here
> > > llvm-config: error: missing:
> > /home/jayfoad2/tmp/lib/libLLVMFrontendOpenMP.a
> > > ^
> > >
> > > --
> > >
> > > ********************
> > > FAIL: LLVM :: tools/llvm-config/booleans.test (32612 of 34640)
> > > ******************** TEST 'LLVM :: tools/llvm-config/booleans.test'
> > > FAILED ********************
> > > Script:
> > > --
> > > : 'RUN: at line 2'; /home/jayfoad2/tmp/bin/llvm-config
> > > --assertion-mode 2>&1 | /home/jayfoad2/tmp/bin/FileCheck
> > > --check-prefix=CHECK-ONOFF
> > > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/booleans.test
> > > : 'RUN: at line 3'; /home/jayfoad2/tmp/bin/llvm-config
> > > --has-global-isel 2>&1 | /home/jayfoad2/tmp/bin/FileCheck
> > > --check-prefix=CHECK-ONOFF
> > > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/booleans.test
> > > : 'RUN: at line 9'; /home/jayfoad2/tmp/bin/llvm-config --has-rtti
> > > 2>&1 | /home/jayfoad2/tmp/bin/FileCheck --check-prefix=CHECK-YESNO
> > > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/booleans.test
> > > : 'RUN: at line 15'; /home/jayfoad2/tmp/bin/llvm-config --build-mode
> > > 2>&1 | /home/jayfoad2/tmp/bin/FileCheck
> > > --check-prefix=CHECK-BUILD-MODE
> > > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/booleans.test
> > > : 'RUN: at line 20'; /home/jayfoad2/tmp/bin/llvm-config
> > > --build-system 2>&1 | /home/jayfoad2/tmp/bin/FileCheck
> > > --check-prefix=CHECK-BUILD-SYSTEM
> > > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/booleans.test
> > > : 'RUN: at line 25'; /home/jayfoad2/tmp/bin/llvm-config
> > > --shared-mode 2>&1 | /home/jayfoad2/tmp/bin/FileCheck
> > > --check-prefix=CHECK-SHARED-MODE
> > > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/booleans.test
> > > --
> > > Exit Code: 1
> > >
> > > Command Output (stderr):
> > > --
> > >
> > /home/jayfoad2/git/llvm-project/llvm/test/tools/llvm-config/booleans.test:27:24:
> > > error: CHECK-SHARED-MODE-NOT: excluded string found in input
> > > CHECK-SHARED-MODE-NOT: error:
> > > ^
> > > <stdin>:3:14: note: found here
> > > llvm-config: error: missing:
> > /home/jayfoad2/tmp/lib/libLLVMFrontendOpenMP.a
> > > ^~~~~~
> > >
> > > --
> > >
> > > ********************
> > >
> > > Testing Time: 69.42s
> > > ********************
> > > Failing Tests (2):
> > > LLVM :: tools/llvm-config/booleans.test
> > > LLVM :: tools/llvm-config/system-libs.test
> > >
> > > Expected Passes : 19544
> > > Expected Failures : 49
> > > Unsupported Tests : 15045
> > > Unexpected Failures: 2
> > > FAILED: test/CMakeFiles/check-llvm
> > > cd /home/jayfoad2/tmp/test && /usr/bin/python
> > > /home/jayfoad2/tmp/./bin/llvm-lit -sv /home/jayfoad2/tmp/test
> > > ninja: build stopped: subcommand failed.
> > >
> > >
> > >
> > > $ ninja && ninja check
> > > [41/41] Linking CXX executable bin/llvm-stress
> > > [0/1] Running the LLVM regression tests
> > >
> > > Testing Time: 69.02s
> > > Expected Passes : 19547
> > > Expected Failures : 49
> > > Unsupported Tests : 15044
> > >
> > >
> > >
> > > Could you take a look please?
> > >
> > > Thanks,
> > > Jay.
> > >
> > > On Tue, 10 Dec 2019 at 06:19, Johannes Doerfert via Phabricator via
> > > llvm-commits <llvm-commits at lists.llvm.org> wrote:
> > > >
> > > > This revision was automatically updated to reflect the committed
> > changes.
> > > > Closed by commit rGeb3e81f43f01: [OpenMP][NFCI] Introduce
> > llvm/IR/OpenMPConstants.h (authored by jdoerfert).
> > > >
> > > > Changed prior to commit:
> > > > https://reviews.llvm.org/D69853?vs=232987&id=233003#toc
> > > >
> > > > Repository:
> > > > rG LLVM Github Monorepo
> > > >
> > > > CHANGES SINCE LAST ACTION
> > > > https://reviews.llvm.org/D69853/new/
> > > >
> > > > https://reviews.llvm.org/D69853
> > > >
> > > > Files:
> > > > clang/include/clang/AST/OpenMPClause.h
> > > > clang/include/clang/AST/StmtOpenMP.h
> > > > clang/include/clang/Basic/OpenMPKinds.def
> > > > clang/include/clang/Basic/OpenMPKinds.h
> > > > clang/lib/AST/CMakeLists.txt
> > > > clang/lib/AST/OpenMPClause.cpp
> > > > clang/lib/AST/StmtOpenMP.cpp
> > > > clang/lib/Basic/OpenMPKinds.cpp
> > > > clang/lib/CodeGen/CGOpenMPRuntime.cpp
> > > > clang/lib/CodeGen/CGOpenMPRuntimeNVPTX.cpp
> > > > clang/lib/CodeGen/CGStmtOpenMP.cpp
> > > > clang/lib/CodeGen/CodeGenFunction.h
> > > > clang/lib/Parse/CMakeLists.txt
> > > > clang/lib/Parse/ParseOpenMP.cpp
> > > > clang/lib/Sema/CMakeLists.txt
> > > > clang/lib/Sema/SemaOpenMP.cpp
> > > > clang/lib/Sema/SemaTemplateInstantiateDecl.cpp
> > > > clang/lib/Sema/TreeTransform.h
> > > > clang/lib/Serialization/ASTWriter.cpp
> > > > clang/lib/Serialization/ASTWriterStmt.cpp
> > > > llvm/include/llvm/Frontend/OpenMP/OMPConstants.h
> > > > llvm/include/llvm/Frontend/OpenMP/OMPKinds.def
> > > > llvm/lib/CMakeLists.txt
> > > > llvm/lib/Frontend/CMakeLists.txt
> > > > llvm/lib/Frontend/LLVMBuild.txt
> > > > llvm/lib/Frontend/OpenMP/CMakeLists.txt
> > > > llvm/lib/Frontend/OpenMP/LLVMBuild.txt
> > > > llvm/lib/Frontend/OpenMP/OMPConstants.cpp
> > > > llvm/lib/LLVMBuild.txt
> > > >
> > > > _______________________________________________
> > > > llvm-commits mailing list
> > > > llvm-commits at lists.llvm.org
> > > > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
> >
> > --
> >
> > Johannes Doerfert
> > Researcher
> >
> > Argonne National Laboratory
> > Lemont, IL 60439, USA
> >
> > jdoerfert at anl.gov
> > _______________________________________________
> > LLVM Developers mailing list
> > llvm-dev at lists.llvm.org
> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
> >
--
Johannes Doerfert
Researcher
Argonne National Laboratory
Lemont, IL 60439, USA
jdoerfert at anl.gov
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20191210/bdd8aa11/attachment.sig>
More information about the llvm-dev
mailing list