Actually no one is calling this guy

src/hsa/uarch.cpp

@@ -306,11 +306,4 @@ const char* get_str_uarch_hsa(struct uarch* arch) {
     return NULL;
   }
   return uarch_str[arch->uarch];
-}
-
-// TODO: Move this to common!
-void free_uarch_struct(struct uarch* arch) {
-  free(arch->uarch_str);
-  free(arch->chip_str);
-  free(arch);
-}
+}

src/hsa/uarch.hpp

@@ -8,6 +8,5 @@ struct uarch;
 struct uarch* get_uarch_from_hsa(struct gpu_info* gpu, char* gpu_name);
 char* get_str_uarch_hsa(struct uarch* arch);
 char* get_str_process(struct uarch* arch); // TODO: Shouldnt we define this in the cpp?
-void free_uarch_struct(struct uarch* arch);
 
 #endif