Changeset 37024fd for src/InitTweak/FixGlobalInit.cc

Timestamp:

May 6, 2016, 3:01:28 PM (8 years ago)

Author:

Rob Schluntz <rschlunt@…>

Branches:

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

Children:

03e5d14

Parents:

9a92216

Message:

cleanup global init code, don't need to disambiguate with libcfa prefix

File:

• src/InitTweak/FixGlobalInit.cc (modified) (5 diffs)

src/InitTweak/FixGlobalInit.cc

@@ -10 +10 @@
 // Created On       : Mon May 04 15:14:56 2016
 // Last Modified By : Rob Schluntz
-// Last Modified On : Fri May 06 13:51:00 2016
+// Last Modified On : Fri May 06 14:59:26 2016
 // Update Count     : 2
 //
@@ -30 +30 @@
         class GlobalFixer : public Visitor {
           public:
-                GlobalFixer();
+                GlobalFixer( const std::string & name );
 
                 virtual void visit( ObjectDecl *objDecl );
@@ -41 +41 @@
 
                 UniqueName tempNamer;
-                CompoundStmt * block;
+                FunctionDecl * initFunction;
         };
 
@@ -88 +88 @@
 
         void fixGlobalInit( std::list< Declaration * > & translationUnit, const std::string & name ) {
-                GlobalFixer fixer;
+                GlobalFixer fixer( name );
                 acceptAll( translationUnit, fixer );
-                // attribute only appears on the forward declaration, so need to make two function decls:
-                // one with the body, one with the attribute.
-                FunctionDecl * initFunction = new FunctionDecl( initName( name ), DeclarationNode::NoStorageClass, LinkageSpec::C, new FunctionType( Type::Qualifiers(), false ), 0, false, false );
-                FunctionDecl * forward = initFunction->clone();
-                forward->set_attribute( FunctionDecl::Constructor );
-                initFunction->set_statements( fixer.block );
-                translationUnit.push_back( forward );
-                translationUnit.push_back( initFunction );
+                translationUnit.push_back( fixer.initFunction );
         }
 
@@ -109 +102 @@
   }
 
-        GlobalFixer::GlobalFixer() : tempNamer( "_global_init" ), block( new CompoundStmt( noLabels ) ) {
+        GlobalFixer::GlobalFixer( const std::string & name ) : tempNamer( "_global_init" ) {
+                initFunction = new FunctionDecl( initName( name ), DeclarationNode::Static, LinkageSpec::C, new FunctionType( Type::Qualifiers(), false ), new CompoundStmt( noLabels ), false, false, FunctionDecl::Constructor );
         }
 
         void GlobalFixer::visit( ObjectDecl *objDecl ) {
-                std::list< Statement * > & statements = block->get_kids();
+                std::list< Statement * > & statements = initFunction->get_statements()->get_kids();
 
                 if ( objDecl->get_init() == NULL ) return;