diff options
author | Sanjay Patel <spatel@rotateright.com> | 2020-12-30 09:11:10 -0500 |
---|---|---|
committer | Sanjay Patel <spatel@rotateright.com> | 2020-12-30 09:51:23 -0500 |
commit | e90ea76380d411bf81861228f23e4716ef337100 (patch) | |
tree | e5716f55cfef7474c2a3c33f179c4c747f11dfdb /llvm/lib/Transforms/Utils/LoopUtils.cpp | |
parent | c6035a7bdf207dc3c1e12c041d5658770893fc68 (diff) | |
download | llvm-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.cpp | 4 |
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: |