diff options
author | Sanjay Patel <spatel@rotateright.com> | 2019-05-02 14:47:59 +0000 |
---|---|---|
committer | Sanjay Patel <spatel@rotateright.com> | 2019-05-02 14:47:59 +0000 |
commit | 284472be6da3353d81dfd25b1ac4218e852d1e5f (patch) | |
tree | cb2d7ce58399facb3870f7cfcfe648fdf1f35f9b /llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp | |
parent | 8e6bf88cf763e7e5baa9b9745fe4616298040e4c (diff) | |
download | bcm5719-llvm-284472be6da3353d81dfd25b1ac4218e852d1e5f.tar.gz bcm5719-llvm-284472be6da3353d81dfd25b1ac4218e852d1e5f.zip |
[SelectionDAG] remove constant folding limitations based on FP exceptions
We don't have FP exception limits in the IR constant folder for the binops (apart from strict ops),
so it does not make sense to have them here in the DAG either. Nothing else in the backend tries
to preserve exceptions (again outside of strict ops), so I don't see how this could have ever
worked for real code that cares about FP exceptions.
There are still cases (examples: unary opcodes in SDAG, FMA in IR) where we are trying (at least
partially) to preserve exceptions without even asking if the target supports FP exceptions. Those
should be corrected in subsequent patches.
Real support for FP exceptions requires several changes to handle the constrained/strict FP ops.
Differential Revision: https://reviews.llvm.org/D61331
llvm-svn: 359791
Diffstat (limited to 'llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp')
-rw-r--r-- | llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp b/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp index 3d805344483..41c82ded69c 100644 --- a/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp +++ b/llvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp @@ -45,9 +45,6 @@ WebAssemblyTargetLowering::WebAssemblyTargetLowering( setBooleanContents(ZeroOrOneBooleanContent); // Except in SIMD vectors setBooleanVectorContents(ZeroOrNegativeOneBooleanContent); - // WebAssembly does not produce floating-point exceptions on normal floating - // point operations. - setHasFloatingPointExceptions(false); // We don't know the microarchitecture here, so just reduce register pressure. setSchedulingPreference(Sched::RegPressure); // Tell ISel that we have a stack pointer. |