[PATCH] D31168: Set FMF for -ffp-contract=fast rather than a TargetConfig

anemet updated this revision to Diff 92564. anemet added a comment. Address Akira's comments https://reviews.llvm.org/D31168 Files: lib/CodeGen/CGExprScalar.cpp test/CodeGen/ffp-contract-fast-option.c Index: test/CodeGen/ffp-contract-fast-option.c =

[PATCH] D31168: Set FMF for -ffp-contract=fast rather than a TargetConfig

anemet added a comment. Thanks, Akira. You're right on both accounts. I actually had a test written but failed to git add it. https://reviews.llvm.org/D31168 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/ma

[PATCH] D31168: Set FMF for -ffp-contract=fast rather than a TargetConfig

ahatanak added a comment. I think you can add a test case in this patch (after the llvm changes are checked in)? Comment at: lib/CodeGen/CGExprScalar.cpp:262 +ApplyFPFeatures(llvm::IRBuilderBase &Builder, Expr *E) +: CGBuilderTy::FastMathFlagGuard(Builder) { +

[PATCH] D31168: Set FMF for -ffp-contract=fast rather than a TargetConfig

anemet created this revision. This is toward moving fp-contraction=fast from an LLVM TargetOption to a FastMathFlag in order to fix PR25721. https://reviews.llvm.org/D31168 Files: lib/CodeGen/CGExprScalar.cpp Index: lib/CodeGen/CGExprScalar.cpp ==