arsenm added a comment.

In D153725#4484711 <https://reviews.llvm.org/D153725#4484711>, @JonChesterfield 
wrote:

> The right thing to do on Linux for this is to query the driver directly. That 
> is, the kernel should populate some string under /sys that we read. That 
> isn't yet implemented.

It should definitely not do that. That's what this redundant thing does 
<https://github.com/RadeonOpenCompute/llvm-project/tree/amd-stg-open/llvm/lib/OffloadArch>
 . The kernel doesn't know the names of these devices. The kernel knows 
different names that map to PCI ids that are not the same as the gfx numbers. 
The compiler should not be responsible for maintaining yet another name mapping 
table and should go through a real API

> Does windows happen to have that functionality available?

This sounds very un-windows like. I assume the equivalent is digging around in 
the registry


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153725

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to