Move HIP_HCC define to CMake

[ROCm/clr commit: f9a31e28ad]

projects/clr/hipamd/CMakeLists.txt

@@ -92,8 +92,8 @@ if(HIP_PLATFORM STREQUAL "hcc")
     set(CMAKE_CXX_COMPILER "${HCC_HOME}/bin/hcc")
     set(CMAKE_C_COMPILER   "${HCC_HOME}/bin/hcc")
 
-    set(CMAKE_CXX_FLAGS " -hc -I${HCC_HOME}/include -I${HSA_PATH}/include -stdlib=libc++ ")
-    set(CMAKE_C_FLAGS   " -hc -I${HCC_HOME}/include -I${HSA_PATH}/include -stdlib=libc++ ")
+    set(CMAKE_CXX_FLAGS " -hc -I${HCC_HOME}/include -I${HSA_PATH}/include -stdlib=libc++ -DHIP_HCC")
+    set(CMAKE_C_FLAGS   " -hc -I${HCC_HOME}/include -I${HSA_PATH}/include -stdlib=libc++ -DHIP_HCC")
 
     set(SOURCE_FILES src/device_util.cpp
                      src/hip_hcc.cpp 

projects/clr/hipamd/include/hcc_detail/hip_hcc.h

@@ -24,7 +24,6 @@ THE SOFTWARE.
 #include "hcc_detail/hip_util.h"
 #include "hcc_detail/staging_buffer.h"
 
-#define HIP_HCC
 
 #if defined(__HCC__) && (__hcc_workweek__ < 1502)
 #error("This version of HIP requires a newer version of HCC.");
@@ -90,8 +89,6 @@ class ihipDevice_t;
 #define API_COLOR KGRN
 
 
-#define HIP_HCC  
-
 // If set, thread-safety is enforced on all stream functions.
 // Stream functions will acquire a mutex before entering critical sections.
 #define STREAM_THREAD_SAFE 1

projects/clr/hipamd/src/hip_hcc.cpp

@@ -43,11 +43,6 @@ THE SOFTWARE.
 #include "hcc_detail/hip_hcc.h"
 #include "hsa_ext_amd.h"
 
-// HIP includes:
-#define HIP_HCC
-//#include "hcc_detail/staging_buffer.h"
-
-
 // TODO, re-org header order.
 extern const char *ihipErrorString(hipError_t hip_error);
 #include "hcc_detail/trace_helper.h"

projects/clr/hipamd/src/staging_buffer.cpp

@@ -24,6 +24,8 @@ THE SOFTWARE.
 #include "hcc_detail/staging_buffer.h"
 
 #ifdef HIP_HCC
+#include "hcc_detail/hip_runtime.h"
+#include "hcc_detail/hip_hcc.h"
 #define THROW_ERROR(e) throw ihipException(e)
 #else
 #define THROW_ERROR(e) throw 
@@ -301,7 +303,7 @@ void StagingBuffer::CopyPeerToPeer(void* dst, hsa_agent_t dstAgent, const void*
 
             tprintf (DB_COPY2, "P2P: wait_completion[%d] bytesRemaining=%zu\n", bufferIndex, bytesRemaining1);
 
-            bool hostWait = 0;
+            bool hostWait = 0; // TODO - remove me
 
             if (hostWait) {
                 // Host-side wait, should not be necessary: