[Openmp-commits] [PATCH] D143192: [Libomptarget] Improve next-gen AMDGPU plugin error messages

[Joseph Huber via Phabricator via Openmp-commits]

jhuber6 added inline comments.


================
Comment at: openmp/libomptarget/plugins-nextgen/amdgpu/src/rtl.cpp:2460
+                           &Node) != HSA_STATUS_SUCCESS)
+      return HSA_STATUS_ERROR;
 
----------------
kevinsala wrote:
> We could print the fatal message even if `hsa_agent_get_info` fails, right? We can print the GPU id or `unknown`/`unspecified` if the call failed.
Yeah, we could probably just initialize it to `-1` and continue like nothing happened.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143192/new/

https://reviews.llvm.org/D143192