[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