From 1447c9790972f501dddbfca371a77dc7fd3f8938 Mon Sep 17 00:00:00 2001 From: Serge Guelton Date: Mon, 19 Mar 2018 21:35:30 +0000 Subject: [PATCH] Accept any filepath in llvm_check_source_file_list Cmake function llvm_check_source_file_list currently only accepts paths relative to current CMAKE_SOURCE_DIR or relative to argument SOURCE_DIR. Extend it to accept any path, including absolute ones. Differential revision: https://reviews.llvm.org/D44625 llvm-svn: 327912 --- llvm/cmake/modules/LLVMProcessSources.cmake | 18 ++++++++++++++---- 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/llvm/cmake/modules/LLVMProcessSources.cmake b/llvm/cmake/modules/LLVMProcessSources.cmake index 8b7dc65d9497..f65f31d797cf 100644 --- a/llvm/cmake/modules/LLVMProcessSources.cmake +++ b/llvm/cmake/modules/LLVMProcessSources.cmake @@ -69,14 +69,18 @@ endfunction(llvm_process_sources) function(llvm_check_source_file_list) cmake_parse_arguments(ARG "" "SOURCE_DIR" "" ${ARGN}) - set(listed ${ARG_UNPARSED_ARGUMENTS}) + foreach(l ${ARG_UNPARSED_ARGUMENTS}) + get_filename_component(fp ${l} REALPATH) + list(APPEND listed ${fp}) + endforeach() + if(ARG_SOURCE_DIR) file(GLOB globbed - RELATIVE "${CMAKE_CURRENT_LIST_DIR}" "${ARG_SOURCE_DIR}/*.c" "${ARG_SOURCE_DIR}/*.cpp") else() file(GLOB globbed *.c *.cpp) endif() + foreach(g ${globbed}) get_filename_component(fn ${g} NAME) if(ARG_SOURCE_DIR) @@ -84,15 +88,21 @@ function(llvm_check_source_file_list) else() set(entry "${fn}") endif() + get_filename_component(gp ${g} REALPATH) # Don't reject hidden files. Some editors create backups in the # same directory as the file. if (NOT "${fn}" MATCHES "^\\.") list(FIND LLVM_OPTIONAL_SOURCES ${entry} idx) if( idx LESS 0 ) - list(FIND listed ${entry} idx) + list(FIND listed ${gp} idx) if( idx LESS 0 ) - message(SEND_ERROR "Found unknown source file ${g} + if(ARG_SOURCE_DIR) + set(fn_relative "${ARG_SOURCE_DIR}/${fn}") + else() + set(fn_relative "${fn}") + endif() + message(SEND_ERROR "Found unknown source file ${fn_relative} Please update ${CMAKE_CURRENT_LIST_FILE}\n") endif() endif()