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

Workaround Clang 18 issues in our test suite #4932

Merged
merged 5 commits into from
Sep 9, 2024

Conversation

CaseyCarter
Copy link
Member

@CaseyCarter CaseyCarter commented Sep 3, 2024

  • Silence -Wnan-infinity-disabled in VSO_0000000_vector_algorithms_float. Clang 18 emits this warning for all of our /fp:fast configurations. Filed Clang 18+ /fp:fast emits -Wnan-infinity-disabled in VSO_0000000_vector_algorithms_floats #4931 to investigate, let's suppress the warnings for now. Since it's a new warning, we also need -Wno-unknown-warning-option to allow Clang 17 to keep accepting the test.
  • Internally skip some tests that need ADDITIONAL_COMPILE_FLAGS to pass with Clang 18.

…loat`

Clang 18 is emitting this warning for all `/fp:fast` configurations. Filed microsoft#4931 to investigate, let's suppress the warnings for now.
@StephanTLavavej StephanTLavavej self-assigned this Sep 4, 2024
@CaseyCarter CaseyCarter changed the title Silence -Wnan-infinity-disabled in VSO_0000000_vector_algorithms_float Workaround Clang 18 issues in our test suite Sep 4, 2024
@CaseyCarter

This comment was marked as resolved.

This comment was marked as resolved.

@StephanTLavavej
Copy link
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit 6477481 into microsoft:main Sep 9, 2024
39 checks passed
@StephanTLavavej
Copy link
Member

Thanks for scouting out the impending Clang version! 🏞️ 🔭 😻

@CaseyCarter CaseyCarter deleted the llvm18-prep branch September 9, 2024 22:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
test Related to test code
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants