[PATCH] D72820: [FPEnv] Add pragma FP_CONTRACT support under strict FP.

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 23 22:12:24 PST 2020


craig.topper added inline comments.


================
Comment at: clang/lib/CodeGen/CGExprScalar.cpp:3381
+           "constrained mode");
+    FMulAdd = Builder.CreateCall(
+        CGF.CGM.getIntrinsic(llvm::Intrinsic::experimental_constrained_fmuladd,
----------------
Doesn't this need to be CreateConstrainedFPCall so that the strictfp attribute is added? That will take care of adding the metadata operands too.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72820





More information about the llvm-commits mailing list