Skip to content

Commit

Permalink
Use correct ittapi source directory
Browse files Browse the repository at this point in the history
ITTAPI_SOURCE_DIR is what IntelJITProfiling uses, so I presume this is what should be used here. This fixes building for me as I set `ITTAPI_SOURCE_DIR`.

Fix if ITTAPI_SOURCE_DIR is not set

Fix IntelJITEvents ase well
  • Loading branch information
Zentrik committed Oct 9, 2024
1 parent edb7a55 commit f774200
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 2 deletions.
5 changes: 4 additions & 1 deletion llvm/lib/ExecutionEngine/IntelJITEvents/CMakeLists.txt
Original file line number Diff line number Diff line change
@@ -1,5 +1,8 @@
include_directories( ${CMAKE_CURRENT_SOURCE_DIR}/.. )
include_directories( ${PROJECT_BINARY_DIR}/ittapi/include/ )
if(NOT DEFINED ITTAPI_SOURCE_DIR)
set(ITTAPI_SOURCE_DIR ${PROJECT_BINARY_DIR})
endif()
include_directories( ${ITTAPI_SOURCE_DIR}/ittapi/include/ )
add_llvm_component_library(LLVMIntelJITEvents
IntelJITEventListener.cpp

Expand Down
5 changes: 4 additions & 1 deletion llvm/lib/ExecutionEngine/Orc/TargetProcess/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,10 @@ set(intel_jit_profiling )
if( LLVM_USE_INTEL_JITEVENTS )
set(intel_jit_profiling IntelJITProfiling)
include_directories(${CMAKE_CURRENT_SOURCE_DIR}/../../IntelJITProfiling)
include_directories(${PROJECT_BINARY_DIR}/ittapi/include/ )
if(NOT DEFINED ITTAPI_SOURCE_DIR)
set(ITTAPI_SOURCE_DIR ${PROJECT_BINARY_DIR})
endif()
include_directories(${ITTAPI_SOURCE_DIR}/ittapi/include/ )
endif()

add_llvm_component_library(LLVMOrcTargetProcess
Expand Down

0 comments on commit f774200

Please sign in to comment.