[FPEnv] Invert sense of MIFlag::FPExcept flag
authorUlrich Weigand <ulrich.weigand@de.ibm.com>
Fri, 10 Jan 2020 14:31:10 +0000 (15:31 +0100)
committerUlrich Weigand <ulrich.weigand@de.ibm.com>
Fri, 10 Jan 2020 14:34:50 +0000 (15:34 +0100)
commitf0fd11df7d5488e2747f26a3bfcf62459fee54ad
tree3930d42159159d66a3efeab601af1b3928d9e8a8
parent3772ea9dd9368cfdc73595854c143bc3f16a5ade
[FPEnv] Invert sense of MIFlag::FPExcept flag

In D71841 we inverted the sense of the SDNode-level flag to ensure all nodes
default to potentially raising FP exceptions unless otherwise specified --
i.e. if we forget to propagate the flag somewhere, the effect is now only
lost performance, not incorrect code.

However, the related flag at the MI level still defaults to nodes not raising
FP exceptions unless otherwise specified. To be fully on the (conservatively)
safe side, we should invert that flag as well.

This patch does so by replacing MIFlag::FPExcept with MIFlag::NoFPExcept.
(Note that this does also introduce an incompatible change in the MIR format.)

Reviewed By: craig.topper

Differential Revision: https://reviews.llvm.org/D72466
13 files changed:
llvm/include/llvm/CodeGen/MachineInstr.h
llvm/lib/CodeGen/MIRParser/MILexer.cpp
llvm/lib/CodeGen/MIRParser/MILexer.h
llvm/lib/CodeGen/MIRParser/MIParser.cpp
llvm/lib/CodeGen/MIRPrinter.cpp
llvm/lib/CodeGen/MachineInstr.cpp
llvm/lib/CodeGen/SelectionDAG/InstrEmitter.cpp
llvm/lib/Target/SystemZ/SystemZElimCompare.cpp
llvm/test/CodeGen/X86/fast-isel-select-sse.ll
llvm/test/CodeGen/X86/fp-intrinsics-flags-x86_64.ll
llvm/test/CodeGen/X86/fp-intrinsics-flags.ll
llvm/test/CodeGen/X86/sqrt-fastmath-mir.ll
llvm/test/CodeGen/X86/vector-constrained-fp-intrinsics-flags.ll