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

Matt Arsenault via Phabricator via Openmp-commits openmp-commits at lists.llvm.org
Mon Jun 1 12:26:00 PDT 2020


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





More information about the Openmp-commits mailing list