[PATCH] D157108: APFloat: Add getExactLog2

[Joshua Cranmer via Phabricator via llvm-commits]

jcranmer-intel added inline comments.


================
Comment at: llvm/lib/Support/APFloat.cpp:4306
+  for (int i = 0; i < PartCount; ++i) {
+    PopCount += llvm::popcount(Parts[i]);
+    if (PopCount > 1)
----------------
I would like to see a test that exercises the case of two different parts each having popcount = 1. Testing `next(false)` on the existing subnormal checks should do the trick, I think.


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

https://reviews.llvm.org/D157108