[llvm-commits] [PATCH] JITEventListener.h: Use llvm-config.h instead of config.h.

[NAKAMURA Takumi]

Oh excuse me, I mistook old address to Eli...

I suppose Andy might be responsible in this area.

...Takumi

2012/12/5 NAKAMURA Takumi <geek4civic at gmail.com>:
> Andy and Eli, could you confirm this in your environments, please?
>
> ...Takumi
>
> 2012/12/5 NAKAMURA Takumi <geek4civic at gmail.com>:
>> A couple of public headers use config.h. One of them is JITEventListener.h.
>>
>> http://llvm.org/bugs/show_bug.cgi?id=7809
>>
>> As discussed in PR7809, we should not include config.h in public headers.
>>
>> ps. The last one is FEnv.h.
>>
>> http://llvm-reviews.chandlerc.com/D171
>>
>> Files:
>>   llvm/include/llvm/Config/llvm-config.h.cmake
>>   llvm/include/llvm/Config/llvm-config.h.in
>>   llvm/include/llvm/ExecutionEngine/JITEventListener.h
>>
>> Index: llvm/include/llvm/Config/llvm-config.h.cmake
>> ===================================================================
>> --- llvm/include/llvm/Config/llvm-config.h.cmake
>> +++ llvm/include/llvm/Config/llvm-config.h.cmake
>> @@ -112,6 +112,12 @@
>>  /* Installation prefix directory */
>>  #cmakedefine LLVM_PREFIX "${LLVM_PREFIX}"
>>
>> +/* Define if we have the Intel JIT API runtime support library */
>> +#cmakedefine LLVM_USE_INTEL_JITEVENTS 1
>> +
>> +/* Define if we have the oprofile JIT-support library */
>> +#cmakedefine LLVM_USE_OPROFILE 1
>> +
>>  /* Major version of the LLVM API */
>>  #cmakedefine LLVM_VERSION_MAJOR ${LLVM_VERSION_MAJOR}
>>
>> Index: llvm/include/llvm/Config/llvm-config.h.in
>> ===================================================================
>> --- llvm/include/llvm/Config/llvm-config.h.in
>> +++ llvm/include/llvm/Config/llvm-config.h.in
>> @@ -112,6 +112,12 @@
>>  /* Installation prefix directory */
>>  #undef LLVM_PREFIX
>>
>> +/* Define if we have the Intel JIT API runtime support library */
>> +#undef LLVM_USE_INTEL_JITEVENTS
>> +
>> +/* Define if we have the oprofile JIT-support library */
>> +#undef LLVM_USE_OPROFILE
>> +
>>  /* Major version of the LLVM API */
>>  #undef LLVM_VERSION_MAJOR
>>
>> Index: llvm/include/llvm/ExecutionEngine/JITEventListener.h
>> ===================================================================
>> --- llvm/include/llvm/ExecutionEngine/JITEventListener.h
>> +++ llvm/include/llvm/ExecutionEngine/JITEventListener.h
>> @@ -15,7 +15,7 @@
>>  #ifndef LLVM_EXECUTION_ENGINE_JIT_EVENTLISTENER_H
>>  #define LLVM_EXECUTION_ENGINE_JIT_EVENTLISTENER_H
>>
>> -#include "llvm/Config/config.h"
>> +#include "llvm/Config/llvm-config.h"
>>  #include "llvm/Support/DataTypes.h"
>>  #include "llvm/Support/DebugLoc.h"
>>  #include <vector>