[PATCH] D25536: [APFloat] Make APFloat an interface class to the internal IEEEFloat. NFC.

Hal Finkel via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 18 12:09:22 PDT 2016


hfinkel added inline comments.


================
Comment at: llvm/include/llvm/ADT/APFloat.h:127
+// implementation classes.
+struct APFloatStatic {
   /// A signed type to represent a floating point numbers unbiased exponent.
----------------
I think this should be named APFloatBase (e.g. we have DenseMapBase, SmallPtrSetImplBase, StringMapEntryBase).


https://reviews.llvm.org/D25536





More information about the llvm-commits mailing list