[PATCH] Fix alignment issues in LLVM.
James Y Knight
jyknight at google.com
Fri Jun 5 06:52:20 PDT 2015
================
Comment at: include/llvm/Support/AlignOf.h:56-57
@@ -47,2 +55,4 @@
+#else
enum { Alignment =
static_cast<unsigned int>(sizeof(AlignmentCalcImpl<T>) - sizeof(T)) };
+#endif
----------------
majnemer wrote:
> Why not just do `enum : unsigned { Alignment = ... ;` ?
I hadn't tried until you suggested, but the answer is: because GCC still warns about it.
http://reviews.llvm.org/D10271
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list