[PATCH] D136097: InstSimplify: Fold fdiv nnan nsz x, 0 -> inf

Mon Oct 17 12:10:59 PDT 2022

arsenm added a comment.

In D136097#3862946 <https://reviews.llvm.org/D136097#3862946>, @spatel wrote:

> In D136097#3862928 <https://reviews.llvm.org/D136097#3862928>, @kpn wrote:
>
>> It looks like by 754 sec 7.3 a (-x)/(+0) == -Inf, but the nsz is documented as saying the sign of the result is "insignificant", so I think we're good here.
>>
>> LGTM
>
> "nsz" means the sign of a **zero** result is insignificant, not any result. -42.0 / 0.0 should be -Inf, not +Inf?

That's what I would expect. alive2 has been giving me some nsz results that puzzle me https://alive2.llvm.org/ce/z/v-9wqk

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D136097/new/

https://reviews.llvm.org/D136097