[PATCH] D95948: Stop traping on sNaN in __builtin_isnan

Kevin P. Neal via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Feb 4 11:24:53 PST 2021


kpn added a comment.

In D95948#2542884 <https://reviews.llvm.org/D95948#2542884>, @mibintc wrote:

> Should we add tests for mlong-double-64, -80, -128?

Assuming Ty->getScalarSizeInBits() returns 64, 80, or 128 in those cases, I think it's good enough to just add an AArch64 (or some other 128-bit long double host) and call it a day. We're testing isnan(), not that command line option, after all.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D95948/new/

https://reviews.llvm.org/D95948



More information about the cfe-commits mailing list