Skip to content

Commit

Permalink
Fix standalone clang builds after fb80b6b.
Browse files Browse the repository at this point in the history
When clang is built against a prebuilt LLVM, LLVM_SOURCE_DIR is
empty, which due to a cmake quirk caused list lengths to get out
of sync. Add a workaround.
  • Loading branch information
nico committed Apr 3, 2020
1 parent 9d9b8a2 commit 18a18b2
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions llvm/cmake/modules/AddLLVM.cmake
Expand Up @@ -1484,9 +1484,14 @@ function(configure_lit_site_cfg site_in site_out)

if (ARG_PATHS)
# Walk ARG_PATHS and collect the current value of the variables in there.
# list(APPEND) ignores empty elements exactly if the list is empty,
# so start the list with a dummy element and drop it, to make sure that
# even empty values make it into the values list.
set(ARG_PATH_VALUES "dummy")
foreach(path ${ARG_PATHS})
list(APPEND ARG_PATH_VALUES "${${path}}")
endforeach()
list(REMOVE_AT ARG_PATH_VALUES 0)

# Compute paths relative to the directory containing output lit.site.cfg.py.
# Passing ARG_PATH_VALUES as-is to execute_process() makes cmake strip
Expand Down

0 comments on commit 18a18b2

Please sign in to comment.