[PATCH] D134369: [Clang] Support constexpr builtin fmax

Joshua Cranmer via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Oct 7 11:57:55 PDT 2022


jcranmer-intel added inline comments.


================
Comment at: clang/lib/AST/ExprConstant.cpp:14029-14030
+  case Builtin::BI__builtin_fmaxf16:
+  case Builtin::BI__builtin_fmaxf128: {
+    APFloat RHS(0.);
+    if (!EvaluateFloat(E->getArg(0), Result, Info) ||
----------------
I think you should add a comment saying `// TODO: Handle sNaN`, just so that we remember to revisit this later.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134369



More information about the cfe-commits mailing list