Skip to content

Fix conflicts with msvc arm neon64 intrinsics to enable win/arm64#28164

Merged
markdroth merged 1 commit intogrpc:masterfrom
niyas-sait:enable_win_arm64
Nov 29, 2021
Merged

Fix conflicts with msvc arm neon64 intrinsics to enable win/arm64#28164
markdroth merged 1 commit intogrpc:masterfrom
niyas-sait:enable_win_arm64

Conversation

@niyas-sait
Copy link
Copy Markdown
Contributor

This patch rename move64 private function from transport.cc to avoid conflicts with MSVC arm64 neon intrinsic function. See arm64 definition here

grpc cannot be build for windows on arm64 devices without this patch.

@markdroth

@linux-foundation-easycla
Copy link
Copy Markdown

linux-foundation-easycla bot commented Nov 20, 2021

CLA Signed

The committers are authorized under a signed CLA.

@niyas-sait
Copy link
Copy Markdown
Contributor Author

@markdroth Can you help with the review please?

@markdroth markdroth added release notes: no Indicates if PR should not be in release notes kokoro:run labels Nov 29, 2021
@markdroth markdroth self-assigned this Nov 29, 2021
@markdroth
Copy link
Copy Markdown
Member

This looks reasonable. I've triggered the tests.

@markdroth markdroth merged commit d88a57b into grpc:master Nov 29, 2021
@copybara-service copybara-service bot added the imported Specifies if the PR has been imported to the internal repository label Nov 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bloat/none imported Specifies if the PR has been imported to the internal repository lang/core perf-change/none release notes: no Indicates if PR should not be in release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants