]> granicus.if.org Git - clang/commitdiff
Use the same type for adjacent bit field members.
authorAkira Hatanaka <ahatanaka@apple.com>
Thu, 30 Jun 2016 00:07:17 +0000 (00:07 +0000)
committerAkira Hatanaka <ahatanaka@apple.com>
Thu, 30 Jun 2016 00:07:17 +0000 (00:07 +0000)
MSVC doesn't pack the bit field members if different types are used.
This came up in a patch review.

http://lists.llvm.org/pipermail/cfe-commits/Week-of-Mon-20160627/163107.html

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@274190 91177308-0d34-0410-b5e6-96231b3b80d8

lib/AST/ExprConstant.cpp

index e44cb1d23cf93461f560388a65e2f0997b240960..48c0695a1da29fd37107808efbcd496fe7941dca 100644 (file)
@@ -158,13 +158,13 @@ namespace {
     /// True if the subobject was named in a manner not supported by C++11. Such
     /// lvalues can still be folded, but they are not core constant expressions
     /// and we cannot perform lvalue-to-rvalue conversions on them.
-    bool Invalid : 1;
+    unsigned Invalid : 1;
 
     /// Is this a pointer one past the end of an object?
-    bool IsOnePastTheEnd : 1;
+    unsigned IsOnePastTheEnd : 1;
 
     /// Indicator of whether the most-derived object is an array element.
-    bool MostDerivedIsArrayElement : 1;
+    unsigned MostDerivedIsArrayElement : 1;
 
     /// The length of the path to the most-derived object of which this is a
     /// subobject.
@@ -1058,7 +1058,7 @@ namespace {
   struct LValue {
     APValue::LValueBase Base;
     CharUnits Offset;
-    bool InvalidBase : 1;
+    unsigned InvalidBase : 1;
     unsigned CallIndex : 31;
     SubobjectDesignator Designator;