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

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 28 07:52:34 PST 2019


uweigand added a comment.

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

> I have not addressed the algorithm issues pointed out by Ulrich. So far I'm trying to just get the existing algorithms converted for strict use. Would it make sense to hold off on algorithm changes until another patch?


For me this would be fine as a first step, but we should at least add FIXME statements to those algorithms where we already know there are strict semantics problems.


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

https://reviews.llvm.org/D69275





More information about the llvm-commits mailing list