[PATCH] D72794: [LegalizeDAG][Mips] Add an assert to protect a uint_to_fp implementation from double rounding. Add a i32->f32 uint_to_fp implementation that avoids this code.

Ulrich Weigand via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 15 11:46:51 PST 2020


uweigand added a comment.

In D72794#1822504 <https://reviews.llvm.org/D72794#1822504>, @uweigand wrote:

> Why do we need to copy this algorithm here?   Can't you simply add this case to TargetLowering::expandUINT_TO_FP instead?


Ah, nevermind -- I see you're talking about this in the summary ...


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

https://reviews.llvm.org/D72794





More information about the llvm-commits mailing list