[PATCH] D145150: clang: Emit nofpclass(nan inf) for -ffinite-math-only

[Eli Friedman via Phabricator via llvm-commits]

efriedma added a comment.

Looks fine from a codegen perspective, assuming these are the semantics we want for -ffinite-math-only.



================
Comment at: clang/lib/CodeGen/CGCall.cpp:3052
+        AI->addAttrs(
+            llvm::AttrBuilder(getLLVMContext()).addNoFPClassAttr(Mask));
+      }
----------------
What cases does this cover that aren't already covered by the code in CodeGenModule::ConstructAttributeList?


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

https://reviews.llvm.org/D145150