[PATCH] D25977: [APFloat] Add DoubleAPFloat mode to APFloat. NFC.

Tim Shen via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 27 16:03:33 PDT 2016


timshen added a comment.

Added a workaround for old clang in r285358.



================
Comment at: llvm/include/llvm/ADT/APFloat.h:723
+    static_assert(std::is_same<T, IEEEFloat>::value ||
+                      std::is_same<T, DoubleAPFloat>::value,
+                  "");
----------------
hfinkel wrote:
> I think this would look better if you lined up the 'std::is_same's vertically. 
Clang-format has its opinion, but I don't really care about format.


Repository:
  rL LLVM

https://reviews.llvm.org/D25977





More information about the llvm-commits mailing list