Skip to content
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

Blazor WASM SIMD support #28997

Merged
merged 2 commits into from
Apr 26, 2023
Merged

Blazor WASM SIMD support #28997

merged 2 commits into from
Apr 26, 2023

Conversation

guardrex
Copy link
Collaborator

@guardrex guardrex commented Apr 17, 2023

Fixes #28996

I left the remark about Apple Safari in the 7.0 version of this content. Was 7.0 ever patched to support SIMD for Safari with <WasmEnableSIMD> enabled?


Internal previews

📄 File 🔗 Preview link
aspnetcore/blazor/tooling.md Tooling for ASP.NET Core Blazor

@guardrex guardrex self-assigned this Apr 17, 2023
@guardrex guardrex requested review from radekdoulik and removed request for danroth27 and mkArtakMSFT April 26, 2023 16:56
@guardrex
Copy link
Collaborator Author

@radekdoulik ... We're going to go right for the PU engineer review in order to get these docs PRs updated/merged. Can you check here and see if I have this content correct?

Copy link
Member

@danroth27 danroth27 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Small nit, otherwise looks good to me.

Co-authored-by: Daniel Roth <daroth@microsoft.com>
@guardrex guardrex removed the request for review from radekdoulik April 26, 2023 21:52
@guardrex guardrex merged commit 1e43422 into main Apr 26, 2023
@guardrex guardrex deleted the guardrex/blazor-wasm-simd-support branch April 26, 2023 21:59
@radekdoulik
Copy link
Member

Looks fine to me.

BTW, the Safari should support SIMD from version 16.4 (according to https://webassembly.org/roadmap/). I don't remember whether it includes Safari on iOS too.

Note that we enabled the exception handling recently as well. dotnet/runtime#84573

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.

Blazor WASM SIMD support
3 participants