-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Clang 18+ /fp:fast
emits -Wnan-infinity-disabled
in VSO_0000000_vector_algorithms_floats
#4931
Comments
…loat` Clang 18 is emitting this warning for all `/fp:fast` configurations. Filed microsoft#4931 to investigate, let's suppress the warnings for now.
I don't know what you would do about it other than suppression. |
Ideally, we don't test undefined behavior. It tends to be unreliable. That could mean adding a macro to the |
I see. The whole point of separating Out of the cursed modes, We can possibly drop infinities from STL/tests/std/tests/VSO_0000000_vector_algorithms_floats/test.cpp Lines 33 to 34 in c7c5ca7
As for NANs. the test doesn't even try them in the standard But most of these warnings (actually all quoted in the description) originate from |
After one minute of my not-especially-clueful attempting during the weekly maintainer meeting, we weren't able to repro warnings from |
When NaN and Inf are not properly supported, should we make the corresponding function return |
I don't think so. Something still works with NaN and Inf as expected even in |
I've verified that it's the case (although for libc++ with |
All 8
clang-cl /fp:fast
configs have the same issue. For example:Error log for
clang-cl /fp:fast /MT
without/D_USE_STD_VECTOR_ALGORITHMS=0
#4932 will silence these warnings for the time being.
The text was updated successfully, but these errors were encountered: