Revert "ValueTracking: Handle freeze in computeKnownFPClass"
authorNikita Popov <npopov@redhat.com>
Mon, 17 Apr 2023 10:52:58 +0000 (12:52 +0200)
committerNikita Popov <npopov@redhat.com>
Mon, 17 Apr 2023 10:59:23 +0000 (12:59 +0200)
This reverts commit 2c8d0048f03d054f13909a26f959ef95b2a0a4de.

This is incorrect: computeKnownFPClass() is only known up to
poison, and freeze poison may have any FP class.

llvm/lib/Analysis/ValueTracking.cpp
llvm/test/Transforms/Attributor/nofpclass.ll

index eae6261..2578957 100644 (file)
@@ -4635,8 +4635,7 @@ void computeKnownFPClass(const Value *V, const APInt &DemandedElts,
 
     break;
   }
-  case Instruction::ExtractValue:
-  case Instruction::Freeze: {
+  case Instruction::ExtractValue: {
     computeKnownFPClass(Op->getOperand(0), DemandedElts, InterestedClasses,
                         Known, Depth + 1, Q, TLI);
     break;
index 6ac9baf..52c9c2e 100644 (file)
@@ -884,27 +884,3 @@ define float @returned_extractvalue([4 x float] nofpclass(nan) %array) {
   %extract = extractvalue [4 x float] %array, 0
   ret float %extract
 }
-
-define float @return_nofpclass_freeze_nan_arg(float nofpclass(nan) %arg) {
-; CHECK: Function Attrs: nofree norecurse nosync nounwind willreturn memory(none)
-; CHECK-LABEL: define noundef nofpclass(nan) float @return_nofpclass_freeze_nan_arg
-; CHECK-SAME: (float nofpclass(nan) [[ARG:%.*]]) #[[ATTR2]] {
-; CHECK-NEXT:    [[FREEZE:%.*]] = freeze float [[ARG]]
-; CHECK-NEXT:    ret float [[FREEZE]]
-;
-  %freeze = freeze float %arg
-  ret float %freeze
-}
-
-define float @return_nofpclass_extractelement_freeze_pinf_arg(<2 x float> nofpclass(pinf) %arg) {
-; CHECK: Function Attrs: nofree norecurse nosync nounwind willreturn memory(none)
-; CHECK-LABEL: define noundef nofpclass(pinf) float @return_nofpclass_extractelement_freeze_pinf_arg
-; CHECK-SAME: (<2 x float> nofpclass(pinf) [[ARG:%.*]]) #[[ATTR2]] {
-; CHECK-NEXT:    [[FREEZE:%.*]] = freeze <2 x float> [[ARG]]
-; CHECK-NEXT:    [[ELT:%.*]] = extractelement <2 x float> [[FREEZE]], i32 0
-; CHECK-NEXT:    ret float [[ELT]]
-;
-  %freeze = freeze <2 x float> %arg
-  %elt = extractelement <2 x float> %freeze, i32 0
-  ret float %elt
-}