[llvm-dev] [PATCH] D69853: [OpenMP][NFCI] Introduce llvm/IR/OpenMPConstants.h
Doerfert, Johannes via llvm-dev
llvm-dev at lists.llvm.org
Fri Dec 13 09:05:20 PST 2019
Yay!
Thanks for reporting this and your patience!
On 12/13, Jay Foad wrote:
> Yes, it works for me. Thank you!
>
> Jay.
>
> On Wed, 11 Dec 2019 at 22:09, Doerfert, Johannes <jdoerfert at anl.gov> wrote:
> >
> > Jay, Hiroshi, did [0] resolve the problem you've seen.
> >
> > [0] https://github.com/llvm/llvm-project/commit/d23c61490c282a7a8f29aaa5c021cbfdaf87fb6f
> >
> > On 12/10, Doerfert, Johannes wrote:
> > > 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
> >
> >
> >
> > --
> >
> > Johannes Doerfert
> > Researcher
> >
> > Argonne National Laboratory
> > Lemont, IL 60439, USA
> >
> > jdoerfert at anl.gov
--
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/20191213/1d33df7a/attachment.sig>
More information about the llvm-dev
mailing list