Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

llvm[openmp] build failure #12377

Closed
tan-wei opened this issue Jul 11, 2020 · 5 comments · Fixed by #13998
Closed

llvm[openmp] build failure #12377

tan-wei opened this issue Jul 11, 2020 · 5 comments · Fixed by #13998
Assignees
Labels
category:port-bug The issue is with a library, which is something the port should already support

Comments

@tan-wei
Copy link

tan-wei commented Jul 11, 2020

Host Environment

  • OS: Windows
  • Compiler: MSVC 16.6.3

To Reproduce
Steps to reproduce the behavior:
./vcpkg install llvm[openmp]

Failure logs

>_ vcpkg install llvm[openmp]
Computing installation plan...
The following packages will be built and installed:
    llvm[clang,core,disable-abi-breaking-checks,disable-assertions,disable-clang-static-analyzer,enable-rtti,lld,openmp,tools]:x86-windows
Starting package 1/1: llvm:x86-windows
Building package llvm[clang,core,disable-abi-breaking-checks,disable-assertions,disable-clang-static-analyzer,enable-rtti,lld,openmp,tools]:x86-windows...
-- Note: llvm only supports static library linkage. Building static library.
-- Using cached D:/vcpkg/downloads/llvm-llvm-project-llvmorg-10.0.0.tar.gz
-- Extracting source D:/vcpkg/downloads/llvm-llvm-project-llvmorg-10.0.0.tar.gz
-- Applying patch 0001-allow-to-use-commas.patch
-- Applying patch 0002-fix-install-paths.patch
-- Applying patch 0003-fix-vs2019-v16.6.patch
-- Using source at D:/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c.clean
-- Configuring x86-windows
CMake Error at scripts/cmake/vcpkg_execute_required_process.cmake:72 (message):
    Command failed: ninja -v
    Working Directory: D:/vcpkg/buildtrees/llvm/x86-windows-rel/vcpkg-parallel-configure
    Error code: 1
    See logs for more information:
      D:\vcpkg\buildtrees\llvm\config-x86-windows-out.log

Call Stack (most recent call first):
  scripts/cmake/vcpkg_configure_cmake.cmake:300 (vcpkg_execute_required_process)
  ports/llvm/portfile.cmake:168 (vcpkg_configure_cmake)
  scripts/ports.cmake:76 (include)


Error: Building package llvm:x86-windows failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with `.\vcpkg update`, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
  Package: llvm:x86-windows
  Vcpkg version: 2020.06.15-nohash

Additionally, attach any relevant sections from the log files above.

config-x86-windows-out.log
install-x86-windows-dbg-err.log

@PhoebeHui PhoebeHui added the category:port-bug The issue is with a library, which is something the port should already support label Jul 15, 2020
@PhoebeHui
Copy link
Contributor

PhoebeHui commented Jul 15, 2020

Confirmed, this issue could repro, check the history, that feature added by #10295, and it has never worked with feature.

Failures:
CMake Error at E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/llvm/cmake/modules/AddLLVM.cmake:1566 (add_dependencies):
The dependency target "FileCheck" of target "check-openmp" does not exist.
Call Stack (most recent call first):
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/llvm/cmake/modules/AddLLVM.cmake:1587 (add_lit_target)
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/openmp/cmake/OpenMPTesting.cmake:183 (add_lit_testsuite)
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/openmp/cmake/OpenMPTesting.cmake:198 (add_openmp_testsuite)
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/openmp/CMakeLists.txt:95 (construct_check_openmp_target)

CMake Error at E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/llvm/cmake/modules/AddLLVM.cmake:1566 (add_dependencies):
The dependency target "FileCheck" of target "check-libomp" does not exist.
Call Stack (most recent call first):
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/llvm/cmake/modules/AddLLVM.cmake:1587 (add_lit_target)
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/openmp/cmake/OpenMPTesting.cmake:183 (add_lit_testsuite)
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/openmp/runtime/test/CMakeLists.txt:33 (add_openmp_testsuite)

CMake Error at E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/llvm/cmake/modules/AddLLVM.cmake:1566 (add_dependencies):
The dependency target "FileCheck" of target "check-ompt" does not exist.
Call Stack (most recent call first):
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/llvm/cmake/modules/AddLLVM.cmake:1587 (add_lit_target)
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/openmp/cmake/OpenMPTesting.cmake:183 (add_lit_testsuite)
E:/VCPKG/boost_173/vcpkg/buildtrees/llvm/src/org-10.0.0-32c002c70c/openmp/runtime/test/CMakeLists.txt:36 (add_openmp_testsuite)

@PhoebeHui
Copy link
Contributor

Looks same issue with https://bugs.llvm.org/show_bug.cgi?id=40557#c2

@tan-wei
Copy link
Author

tan-wei commented Jul 15, 2020

Thanks very much. Does it mean we should wait for llvm team fixing CMake module files?

@yurybura
Copy link
Contributor

@PhoebeHui @tan-wei Let me check this issue. I think OpenMP doesn't support build without LLVM test tools (lit, FileCheck).

@PhoebeHui
Copy link
Contributor

@yurybura, thank you as always!

After some investigation, found it requires feature 'utils', which setting 'LLVM_INCLUDE_UTILS' to 'ON', then it configured successfully.

However, it start to build failed with 'ninja: error: 'lib/libomp.dll.lib', needed by 'lib/libomp.libd', missing and no known rule to make it'

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants