Changeset de8d7fb1

Timestamp:

Oct 19, 2019, 11:58:30 PM (5 years ago)

Author:

Peter A. Buhr <pabuhr@…>

Branches:

ADT, arm-eh, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

16a18f3

Parents:

ecc0fec

Message:

do not generate anonymous name for bit field with empty declarator

File:

• src/CodeGen/CodeGenerator.cc (modified) (2 diffs)

src/CodeGen/CodeGenerator.cc

@@ -9 +9 @@
 // Author           : Richard C. Bilson
 // Created On       : Mon May 18 07:44:20 2015
-// Last Modified By : Andrew Beach
-// Last Modified On : Thr May  2 10:47:00 2019
-// Update Count     : 497
+// Last Modified By : Peter A. Buhr
+// Last Modified On : Sat Oct 19 19:30:38 2019
+// Update Count     : 506
 //
 #include "CodeGenerator.h"
@@ -198 +198 @@
                 // deleted decls should never be used, so don't print them
                 if ( objectDecl->isDeleted && options.genC ) return;
-                if (objectDecl->get_name().empty() && options.genC ) {
+
+                // gcc allows an empty declarator (no name) for bit-fields and C states: 6.7.2.1 Structure and union specifiers,
+                // point 4, page 113: If the (bit field) value is zero, the declaration shall have no declarator.  For anything
+                // else, the anonymous name refers to the anonymous object for plan9 inheritance.
+                if ( objectDecl->get_name().empty() && options.genC && ! objectDecl->get_bitfieldWidth() ) {
                         // only generate an anonymous name when generating C code, otherwise it clutters the output too much
                         static UniqueName name = { "__anonymous_object" };
                         objectDecl->set_name( name.newName() );
-            // Stops unused parameter warnings.
-            if ( options.anonymousUnused ) {
-                objectDecl->attributes.push_back( new Attribute( "unused" ) );
-            }
+                        // Stops unused parameter warnings.
+                        if ( options.anonymousUnused ) {
+                                objectDecl->attributes.push_back( new Attribute( "unused" ) );
+                        }
                 }