Skip to content

Conversation

@voxelbee
Copy link
Contributor

Fix for issue #501

@Firestar99
Copy link
Member

Firestar99 commented Jan 5, 2026

  1. Looks like swiftshader doesn't support int64... I was looking into switching to lavapipe anyway for unrelated reasons, guess I got to finish that one.

  2. wgpu has a native-only feature called SHADER_INT64, so you don't necessarily need to use the ash runner

  3. I wouldn't be surprised if there's more integer intrinsics that have issues. These in particular have been added before we had difftests, and I wanted to some for quite a while already.

@Firestar99
Copy link
Member

Firestar99 commented Jan 23, 2026

I just rebased it and split out the fix from the test.

I started working on a difftest that would test all the various functions with wgpu, only to notice that our wgpu difftest runners are a mess and I need to clean them up first before we can use wgpu for 64bit testing. So I'm fine with keeping the ash tests as is, and I may clean them up later. (There's still #334 waiting for a review, which should speed up difftests significantly...)

Anyway, I think it's better if we land it now and take care of testing the remaining ops some other time. I may even have found an alternative way of testing them even better. Thanks for the patch ❤️

@Firestar99 Firestar99 enabled auto-merge January 23, 2026 15:06
@Firestar99 Firestar99 added this pull request to the merge queue Jan 23, 2026
Merged via the queue into Rust-GPU:main with commit 7ad4c4d Jan 23, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants