[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