[PATCH] D85961: [Fixed Point] Add floating point methods to APFixedPoint.

[John McCall via Phabricator via llvm-commits]

rjmccall added a comment.

> Would it be completely unthinkable to "promote" calculations to a larger FP type (both here and in codegen) if the exponent bits are insufficient to hold the necessary scaling?

You could have the same problem with float/bfloat and a 128-bit fixed-point type, right?  128-bit integer types aren't at all ridiculous.  I think this is something you should accommodate properly in your design; I don't think it's that problematic.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D85961