-
Notifications
You must be signed in to change notification settings - Fork 11.6k
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] Cannot select BUILD_VECTOR when enabling SIMD128 #63817
Comments
@llvm/issue-subscribers-backend-webassembly |
Thanks for the excellent reproducer. I'll have a look at this tomorrow. |
Candidate fix: https://reviews.llvm.org/D155386 |
Great! Thanks. |
It seems like this patch is never applied. Now that Phabricator is offline, how can we merge it? |
@DataCorrupted: Just create pull request from Phabricator patch. |
Cool, I'll come back next week and send the PR on behalf of @tlively if he hasn't by then. |
Previously we expected lane constants to be in the range of signed values for each lane size, but the included test case produced large unsigned values that fall outside that range. Allow instruction selection to proceed in this case rather than failing. Fixes llvm#63817.
I just re-uploaded the patch as a PR with the fixes it was waiting on applied. |
…vm#85880) Previously we expected lane constants to be in the range of signed values for each lane size, but the included test case produced large unsigned values that fall outside that range. Allow instruction selection to proceed in this case rather than failing. Fixes llvm#63817.
Description
When feature
simd128
is on, the following code crashes the WebAssembly backend (bothwasm32
andwasm64
) with errorCannot select: v16i8 = BUILD_VECTOR
。Minimal Reproduction
https://godbolt.org/z/o71M96j4P
Code
Stack Trace
The text was updated successfully, but these errors were encountered: