-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
[wasm] Clang instruction selection crash for PackedSimd.Shuffle in AOT #85023
Comments
Tagging subscribers to 'arch-wasm': @lewing Issue DetailsWhen AOTing the System.Numerics.Vectors.Tests assembly on main, clang instruction selection crashes when compiling PackedSimd.Shuffle, see https://gist.github.com/kg/450e9f560df6ca5c82c38eff1902a29b . If I remove PackedSimd.Shuffle entirely from the .cs files the AOT succeeds. cc @radekdoulik
|
Can't reproduce this, what is the command line ? |
Radek appears to have hit it before as well, see llvm/llvm-project#55559 . Might have something to do with OS packages or be a nondeterminism thing. Can't reproduce it on main right now, but I've blown away my checkout multiple times since then - will see if I can reconstruct it. |
Hit this again trying to AOT a different test suite.
|
Looks like both the wasm.shuffle intrinsic, and the llvm shufflevector instruction requires constant arguments, so this will be hard to implement efficiently. |
@radekdoulik update this with the current thinking |
I forget, is this really a chrome issue? |
What was the fix @lewing @radekdoulik ? |
When AOTing the System.Numerics.Vectors.Tests assembly on main, clang instruction selection crashes when compiling PackedSimd.Shuffle, see https://gist.github.com/kg/450e9f560df6ca5c82c38eff1902a29b . If I remove PackedSimd.Shuffle entirely from the .cs files the AOT succeeds.
cc @radekdoulik
The text was updated successfully, but these errors were encountered: