[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