stop distributing radv
zmike opened this issue · comments
RADV does not work on Windows. This is not a bug or a missing feature, it just isn't going to work. Distributing it in this package implies that it will, and this wastes the time of anyone who tries to use it.
@zmike you should be aware that if LLVM is built with AMDGPU target, Microsoft CLC fails to build if RADV isn't built as well.
Anyway I took your suggestion for 22.1.0.
Great, thanks!
@zmike you should be aware that if LLVM is built with AMDGPU target, Microsoft CLC fails to build if RADV isn't built as well.
I don't think this is a RADV issue though? At a minimum, if you have to build RADV to work around this for now, you could just not distribute RADV after.
I don't think this is a RADV issue though?
It's not. It's a Microsoft CLC / clover issue. It was worked around for clover only when using LLVM config tool. Reading meson.build I can tell the work around doesn't apply for
Microsoft CLC though.
Maybe file another bug? Or update that one. Dunno, but it sounds like it should be fixed.
don't asking for enabling RADV again but note I found some usage for RADV for Windows in :
tgjones/shader-playground#102
"It might be nice to experiment with, and it already compiles as a Vulkan ICD on Windows. It just needs RADV_FORCE_FAMILY to be set, and NIR/ACO output is available as text via VK_KHR_pipeline_executable_properties."
Reopen this issue.
https://www.phoronix.com/news/Experimental-RADV-Windows
Time to pop some confetti!