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

NAKAMURA Takumi geek4civic at gmail.com
Wed Dec 5 02:15:41 PST 2012


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>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D171.1.patch
Type: text/x-patch
Size: 1689 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20121205/f3d93811/attachment.bin>


More information about the llvm-commits mailing list