Merge pull request #22 from jvesely/master

minor fixes
ROCm · May 19, 2017 · f92eba2 · f92eba2
2 parents 8dc021c + add71bf
commit f92eba2
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/sample/vector_copy.c b/sample/vector_copy.c
@@ -49,7 +49,7 @@
 
 #define check(msg, status) \
 if (status != HSA_STATUS_SUCCESS) { \
-    printf("%s failed.\n", #msg); \
+    printf("%s failed: %x.\n", #msg, status); \
     exit(1); \
 } else { \
    printf("%s succeeded.\n", #msg); \
@@ -230,10 +230,11 @@ int main(int argc, char **argv) {
      */
     hsa_ext_module_t module;
     if(HSA_PROFILE_FULL == profile) {
-        load_module_from_file("vector_copy_full.brig",&module);
+        err = load_module_from_file("vector_copy_full.brig",&module);
     } else {
-        load_module_from_file("vector_copy_base.brig",&module);
+        err = load_module_from_file("vector_copy_base.brig",&module);
     }
+    check(Load module from file, err);
 
     /*
      * Create hsa program.

diff --git a/src/cmake_modules/hsa_common.cmake b/src/cmake_modules/hsa_common.cmake
@@ -55,7 +55,7 @@ if(UNIX)
   set(PS ":")
   set(CMAKE_CXX_FLAGS "-Wall -std=c++11 ${EXTRA_CFLAGS}")
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fpic")
-  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wl,--unresolved-symbols=ignore-in-shared-libs")
+  set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wl,--no-undefined")
   set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -fno-strict-aliasing")
   if ( CMAKE_SYSTEM_PROCESSOR STREQUAL "x86_64" )
     set( CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -m64  -msse -msse2" )