Changes in src/CodeGen/CodeGenerator.cc [8e9cbb2:4b2589a]

File:

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

src/CodeGen/CodeGenerator.cc

@@ -10 +10 @@
 // Created On       : Mon May 18 07:44:20 2015
 // Last Modified By : Peter A. Buhr
-// Last Modified On : Mon Jul  4 20:38:32 2016
-// Update Count     : 300
+// Last Modified On : Thu Jun  9 13:21:00 2016
+// Update Count     : 256
 //
 
@@ -47 +47 @@
                         dynamic_cast< WhileStmt * >( stmt ) || dynamic_cast< ForStmt * > ( stmt ) || dynamic_cast< SwitchStmt *>( stmt );
         }
-
-        void CodeGenerator::extension( Expression *expr ) {
-                if ( expr->get_extension() ) {
-                        output << "__extension__ ";
-                } // if
-        } // extension
-
-        void CodeGenerator::extension( Declaration *decl ) {
-                if ( decl->get_extension() ) {
-                        output << "__extension__ ";
-                } // if
-        } // extension
 
         ostream & CodeGenerator::Indenter::operator()( ostream & output ) const {
@@ -122 +110 @@
         //*** Declarations
         void CodeGenerator::visit( FunctionDecl *functionDecl ) {
-                extension( functionDecl );
                 genAttributes( functionDecl->get_attributes() );
 
@@ -147 +134 @@
 
         void CodeGenerator::visit( ObjectDecl *objectDecl ) {
-                extension( objectDecl );
                 handleStorageClass( objectDecl );
                 output << genType( objectDecl->get_type(), mangleName( objectDecl ) );
@@ -184 +170 @@
 
         void CodeGenerator::visit( StructDecl *structDecl ) {
-                extension( structDecl );
                 output << "struct ";
                 handleAggregate( structDecl );
         }
 
-        void CodeGenerator::visit( UnionDecl *unionDecl ) {
-                extension( unionDecl );
+        void CodeGenerator::visit( UnionDecl *aggregateDecl ) {
                 output << "union ";
-                handleAggregate( unionDecl );
-        }
-
-        void CodeGenerator::visit( EnumDecl *enumDecl ) {
-                extension( enumDecl );
+                handleAggregate( aggregateDecl );
+        }
+
+        void CodeGenerator::visit( EnumDecl *aggDecl ) {
                 output << "enum ";
 
-                if ( enumDecl->get_name() != "" )
-                        output << enumDecl->get_name();
-
-                std::list< Declaration* > &memb = enumDecl->get_members();
+                if ( aggDecl->get_name() != "" )
+                        output << aggDecl->get_name();
+
+                std::list< Declaration* > &memb = aggDecl->get_members();
 
                 if ( ! memb.empty() ) {
@@ -225 +208 @@
         }
 
-        void CodeGenerator::visit( TraitDecl *traitDecl ) {}
+        void CodeGenerator::visit( TraitDecl *aggregateDecl ) {}
 
         void CodeGenerator::visit( TypedefDecl *typeDecl ) {
-                assert( false && "Typedefs are removed and substituted in earlier passes." );
-                //output << "typedef ";
-                //output << genType( typeDecl->get_base(), typeDecl->get_name() );
+                output << "typedef ";
+                output << genType( typeDecl->get_base(), typeDecl->get_name() );
         }
 
@@ -236 +218 @@
                 // really, we should mutate this into something that isn't a TypeDecl but that requires large-scale changes,
                 // still to be done
-                extension( typeDecl );
                 output << "extern unsigned long " << typeDecl->get_name();
                 if ( typeDecl->get_base() ) {
@@ -270 +251 @@
                 output << "{ ";
                 if ( init->begin_initializers() == init->end_initializers() ) {
-                        // illegal to leave initializer list empty for scalar initializers, but always legal to have 0
+                        // illegal to leave initializer list empty for scalar initializers,
+                        // but always legal to have 0
                         output << "0";
                 } else {
@@ -335 +317 @@
                                   case OT_DTOR:
                                         if ( applicationExpr->get_args().size() == 1 ) {
-                                                // the expression fed into a single parameter constructor or destructor may contain side
-                                                // effects, so must still output this expression
+                                                // the expression fed into a single parameter constructor or destructor
+                                                // may contain side effects, so must still output this expression
                                                 output << "(";
                                                 (*arg++)->accept( *this );
@@ -421 +403 @@
                                   case OT_DTOR:
                                         if ( untypedExpr->get_args().size() == 1 ) {
-                                                // the expression fed into a single parameter constructor or destructor may contain side
-                                                // effects, so must still output this expression
+                                                // the expression fed into a single parameter constructor or destructor
+                                                // may contain side effects, so must still output this expression
                                                 output << "(";
                                                 (*arg++)->accept( *this );
@@ -518 +500 @@
                         output << ")";
                 } else {
-                        // otherwise, the cast is to an lvalue type, so the cast should be dropped, since the result of a cast is
+                        // otherwise, the cast is to an lvalue type, so the cast
+                        // should be dropped, since the result of a cast is
                         // never an lvalue in C
                 }
@@ -563 +546 @@
 
         void CodeGenerator::visit( AlignofExpr *alignofExpr ) {
+                extension( alignofExpr );
                 // use GCC extension to avoid bumping std to C11
-                extension( alignofExpr );
                 output << "__alignof__(";
                 if ( alignofExpr->get_isType() ) {
@@ -575 +558 @@
 
         void CodeGenerator::visit( UntypedOffsetofExpr *offsetofExpr ) {
-                assert( false && "UntypedOffsetofExpr should not reach code generation." );
+                assert( false && "UntypedOffsetofExpr should not reach code generation" );
         }
 
         void CodeGenerator::visit( OffsetofExpr *offsetofExpr ) {
+                extension( offsetofExpr );
                 // use GCC builtin
                 output << "__builtin_offsetof(";
@@ -587 +571 @@
 
         void CodeGenerator::visit( OffsetPackExpr *offsetPackExpr ) {
-                assert( false && "OffsetPackExpr should not reach code generation." );
+                assert( false && "OffsetPackExpr should not reach code generation" );
         }
 
@@ -628 +612 @@
 
         void CodeGenerator::visit( AsmExpr *asmExpr ) {
+                extension( asmExpr );
                 if ( asmExpr->get_inout() ) {
                         output << "[ ";
@@ -792 +777 @@
 
         void CodeGenerator::visit( ForStmt *forStmt ) {
-                // initialization is always hoisted, so don't bother doing anything with that
+                // initialization is always hoisted, so don't
+                // bother doing anything with that
                 output << "for (;";