boostorg / boostorg/math

The future of fpclassify.hpp

Open
#548 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
C++
Stars
374
Forks
264
Avg merge
1d 23h
Merged PRs (30d)
17

Description

I'm opening this as a place for discussion really.

We currently have boost::math::fpclassify/isnan/isinf/isnormal etc which began life as workarounds for C++03 compilers, but they have another role: for IEEE conforming types they work even in the presence of -ffast-math.

Now, I have some dim recollection, that gcc/clang used to replace say `std::isnan(x)` with a literal `false` even with just -O3. And that we had some very inscrutable bug reports caused by this. However, that appears not to be the case with current releases, so perhaps this has been fixed/changed?

So... I'm undecided whether the boost functions should become synonyms for the std ones, but just wanted to flag the issue.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.