-
Notifications
You must be signed in to change notification settings - Fork 89
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
SIMD trap at avx2::_mm256_sad_epu8 #1412
Comments
I think something somewhere is suppressing those warnings. Maybe it is cargo, maybe it is rustc when
Should be fixed on the implement_xgetbv branch now. I'm currently doing a local build of veloren to check if everything works. |
Looks like there is another image decoding issue:
|
Did you want me to look at further into this when I get time? |
That is not necessary. I know what the issue is (another unimplemented intrinsic), I am working on implementing it. |
Progress update: I got to the login screen and after I tried to login it crashed with
in |
Oooh, lots of progress! Definitely keep me updated and let me know if I can lend a hand, I'd love to see this as a viable alternative to |
If you could quickly get rid of shaderc and spirv_cross that would be great :) Implementing new intrinsics is reasonably quickly as I can test the respective crate in isolation, but recompiling the entirety of veloren once I implemented some intrinsics takes 15min on my 2 core + HT intel core i3 laptop (can't use the dev-desktop-eu-2.infra.rust-lang.org due veloren to depending on vulkan). Like half of that is spent in compiling shaderc and spriv_cross. (Yes, I'm fully aware that you can't do this quickly, but maybe for the long term switching to wgsl using naga would be possible? Naga is already a dependency of veloren through wgpu.) I will keep you updated! |
IIRC you can disable the |
Works with the latest nightly now. |
Hello,
I made an attempt at building Veloren with the cranelift backend!
I'll preface this by saying that I did not expect this to work, and the fact that the thing even compiled at all is miraculous to me. Veloren is an enormous codebase nowadays that pulls in a terrifying number of dependencies that do all sorts of weird and unusual things that likely represent a headache for a codegen backend like JIT, dynamic linking, horrible multi-threading things, linking to several C and C++ codebases, a lot of SIMD (both explicit and implicit), atomics all over the place, etc.
When running the executable, I get:
(this happens after both wgpu selects a graphics adapter and the internal server boots up, so that it got this far is impressive!)
That said, there was no warning about this intrinsic (or any warnings at all, for that manner) reported during the build process, despite this post implying that there should be.
Hopefully this is useful information!
The text was updated successfully, but these errors were encountered: