From dc209b8a83677910445f36b870dfc26e16404bc9 Mon Sep 17 00:00:00 2001 From: NAKAMURA Takumi Date: Sun, 26 Jan 2014 12:41:38 +0000 Subject: [PATCH] [CMake] tablegen(): Use -I according to the list by include_directories(). For now, local_tds and global_tds are integrated to dependent_tds. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@200150 91177308-0d34-0410-b5e6-96231b3b80d8 --- cmake/modules/TableGen.cmake | 22 +++++++++++++++++----- 1 file changed, 17 insertions(+), 5 deletions(-) diff --git a/cmake/modules/TableGen.cmake b/cmake/modules/TableGen.cmake index b1e15492471..7b69882dee5 100644 --- a/cmake/modules/TableGen.cmake +++ b/cmake/modules/TableGen.cmake @@ -3,8 +3,16 @@ # Adds the name of the generated file to TABLEGEN_OUTPUT. function(tablegen project ofn) - file(GLOB local_tds "*.td") - file(GLOB_RECURSE global_tds "${LLVM_MAIN_INCLUDE_DIR}/llvm/*.td") + # Use the list by include_directories(). + get_property(include_dirs DIRECTORY PROPERTY INCLUDE_DIRECTORIES) + + # Collect possible dependent *.td(s). + # FIXME: It is far from optimal. + file(GLOB dependent_tds "*.td") + foreach(inc ${include_dirs}) + file(GLOB tds "${inc}/*.td") + list(APPEND dependent_tds ${tds}) + endforeach() if (IS_ABSOLUTE ${LLVM_TARGET_DEFINITIONS}) set(LLVM_TARGET_DEFINITIONS_ABSOLUTE ${LLVM_TARGET_DEFINITIONS}) @@ -12,16 +20,20 @@ function(tablegen project ofn) set(LLVM_TARGET_DEFINITIONS_ABSOLUTE ${CMAKE_CURRENT_SOURCE_DIR}/${LLVM_TARGET_DEFINITIONS}) endif() + foreach(inc ${include_dirs}) + list(APPEND TABLEGEN_INCLUDE_DIRECTORIES -I ${inc}) + endforeach() add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${ofn}.tmp # Generate tablegen output in a temporary file. - COMMAND ${${project}_TABLEGEN_EXE} ${ARGN} -I ${CMAKE_CURRENT_SOURCE_DIR} - -I ${LLVM_MAIN_SRC_DIR}/lib/Target -I ${LLVM_MAIN_INCLUDE_DIR} + COMMAND ${${project}_TABLEGEN_EXE} ${ARGN} + -I ${CMAKE_CURRENT_SOURCE_DIR} + ${TABLEGEN_INCLUDE_DIRECTORIES} ${LLVM_TARGET_DEFINITIONS_ABSOLUTE} -o ${CMAKE_CURRENT_BINARY_DIR}/${ofn}.tmp # The file in LLVM_TARGET_DEFINITIONS may be not in the current # directory and local_tds may not contain it, so we must # explicitly list it here: - DEPENDS ${${project}_TABLEGEN_EXE} ${local_tds} ${global_tds} + DEPENDS ${${project}_TABLEGEN_EXE} ${dependent_tds} ${LLVM_TARGET_DEFINITIONS_ABSOLUTE} COMMENT "Building ${ofn}..." ) -- 2.34.1