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

Optimize System.Text.Unicode using arm64 intrinsics #35035

Closed
BruceForstall opened this issue Apr 16, 2020 · 5 comments
Closed

Optimize System.Text.Unicode using arm64 intrinsics #35035

BruceForstall opened this issue Apr 16, 2020 · 5 comments

Comments

@BruceForstall
Copy link
Member

BruceForstall commented Apr 16, 2020

This item tracks the conversion of the System.Text.Unicode class to use arm64 intrinsics.

Related: #33308

@BruceForstall BruceForstall added this to the 5.0 milestone Apr 16, 2020
@Dotnet-GitSync-Bot Dotnet-GitSync-Bot added area-System.Runtime untriaged New issue has not been triaged by the area owner labels Apr 16, 2020
@BruceForstall BruceForstall removed the untriaged New issue has not been triaged by the area owner label Jun 3, 2020
@BruceForstall
Copy link
Member Author

I'm moving this to Future, as there is no current plan to do the work for .NET 5.

@BruceForstall BruceForstall modified the milestones: 5.0, Future Jun 11, 2020
@jeffhandley jeffhandley modified the milestones: Future, 5.0 Jun 15, 2020
@carlossanlop
Copy link
Member

carlossanlop commented Jun 24, 2020

@BruceForstall @tannergooding below are the 3 APIs to fix for this area, but I realized that only the first API can be found in Utf16Utility. The other two exist in Utf8Utility instead (there's no Utf16 equivalent):

Was that a typo? Should I work on the Utf8 ones instead?

@danmoseley
Copy link
Member

cc @GrabYourPitchforks

@BruceForstall
Copy link
Member Author

It's certainly likely to be a typo. I created the initial list using, essentially, findstr/grep with a little bit of intelligence thrown in. I easily could have made a mistake.

Bottom-line: anything in System.Text.Unicode that has been intrinsics optimized for Intel should be considered tracked by this issue for optimizing for arm64. Feel free to update the list of APIs listed in #33308 to be more correct.

@carlossanlop
Copy link
Member

Thanks Bruce. I'll adjust the signatures in the original issue.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

5 participants