[PATCH] D149715: IR: Add llvm.frexp intrinsic

Tue May 23 06:47:40 PDT 2023

arsenm added a comment.

In D149715#4364321 <https://reviews.llvm.org/D149715#4364321>, @arsenm wrote:

> In D149715#4364233 <https://reviews.llvm.org/D149715#4364233>, @kpn wrote:
>
>> Would a constrained version of this be needed?
>
> Yes, I think so. Feels like it shouldn't be necessary but on my system at least the library function returns 0 with DAZ enabled so that implies a dependence on the FP mode

Actually the system one works as you would expect with or without DAZ. The musl implementation (and the expansion here) usage of fmul is what breaks denormal handling

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

https://reviews.llvm.org/D149715