Merge pull request #6643 from lioncash/oprofile

CMakeLists: Define an OProfile target and use it
dolphin-emu · Apr 14, 2018 · aafa8a1 · aafa8a1
2 parents e253d3d + b312815
commit aafa8a1
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 4 deletions.
diff --git a/CMake/FindOProfile.cmake b/CMake/FindOProfile.cmake
@@ -7,13 +7,25 @@
 find_path(OPROFILE_INCLUDE_DIR opagent.h)
 
 find_library(OPROFILE_LIBRARY opagent
-	PATH_SUFFIXES oprofile)
+  PATH_SUFFIXES oprofile
+)
 
 set(OPROFILE_INCLUDE_DIRS ${OPROFILE_INCLUDE_DIR})
 set(OPROFILE_LIBRARIES ${OPROFILE_LIBRARY})
 
 include(FindPackageHandleStandardArgs)
 find_package_handle_standard_args(OProfile DEFAULT_MSG
-	OPROFILE_LIBRARY OPROFILE_INCLUDE_DIR)
+  OPROFILE_LIBRARY OPROFILE_INCLUDE_DIR
+)
+
+if (OPROFILE_FOUND)
+  if (NOT TARGET OProfile::OProfile)
+    add_library(OProfile::OProfile UNKNOWN IMPORTED)
+    set_target_properties(OProfile::OProfile PROPERTIES
+      IMPORTED_LOCATION ${OPROFILE_LIBRARIES}
+      INTERFACE_INCLUDE_DIRECTORIES ${OPROFILE_INCLUDE_DIRS}
+    )
+  endif()
+endif()
 
 mark_as_advanced(OPROFILE_INCLUDE_DIR OPROFILE_LIBRARY)
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -473,7 +473,6 @@ if(OPROFILING)
   if(OPROFILE_FOUND)
     message(STATUS "OProfile found, enabling profiling support")
     add_definitions(-DUSE_OPROFILE=1)
-    include_directories(${OPROFILE_INCLUDE_DIRS})
   else()
     message(FATAL_ERROR "OProfile not found. Can't build profiling support.")
   endif()

diff --git a/Source/Core/Common/CMakeLists.txt b/Source/Core/Common/CMakeLists.txt
@@ -146,7 +146,7 @@ if(USE_UPNP)
 endif()
 
 if(OPROFILE_FOUND)
-  target_link_libraries(common PRIVATE ${OPROFILE_LIBRARIES})
+  target_link_libraries(common PRIVATE OProfile::OProfile)
 endif()
 
 if(UNIX)