-
-
Notifications
You must be signed in to change notification settings - Fork 83
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
stop distributing radv #103
Comments
@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!
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. |
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 |
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 : |
Reopen this issue. Time to pop some confetti! |
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.
The text was updated successfully, but these errors were encountered: