[PATCH] D26817: [APFloat] Implement PPCDoubleDouble add and subtract.

Eric Christopher via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 9 19:36:46 PST 2016


echristo added a reviewer: scanon.
echristo added a comment.

Couple of small inline nits. Willing to go with it on the implementation details.

Adding Steve in case he wishes to comment.

-eric



================
Comment at: llvm/lib/Support/APFloat.cpp:4069
+  }
+  llvm_unreachable("");
+}
----------------
kbarton wrote:
> Why is this an empty string?
Still an empty string :)


================
Comment at: llvm/lib/Support/APFloat.cpp:4167
+}
+
 } // End llvm namespace
----------------
Since you're adding a Print you might want to add a dump as well. (And while I realize it's against how the class currently works Print->print please).


https://reviews.llvm.org/D26817





More information about the llvm-commits mailing list