[PATCH] D55460: APFloat: allow 64-bit of payload

JF Bastien via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 7 15:56:48 PST 2018


jfb added a comment.

Testing-wise, this isn't tested in APFloatTest.cpp, so I'm keeping the status-quo.
The only place where it's used in an odd way is for the floating-literals.s test (which tries to match the GNU as behavior). The test makes me uneasy because it's using `~0` to create a `.single` NaN... and I have no idea what's the right behavior for `.double`... This is better fixed in a follow-up.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D55460





More information about the llvm-commits mailing list