[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