Skip to content

Fix clang19 build #105141

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

Merged
merged 1 commit into from
Jul 20, 2024
Merged

Fix clang19 build #105141

merged 1 commit into from
Jul 20, 2024

Conversation

am11
Copy link
Member

@am11 am11 commented Jul 19, 2024

tested on linux-arm64

@dotnet-policy-service dotnet-policy-service bot added the community-contribution Indicates that the PR has been added by a community member label Jul 19, 2024
Copy link
Contributor

Tagging subscribers to this area: @hoyosjs
See info in area-owners.md if you want to be subscribed.

@am11 am11 force-pushed the feature/build/clang19 branch from 6eeafed to 5b4a58e Compare July 19, 2024 11:36
@@ -105,7 +105,7 @@ find_first_matching_suffix_simd (
dn_simdhash_suffixes match_vector;
// Completely untested.
static const dn_simdhash_suffixes byte_mask = {
1, 2, 4, 8, 16, 32, 64, 128, 1, 2, 4, 8, 16, 32, 64, 128
{ 1, 2, 4, 8, 16, 32, 64, 128, 1, 2, 4, 8, 16, 32, 64, 128 }
Copy link
Member Author

Choose a reason for hiding this comment

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

This one was reported by gcc on arm64.

Copy link
Member

@jkotas jkotas left a comment

Choose a reason for hiding this comment

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

Thanks

@jkotas jkotas merged commit 043ae8c into dotnet:main Jul 20, 2024
147 of 151 checks passed
@am11 am11 deleted the feature/build/clang19 branch July 20, 2024 18:37
@github-actions github-actions bot locked and limited conversation to collaborators Aug 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Infrastructure-coreclr community-contribution Indicates that the PR has been added by a community member
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants