[Openmp-commits] [PATCH] D80897: [OpenMP] Initial support for std::complex in target regions

[Matt Arsenault via Phabricator via Openmp-commits]

arsenm added inline comments.


================
Comment at: clang/lib/Headers/__clang_cuda_complex_builtins.h:136-137
+      __d = _COPYSIGNf(_ISINFf(__d) ? 1 : 0, __d);
+      if (_ISNANf(__a))
+        __a = _COPYSIGNf(0, __a);
+      if (_ISNANf(__b))
----------------
Why does this try to preserve the sign of a nan? They are meaningless


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80897