search

pal1000 / mesa-dist-win

Pre-built Mesa3D drivers for Windows
MIT License
864 stars 78 forks source link

stop distributing radv #103

Closed zmike closed 2 years ago

zmike commented 2 years ago

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.

pal1000 commented 2 years ago

@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.

zmike commented 2 years ago

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.

pal1000 commented 2 years ago

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.

zmike commented 2 years ago

Maybe file another bug? Or update that one. Dunno, but it sounds like it should be fixed.

oscarbg commented 1 year ago

don't asking for enabling RADV again but note I found some usage for RADV for Windows in : https://github.com/tgjones/shader-playground/issues/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."

Weather-OS commented 1 week ago

Reopen this issue. https://www.phoronix.com/news/Experimental-RADV-Windows

Time to pop some confetti!