This allows std::isnan to be fully inlined, instead of generating calls.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
OK, this LGTM.
The C library header tries to perform this optimization for us, but it fails to detect that Clang has __builtin_isnan because it reports as GCC 4.2, and the C library checks for GCC 4.4.
It's unfortunate we need to do this ourselves, but I'm OK making this change.
Comment Actions
Since the __libcpp_is_nan_or_builtin hacks are no longer needed (they can directly call isnan), could you please send a separate change to remove it?