<div dir="ltr">Please ignore this patch as it had a bug - sending an updated one now. Apologies for the spam<br></div><div class="gmail_extra"><br clear="all"><div><div class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Pradeep Ramachandran, PhD<div>Solution Architect,</div><div><a href="http://pradeeprama.info/" target="_blank">pradeeprama.info/</a></div><div><a href="http://www.multicorewareinc.com/" target="_blank">www.multicorewareinc.com/</a></div><div>Ph:   +91 99627 82018</div></div></div></div></div></div></div></div></div>
<br><div class="gmail_quote">On Thu, Oct 29, 2015 at 2:23 PM, Pradeep Ramachandran <span dir="ltr"><<a href="mailto:pradeep@multicorewareinc.com" target="_blank">pradeep@multicorewareinc.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"># HG changeset patch<br>
# User Pradeep Ramachandran <<a href="mailto:pradeep@multicorewareinc.com">pradeep@multicorewareinc.com</a>><br>
# Date 1446108732 -19800<br>
#      Thu Oct 29 14:22:12 2015 +0530<br>
# Node ID b6ad5a853f1588b5af7603a9a275e8e64eb7c644<br>
# Parent  6563218ce342c30bfd4f9bc172a1dab510e6e55b<br>
cmake: moving vtune compilation into separate file<br>
<br>
diff -r 6563218ce342 -r b6ad5a853f15 source/CMakeLists.txt<br>
--- a/source/CMakeLists.txt     Mon Oct 26 12:13:53 2015 +0530<br>
+++ b/source/CMakeLists.txt     Thu Oct 29 14:22:12 2015 +0530<br>
@@ -385,16 +385,19 @@<br>
<br>
 option(ENABLE_VTUNE "Enable Vtune profiling instrumentation" OFF)<br>
 if(ENABLE_VTUNE)<br>
-    add_definitions(-DENABLE_VTUNE)<br>
-    include_directories($ENV{VTUNE_AMPLIFIER_XE_2015_DIR}/include)<br>
-    list(APPEND PLATFORM_LIBS vtune)<br>
-    link_directories($ENV{VTUNE_AMPLIFIER_XE_2015_DIR}/lib64)<br>
-    if(WIN32)<br>
-        list(APPEND PLATFORM_LIBS libittnotify.lib)<br>
-    else()<br>
-        list(APPEND PLATFORM_LIBS libittnotify.a dl)<br>
-    endif()<br>
-    add_subdirectory(profile/vtune)<br>
+    find_package(Vtune)<br>
+    if(VTUNE_FOUND)<br>
+        add_definitions(-DENABLE_VTUNE)<br>
+        include_directories(${VTUNE_INCLUDE_DIR})<br>
+        list(APPEND PLATFORM_LIBS vtune)<br>
+        link_directories(${VTUNE_LIBRARY_DIR})<br>
+        if(WIN32)<br>
+            list(APPEND PLATFORM_LIBS libittnotify.lib)<br>
+        else()<br>
+            list(APPEND PLATFORM_LIBS libittnotify.a dl)<br>
+        endif()<br>
+        add_subdirectory(profile/vtune)<br>
+    endif(VTUNE_FOUND)<br>
 endif(ENABLE_VTUNE)<br>
<br>
 option(DETAILED_CU_STATS "Enable internal profiling of encoder work" OFF)<br>
diff -r 6563218ce342 -r b6ad5a853f15 source/cmake/FindVtune.cmake<br>
--- /dev/null   Thu Jan 01 00:00:00 1970 +0000<br>
+++ b/source/cmake/FindVtune.cmake      Thu Oct 29 14:22:12 2015 +0530<br>
@@ -0,0 +1,22 @@<br>
+# Module for locating Vtune<br>
+#<br>
+# Read-only variables<br>
+#   VTUNE_FOUND: Indicates that the library has been found<br>
+#   VTUNE_INCLUDE_DIR: Points to the vtunes include dir<br>
+#   VTUNE_LIBRARY_DIR: Points to the directory with libraries<br>
+#<br>
+# Copyright (c) 2015 Pradeep Ramachandran<br>
+<br>
+include(FindPackageHandleStandardArgs)<br>
+<br>
+find_path(VTUNE_DIR<br>
+    NAMES amplxe-vars.sh<br>
+    PATHS ENV NUMA_ROOT<br>
+    HINTS $ENV{VTUNE_AMPLIFIER_XE_2016_DIR}, $ENV{VTUNE_AMPLIFIER_XE_2015_DIR}<br>
+    DOC "Vtune root directory")<br>
+<br>
+set (VTUNE_INCLUDE_DIR ${VTUNE_DIR}/include)<br>
+set (VTUNE_LIBRARY_DIR ${VTUNE_DIR}/lib64)<br>
+<br>
+mark_as_advanced(VTUNE_DIR)<br>
+find_package_handle_standard_args(VTUNE REQUIRED_VARS VTUNE_DIR VTUNE_INCLUDE_DIR VTUNE_LIBRARY_DIR)<br>
diff -r 6563218ce342 -r b6ad5a853f15 source/profile/vtune/CMakeLists.txt<br>
--- a/source/profile/vtune/CMakeLists.txt       Mon Oct 26 12:13:53 2015 +0530<br>
+++ b/source/profile/vtune/CMakeLists.txt       Thu Oct 29 14:22:12 2015 +0530<br>
@@ -1,2 +1,2 @@<br>
-include_directories($ENV{VTUNE_AMPLIFIER_XE_2015_DIR}/include)<br>
+include_directories(${VTUNE_INCLUDE_DIR})<br>
 add_library(vtune vtune.h vtune.cpp ../cpuEvents.h)<br>
diff -r 6563218ce342 -r b6ad5a853f15 source/profile/vtune/vtune.cpp<br>
--- a/source/profile/vtune/vtune.cpp    Mon Oct 26 12:13:53 2015 +0530<br>
+++ b/source/profile/vtune/vtune.cpp    Thu Oct 29 14:22:12 2015 +0530<br>
@@ -30,7 +30,6 @@<br>
 const char *stringNames[] =<br>
 {<br>
 #include "../cpuEvents.h"<br>
-    ""<br>
 };<br>
 #undef CPU_EVENT<br>
<br>
@@ -44,7 +43,8 @@<br>
 void vtuneInit()<br>
 {<br>
     domain = __itt_domain_create("x265");<br>
-    for (size_t i = 0; i < sizeof(stringNames) / sizeof(const char *); i++)<br>
+    size_t length = sizeof(stringNames) / sizeof(const char *);<br>
+    for (size_t i = 0; i < length; i++)<br>
         taskHandle[i] = __itt_string_handle_create(stringNames[i]);<br>
 }<br>
<br>
</blockquote></div><br></div>