[PATCH] D69275: Add constrained int->FP intrinsics

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 6 08:48:05 PST 2019


craig.topper added a comment.

In D69275#1772541 <https://reviews.llvm.org/D69275#1772541>, @kpn wrote:

> I'm still working on this ticket daily! I'm trying to merge the two vector unrolling functions like Ulrich suggested. But I ran into problems that lead me to think we may have a serious issue lurking that we'll need to fix. That's what I've been working on: trying to understand the issue and see if it needs further investigation.
>
> If you are in a hurry then you could have sent me an email and I would have uploaded the diffs I've got without further investigation.
>
> I'm leaving attached the comments on my work that I've been adding but haven't submitted until now.


Sorry. I should have sent a mail. If you want to upload over my last patch, I can merge my X86 changes into it.

What’s the serious issue you’ve encountered?


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

https://reviews.llvm.org/D69275





More information about the llvm-commits mailing list