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

[Kevin Sala via Phabricator via Openmp-commits]

kevinsala accepted this revision.
kevinsala added a comment.
This revision is now accepted and ready to land.

LGTM



================
Comment at: openmp/libomptarget/plugins-nextgen/amdgpu/src/rtl.cpp:2460
+                           &Node) != HSA_STATUS_SUCCESS)
+      return HSA_STATUS_ERROR;
 
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143192