Changeset 08d5507b for src/Parser/DeclarationNode.cc

Timestamp:

Mar 14, 2017, 11:29:34 AM (8 years ago)

Author:

Peter A. Buhr <pabuhr@…>

Branches:

ADT, aaron-thesis, arm-eh, ast-experimental, cleanup-dtors, deferred_resn, demangler, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, with_gc

Children:

4da6a6c, e496303

Parents:

68ac32e

Message:

change type of function specifier and storage class to bit fields

File:

• src/Parser/DeclarationNode.cc (modified) (12 diffs)

src/Parser/DeclarationNode.cc

@@ -10 +10 @@
 // Created On       : Sat May 16 12:34:05 2015
 // Last Modified By : Peter A. Buhr
-// Last Modified On : Tue Mar  7 17:28:56 2017
-// Update Count     : 937
+// Last Modified On : Tue Mar 14 10:19:38 2017
+// Update Count     : 964
 //
 
@@ -90 +90 @@
         newnode->type = maybeClone( type );
         newnode->storageClasses = storageClasses;
+        newnode->funcSpecs = funcSpecs;
         newnode->bitfieldWidth = maybeClone( bitfieldWidth );
-        newnode->funcSpecs = funcSpecs;
         newnode->enumeratorValue.reset( maybeClone( enumeratorValue.get() ) );
         newnode->hasEllipsis = hasEllipsis;
@@ -117 +117 @@
 
 void DeclarationNode::print_StorageClass( std::ostream & output, StorageClasses storageClasses ) {
-        if ( storageClasses.any() ) {                                                           // function specifiers?
+        if ( storageClasses.val != 0 ) {                                        // storage classes ?
                 for ( unsigned int i = 0; i < DeclarationNode::NoStorageClass; i += 1 ) {
                         if ( storageClasses[i] ) {
@@ -127 +127 @@
 
 void DeclarationNode::print_FuncSpec( std::ostream & output, DeclarationNode::FuncSpecifiers funcSpec ) {
-        if ( funcSpec.any() ) {                                                         // function specifiers?
+        if ( funcSpec.val != 0 ) {                                                      // function specifiers ?
                 for ( unsigned int i = 0; i < DeclarationNode::NoFuncSpecifier; i += 1 ) {
                         if ( funcSpec[i] ) {
@@ -202 +202 @@
 
 
-DeclarationNode * DeclarationNode::newStorageClass( DeclarationNode::StorageClass sc ) {
-        DeclarationNode * newnode = new DeclarationNode;
-        newnode->storageClasses[ sc ] = true;
+DeclarationNode * DeclarationNode::newStorageClass( DeclarationNode::StorageClasses sc ) {
+        DeclarationNode * newnode = new DeclarationNode;
+        newnode->storageClasses = sc;
         return newnode;
 } // DeclarationNode::newStorageClass
 
-DeclarationNode * DeclarationNode::newFuncSpecifier( DeclarationNode::FuncSpecifier fs ) {
-        DeclarationNode * newnode = new DeclarationNode;
-        newnode->funcSpecs[ fs ] = true;
+DeclarationNode * DeclarationNode::newFuncSpecifier( DeclarationNode::FuncSpecifiers fs ) {
+        DeclarationNode * newnode = new DeclarationNode;
+        newnode->funcSpecs = fs;
         return newnode;
 } // DeclarationNode::newFuncSpecifier
@@ -459 +459 @@
         const TypeData::TypeQualifiers qsrc = src->typeQualifiers, qdst = dst->typeQualifiers; // optimization
 
-        if ( (qsrc & qdst).any() ) {                                            // common qualifier ?
-                for ( unsigned int i = 0; i < NoTypeQualifier; i += 1 ) { // find common qualifiers
+        if ( (qsrc & qdst).any() ) {                                            // duplicates ?
+                for ( unsigned int i = 0; i < NoTypeQualifier; i += 1 ) { // find duplicates
                         if ( qsrc[i] && qdst[i] ) {
                                 appendError( error, string( "duplicate " ) + DeclarationNode::typeQualifierNames[i] );
@@ -469 +469 @@
 
 void DeclarationNode::checkSpecifiers( DeclarationNode * src ) {
-        if ( (funcSpecs & src->funcSpecs).any() ) {                     // common specifier ?
-                for ( unsigned int i = 0; i < NoFuncSpecifier; i += 1 ) { // find common specifier
+        if ( (funcSpecs.val & src->funcSpecs.val) != 0 ) {                // duplicates ?
+                for ( unsigned int i = 0; i < NoFuncSpecifier; i += 1 ) { // find duplicates
                         if ( funcSpecs[i] && src->funcSpecs[i] ) {
                                 appendError( error, string( "duplicate " ) + DeclarationNode::funcSpecifierNames[i] );
@@ -477 +477 @@
         } // if
 
-        if ( storageClasses != 0 && src->storageClasses != 0 ) { // any reason to check ?
-                if ( (storageClasses & src->storageClasses).any() ) { // duplicates ?
+        if ( storageClasses.val != 0 && src->storageClasses.val != 0 ) { // any reason to check ?
+                if ( (storageClasses.val & src->storageClasses.val ) != 0 ) { // duplicates ?
                         for ( unsigned int i = 0; i < NoStorageClass; i += 1 ) { // find duplicates
                                 if ( storageClasses[i] && src->storageClasses[i] ) {
@@ -485 +485 @@
                         } // for
                         // src is the new item being added and has a single bit
-                } else if ( ! src->storageClasses[ Threadlocal ] ) { // conflict ?
-                        appendError( error, string( "conflicting " ) + storageClassNames[ffs( storageClasses.to_ulong() ) - 1] +
-                                                 " & " + storageClassNames[ffs( src->storageClasses.to_ulong() ) - 1] );
-                        src->storageClasses.reset();                            // FIX to preserve invariant of one basic storage specifier
+                } else if ( ! src->storageClasses.is_threadlocal ) { // conflict ?
+                        appendError( error, string( "conflicting " ) + storageClassNames[ffs( storageClasses.val ) - 1] +
+                                                 " & " + storageClassNames[ffs( src->storageClasses.val ) - 1] );
+                        src->storageClasses.val = 0;                            // FIX to preserve invariant of one basic storage specifier
                 } // if
         } // if
@@ -496 +496 @@
 
 DeclarationNode * DeclarationNode::copySpecifiers( DeclarationNode * q ) {
-        funcSpecs = funcSpecs | q->funcSpecs;
-        storageClasses = storageClasses | q->storageClasses;
+        funcSpecs.val = funcSpecs.val | q->funcSpecs.val;
+        storageClasses.val = storageClasses.val | q->storageClasses.val;
 
         for ( Attribute *attr: reverseIterate( q->attributes ) ) {
@@ -1058 +1058 @@
                 //    inline _Noreturn int g( int i );  // allowed
                 //    inline _Noreturn int i;                   // disallowed
-                if ( type->kind != TypeData::Function && funcSpecs.any() ) {
+                if ( type->kind != TypeData::Function && funcSpecs.val != 0 ) {
                         throw SemanticError( "invalid function specifier for ", this );
                 } // if
@@ -1068 +1068 @@
         //    inlne _Noreturn struct S { ... };         // disallowed
         //    inlne _Noreturn enum   E { ... };         // disallowed
-        if ( funcSpecs.any() ) {
+        if ( funcSpecs.val != 0 ) {
                 throw SemanticError( "invalid function specifier for ", this );
         } // if