Skip to content

Commit

Permalink
[cmake] Remove unused OUTPUT_MAPPING param to configure_lit_site_cfg(…
Browse files Browse the repository at this point in the history
…). NFC

Looks like it never did anything.
Added in 71deeee
  • Loading branch information
sam-mccall committed Mar 10, 2022
1 parent 4308f03 commit 895653d
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
5 changes: 1 addition & 4 deletions llvm/cmake/modules/AddLLVM.cmake
Expand Up @@ -1614,15 +1614,12 @@ string(CONCAT LLVM_LIT_PATH_FUNCTION
# path back to absolute form. This makes it possible to move a build directory
# containing lit.cfg.py files from one machine to another.
function(configure_lit_site_cfg site_in site_out)
cmake_parse_arguments(ARG "" "" "MAIN_CONFIG;OUTPUT_MAPPING;PATHS" ${ARGN})
cmake_parse_arguments(ARG "" "" "MAIN_CONFIG;PATHS" ${ARGN})

if ("${ARG_MAIN_CONFIG}" STREQUAL "")
get_filename_component(INPUT_DIR ${site_in} DIRECTORY)
set(ARG_MAIN_CONFIG "${INPUT_DIR}/lit.cfg")
endif()
if ("${ARG_OUTPUT_MAPPING}" STREQUAL "")
set(ARG_OUTPUT_MAPPING "${site_out}")
endif()

foreach(c ${LLVM_TARGETS_TO_BUILD})
set(TARGETS_BUILT "${TARGETS_BUILT} ${c}")
Expand Down
2 changes: 0 additions & 2 deletions llvm/utils/lit/CMakeLists.txt
Expand Up @@ -4,8 +4,6 @@
configure_lit_site_cfg(
"${CMAKE_CURRENT_SOURCE_DIR}/tests/lit.site.cfg.in"
"${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg"
OUTPUT_MAPPING
"${CMAKE_CURRENT_BINARY_DIR}/tests/lit.site.cfg"
)

# Lit's test suite creates output files next to the sources which makes the
Expand Down

0 comments on commit 895653d

Please sign in to comment.