HomePhabricator

[FPEnv] Invert sense of MIFlag::FPExcept flag

Authored by uweigand on Jan 10 2020, 6:31 AM.

Description

[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

Details

Committed
uweigandJan 10 2020, 6:34 AM
Reviewer
craig.topper
Differential Revision
D72466: [FPEnv][RFC] Invert sense of MIFlag::FPExcept flag
Parents
rG3772ea9dd936: [ARM][MVE] Tail predicate VMAX,VMAXA,VMIN,VMINA
Branches
Unknown
Tags
Unknown