[PATCH] D67105: [TargetLowering] Fix another potential FPE in expandFP_TO_UINT

Andy Kaylor via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 4 12:26:06 PST 2019


andrew.w.kaylor added a comment.

I'm afraid we may have made a mess of this for X86 in D70214 <https://reviews.llvm.org/D70214>. We noticed the problem that you're fixing here, but I didn't realize you had a solution for it. I guess I hadn't looked at this patch. I'm sorry about that. Your solution looks good to me.


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

https://reviews.llvm.org/D67105





More information about the llvm-commits mailing list