[llvm] ValueTracking: Merge fcmpImpliesClass and fcmpToClassTest (PR #66522)

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 21 09:29:07 PST 2024


================
@@ -2641,8 +2641,8 @@ define float @assume_false_smallest_normal(float %arg) {
 }
 
 define float @clamp_false_nan(float %arg) {
-; CHECK-LABEL: define float @clamp_false_nan(
-; CHECK-SAME: float returned [[ARG:%.*]]) #[[ATTR2]] {
+; CHECK-LABEL: define nofpclass(nan inf nzero sub norm) float @clamp_false_nan(
+; CHECK-SAME: float returned nofpclass(nan inf nzero sub norm) [[ARG:%.*]]) #[[ATTR2]] {
----------------
jdoerfert wrote:

I know what is wrong, did https://github.com/jdoerfert/llvm-project/tree/fpclass_fixes help?
It is just very intrusive and you probably want to populate the switch.

https://github.com/llvm/llvm-project/pull/66522


More information about the llvm-commits mailing list