aboutsummaryrefslogtreecommitdiff
path: root/llvm/lib/CodeGen/MachinePostDominators.cpp
diff options
context:
space:
mode:
authorSanjay Patel <spatel@rotateright.com>2019-09-25 13:14:12 +0000
committerSanjay Patel <spatel@rotateright.com>2019-09-25 13:14:12 +0000
commitdec03223a97af0e4dfcb23da55c0f7f8c9b62d00 (patch)
treec6886ee26d17caa3e61557e0a8f76d902a8c1c79 /llvm/lib/CodeGen/MachinePostDominators.cpp
parent12e30999212123b76acae46bb888b9738e989f9a (diff)
downloadllvm-dec03223a97af0e4dfcb23da55c0f7f8c9b62d00.zip
llvm-dec03223a97af0e4dfcb23da55c0f7f8c9b62d00.tar.gz
llvm-dec03223a97af0e4dfcb23da55c0f7f8c9b62d00.tar.bz2
[IR] allow fast-math-flags on phi of FP values
The changes here are based on the corresponding diffs for allowing FMF on 'select': D61917 As discussed there, we want to have fast-math-flags be a property of an FP value because the alternative (having them on things like fcmp) leads to logical inconsistency such as: https://bugs.llvm.org/show_bug.cgi?id=38086 The earlier patch for select made almost no practical difference because most unoptimized conditional code begins life as a phi (based on what I see in clang). Similarly, I don't expect this patch to do much on its own either because SimplifyCFG promptly drops the flags when converting to select on a minimal example like: https://bugs.llvm.org/show_bug.cgi?id=39535 But once we have this plumbing in place, we should be able to wire up the FMF propagation and start solving cases like that. The change to RecurrenceDescriptor::AddReductionVar() is required to prevent a regression in a LoopVectorize test. We are intersecting the FMF of any FPMathOperator there, so if a phi is not properly annotated, new math instructions may not be either. Once we fix the propagation in SimplifyCFG, it may be safe to remove that hack. Differential Revision: https://reviews.llvm.org/D67564 llvm-svn: 372866
Diffstat (limited to 'llvm/lib/CodeGen/MachinePostDominators.cpp')
0 files changed, 0 insertions, 0 deletions