[PATCH] D153725: [clang] Make amdgpu-arch tool work on Windows
Matt Arsenault via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jul 10 04:23:50 PDT 2023
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
More information about the cfe-commits
mailing list