[PATCH] D91468: [llvm] Add OpenACC as a dependency to llvm-config

[Leonard Chan via Phabricator via llvm-commits]

leonardchan created this revision.
leonardchan added reviewers: phosek, mehdi_amini, serge-sans-paille, Meinersbur.
leonardchan added a project: LLVM.
Herald added a subscriber: mgorny.
leonardchan requested review of this revision.

As of D90848 <https://reviews.llvm.org/D90848>, running `ninja check-llvm` after a clean cmake configuration will cause these tests to fail:

  Failed Tests (2):
    LLVM :: tools/llvm-config/booleans.test
    LLVM :: tools/llvm-config/system-libs.test

from missing `lib/libLLVMFrontendOpenACC.a`. See https://luci-milo.appspot.com/p/fuchsia/builders/ci/clang-linux-x64/b8863755892480463712?.

This just adds LLVMFrontendOpenACC as a dependency to llvm-config so it we be built when building llvm-config.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D91468

Files:
  llvm/tools/llvm-config/CMakeLists.txt


Index: llvm/tools/llvm-config/CMakeLists.txt
===================================================================
--- llvm/tools/llvm-config/CMakeLists.txt
+++ llvm/tools/llvm-config/CMakeLists.txt
@@ -76,6 +76,8 @@
 # Add the dependency on the generation step.
 add_file_dependencies(${CMAKE_CURRENT_SOURCE_DIR}/llvm-config.cpp ${BUILDVARIABLES_OBJPATH})
 
+add_dependencies(llvm-config LLVMFrontendOpenACC)
+
 if(CMAKE_CROSSCOMPILING AND NOT LLVM_CONFIG_PATH)
   build_native_tool(llvm-config LLVM_CONFIG_PATH)
   set(LLVM_CONFIG_PATH "${LLVM_CONFIG_PATH}" CACHE STRING "")


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D91468.305280.patch
Type: text/x-patch
Size: 575 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201114/fbeb018a/attachment.bin>