[libomptarget][amdgpu] Remove call into hsa that follows failing to init hsa

[JonChesterfield]

Summary

[libomptarget][amdgpu] Remove call into hsa that follows failing to init hsa

The call in rtl.cpp carefully avoids calling into hsa to find out why it can't call
into hsa, but the call nested within system.cpp did not. Given the failure is reported
anyway, drop the printf. Also drop an adjacent block that looks like a merge failure.

Reported by jdoerfert

[jdoerfert]

Same problem for the RTLDeviceInfoTy destructor. let's fix both together.

[JonChesterfield]

sure, can roll them together

openmp/libomptarget/plugins/amdgpu/impl/system.cpp

static hsa_status_t init_compute_and_memory() {
   else
     return HSA_STATUS_ERROR_NOT_INITIALIZED;
 }
 
 hsa_status_t init_hsa() {
   DEBUG_PRINT("Initializing HSA...");
   hsa_status_t err = hsa_init();
   if (err != HSA_STATUS_SUCCESS) {
-    printf("[%s:%d] %s failed: %s\n", __FILE__, __LINE__,
-           "Initializing the hsa runtime", get_error_string(err));
     return err;
   }
-  if (err != HSA_STATUS_SUCCESS)
-    return err;
 
   err = init_compute_and_memory();
   if (err != HSA_STATUS_SUCCESS)
     return err;
   if (err != HSA_STATUS_SUCCESS) {
     printf("[%s:%d] %s failed: %s\n", __FILE__, __LINE__,
            "After initializing compute and memory", get_error_string(err));
     return err;