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

Wed Jun 7 02:36:41 PDT 2023

sepavloff added a comment.

LGTM.

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

> In D149715#4381350 <https://reviews.llvm.org/D149715#4381350>, @sepavloff wrote:
>
>> There is no code for WidenVecRes_FREXP and WidenVecOp_FREXP?
>
> I also don't know of a use case for them, so no way to test it

In the test X86/llvm.frexp.ll there is a note `; FIXME: Widen vector result`, probably this the use case. I think the support could be implemented in another patch, this one is already large.

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

https://reviews.llvm.org/D149715