core.stdc.fenv: Add platform-specific FE_XXX flags#1554
core.stdc.fenv: Add platform-specific FE_XXX flags#1554dlang-bot merged 1 commit intodlang:masterfrom ibuclaw:fenv
Conversation
| FE_TONEAREST = 0, /// | ||
| FE_DOWNWARD = 0x400, /// | ||
| FE_UPWARD = 0x800, /// | ||
| FE_TOWARDZERO = 0xC00, /// |
There was a problem hiding this comment.
Just a dumb question - why do we duplicate this? Can't we combine these two?
There was a problem hiding this comment.
The use of X86_Any or PPC_Any is discouraged.
|
Ping? Anything else apart from nitpicking that needs addressing? |
|
Thanks for your pull request, @ibuclaw! Bugzilla referencesYour PR doesn't reference any Bugzilla issue. If your PR contains non-trivial changes, please reference a Bugzilla issue or create a manual changelog. |
|
Rebased, added more targets. |
@ibuclaw The dub test failed on jenkins because of network issues, which usually go away if you restart the test (I've just done so). If that happens in the future you login on Jenkins with your GitHub account and restart it yourself. |
(I missed the fact that Jenkins is not required when I added the |
4 participants
It has been a pet peeve of mine for a while that these are defined as a different name in
std.math, but not reflected here.