[PATCH] D86632: [Fixed Point] Add codegen for conversion between fixed-point and floating point.

[John McCall via Phabricator via llvm-commits]

rjmccall added a comment.

I had a question in the other patch about whether you should just have a method on FixedPointSemantics that returns the unscaled semantics (since FixedPointSemantics is totally capable of expressing integer types), which would let fitsInFloatSemantics have more obvious semantics.  That would affect this as well.  But otherwise I have no concerns.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86632