aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/Transforms/Utils/LoopUtils.cpp
diff options
context:
space:
mode:
authorSanjay Patel <spatel@rotateright.com>2020-12-30 09:11:10 -0500
committerSanjay Patel <spatel@rotateright.com>2020-12-30 09:51:23 -0500
commite90ea76380d411bf81861228f23e4716ef337100 (patch)
treee5716f55cfef7474c2a3c33f179c4c747f11dfdb /llvm/lib/Transforms/Utils/LoopUtils.cpp
parentc6035a7bdf207dc3c1e12c041d5658770893fc68 (diff)
downloadllvm-e90ea76380d411bf81861228f23e4716ef337100.zip
llvm-e90ea76380d411bf81861228f23e4716ef337100.tar.gz
llvm-e90ea76380d411bf81861228f23e4716ef337100.tar.bz2
[IR] remove 'NoNan' param when creating FP reductions
This is no-functional-change-intended (AFAIK, we can't isolate this difference in a regression test). That's because the callers should be setting the IRBuilder's FMF field when creating the reduction and/or setting those flags after creating. It doesn't make sense to override this one flag alone. This is part of a multi-step process to clean up the FMF setting/propagation. See PR35538 for an example.
Diffstat (limited to 'llvm/lib/Transforms/Utils/LoopUtils.cpp')
-rw-r--r--llvm/lib/Transforms/Utils/LoopUtils.cpp4
1 files changed, 2 insertions, 2 deletions
diff --git a/llvm/lib/Transforms/Utils/LoopUtils.cpp b/llvm/lib/Transforms/Utils/LoopUtils.cpp
index 80ae6b3..a3665a5 100644
--- a/llvm/lib/Transforms/Utils/LoopUtils.cpp
+++ b/llvm/lib/Transforms/Utils/LoopUtils.cpp
@@ -1039,10 +1039,10 @@ Value *llvm::createSimpleTargetReduction(
case Instruction::FCmp:
if (Flags.IsMaxOp) {
MinMaxKind = RD::MRK_FloatMax;
- BuildFunc = [&]() { return Builder.CreateFPMaxReduce(Src, Flags.NoNaN); };
+ BuildFunc = [&]() { return Builder.CreateFPMaxReduce(Src); };
} else {
MinMaxKind = RD::MRK_FloatMin;
- BuildFunc = [&]() { return Builder.CreateFPMinReduce(Src, Flags.NoNaN); };
+ BuildFunc = [&]() { return Builder.CreateFPMinReduce(Src); };
}
break;
default: