Differential D15895 Diff 42857 cmake/modules/FindClang.cmake

Changeset View

Standalone View

View Options

cmake/modules/FindClang.cmake

Context not available.
4# CLANG_FOUND - True if Clang was found 4​# CLANG_FOUND - True if Clang was found
5# CLANG_INCLUDE_DIRS - Where to find Clang includes 5​# CLANG_INCLUDE_DIRS - Where to find Clang includes
6# CLANG_LIBRARY_DIRS - Where to find Clang libraries 6​# CLANG_LIBRARY_DIRS - Where to find Clang libraries
7​# CLANG_BUILTIN_DIR - Where to find Clang builtin includes
7# 8​#
8# CLANG_CLANG_LIB - Libclang C library 9​# CLANG_CLANG_LIB - Libclang C library
9# 10​#
Context not available.
92 set(CLANG_INCLUDE_DIRS ${LLVM_INCLUDE_DIRS}) 93​ set(CLANG_INCLUDE_DIRS ${LLVM_INCLUDE_DIRS})
kfunkUnsubmitted
Not Done

IIUC then CLANG_INCLUDE_DIRS shouldn't be touched. CLANG_INCLUDE_DIRS is for finding the correct include dirs for building KDevClangSupport against libclang -- we don't need to have things like compiler builtin includes in there.

Rather, leave CLANG_INCLUDE_DIRS as it is and do find_path(CLANG_BUILTIN_DIR ...), in other words: create another variable.

@buschinski Just to double-check: /usr/lib64/llvm/7/include has a clang subfolder on Gentoo, right?

kfunk: IIUC then `CLANG_INCLUDE_DIRS` shouldn't be touched. `CLANG_INCLUDE_DIRS` is for finding the…
buschinskiAuthorUnsubmitted
Not Done

Yes, there is a /usr/lib64/llvm/7/include/clang, but it does not contain cpuid.h or such headers.

And just to be clear, the original version looked for
"/usr/lib64/llvm/7/lib64/clang/7.0.0/include", which does not exist.

$ llvm-config --libdir
/usr/lib64/llvm/7/lib64
the double "lib64" sounds wrong, but it is correct, exists and does also contains lots of libclang*.

I will change it to CLANG_BUILTIN_DIR and revert the CLANG_INCLUDE_DIRS changes.

buschinski: Yes, there is a /usr/lib64/llvm/7/include/clang, but it does not contain cpuid.h or such…
93 set(CLANG_VERSION ${LLVM_VERSION}) 94​ set(CLANG_VERSION ${LLVM_VERSION})
94 95​
kossebauUnsubmitted
Not Done

I would like some comment why "cpuid.h" and some cross-linking between ClangSupport constructor and this.
Unless cpuid.h is a header well known for such compiler/system includes testing, and I yet have to learn much more :)

kossebau: I would like some comment why "cpuid.h" and some cross-linking between ClangSupport constructor…
buschinskiAuthorUnsubmitted
Not Done

"cpuid.h" is common (as far as I know) for compiler (gcc/clang), but not very clang specific. I will add a comment regarding ClangSupport constructor

buschinski: "cpuid.h" is common (as far as I know) for compiler (gcc/clang), but not very clang specific. I…
96​ find_path(CLANG_BUILTIN_DIR
97​ # cpuid.h because it is defined in ClangSupport constructor as valid clang builtin dir indicator
98​ NAMES "cpuid.h"
99​ PATHS "${CLANG_LIBRARY_DIRS}"
100​ "${CLANG_INCLUDE_DIRS}"
101​ PATH_SUFFIXES "clang/${CLANG_VERSION}/include"
102​ "../../../clang/${CLANG_VERSION}/include"
103​ )
kfunkUnsubmitted
Not Done

Needs NO_DEFAULT_PATH otherwise include dirs from e.g. /usr/include may take precedence.

This is important if you (for instance) have a self-built version of LLVM/Clang in $HOME, and a distro-provided version in /usr, with the exact same version.

Your patch as-is returns:

/usr/include/clang/6.0.1/include

... while it should return this for me:

/home/kfunk/devel/build/llvm/lib/clang/6.0.1/include

Adding NO_DEFAULT_PATH fixes the issue for me.

kfunk: Needs `NO_DEFAULT_PATH` otherwise include dirs from e.g. /usr/include may take precedence.
104​
kfunkUnsubmitted
Not Done

Typo: "Unabled"

kfunk: Typo: "Unabled"
buschinskiAuthorUnsubmitted
Not Done

rephrased to match other error messages to:
"Could not find Clang builtin directory"

buschinski: rephrased to match other error messages to: "Could not find Clang builtin directory"
105​ if (NOT CLANG_BUILTIN_DIR)
106​ message(FATAL_ERROR "Could not find Clang builtin directory")
107​ endif()
108​
95 # check whether llvm-config comes from an install prefix 109​ # check whether llvm-config comes from an install prefix
96 execute_process( 110​ execute_process(
97 COMMAND ${LLVM_CONFIG_EXECUTABLE} --src-root 111​ COMMAND ${LLVM_CONFIG_EXECUTABLE} --src-root
Context not available.
108 endif() 122​ endif()
109 123​
110 message(STATUS "Found Clang (LLVM version: ${CLANG_VERSION})") 124​ message(STATUS "Found Clang (LLVM version: ${CLANG_VERSION})")
111 message(STATUS " Include dirs: ${CLANG_INCLUDE_DIRS}") 125​ message(STATUS " Include dirs: ${CLANG_INCLUDE_DIRS}")
112 message(STATUS " Clang libraries: ${CLANG_LIBS}") 126​ message(STATUS " Clang libraries: ${CLANG_LIBS}")
113 message(STATUS " Libclang C library: ${CLANG_CLANG_LIB}") 127​ message(STATUS " Libclang C library: ${CLANG_CLANG_LIB}")
128​ message(STATUS " Builtin include dir: ${CLANG_BUILTIN_DIR}")
114else() 129​else()
115 if(Clang_FIND_REQUIRED) 130​ if(Clang_FIND_REQUIRED)
116 message(FATAL_ERROR "Could NOT find Clang") 131​ message(FATAL_ERROR "Could NOT find Clang")
Context not available.