Changes in / [7bd712d:637568b]

File:

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

src/InitTweak/GenInit.cc

@@ -16 +16 @@
 #include <stack>
 #include <list>
-
+#include "GenInit.h"
 #include "InitTweak.h"
-#include "GenInit.h"
-
-#include "Common/PassVisitor.h"
-
 #include "SynTree/Declaration.h"
+#include "SynTree/Type.h"
 #include "SynTree/Expression.h"
+#include "SynTree/Statement.h"
 #include "SynTree/Initializer.h"
 #include "SynTree/Mutator.h"
-#include "SynTree/Statement.h"
-#include "SynTree/Type.h"
-
 #include "SymTab/Autogen.h"
 #include "SymTab/Mangler.h"
-
+#include "GenPoly/PolyMutator.h"
 #include "GenPoly/DeclMutator.h"
-#include "GenPoly/PolyMutator.h"
 #include "GenPoly/ScopedSet.h"
-
 #include "ResolvExpr/typeops.h"
 
@@ -44 +37 @@
         }
 
-        class ReturnFixer {
+        class ReturnFixer final : public GenPoly::PolyMutator {
           public:
                 /// consistently allocates a temporary variable for the return value
@@ -51 +44 @@
                 static void makeReturnTemp( std::list< Declaration * > &translationUnit );
 
-                void premutate( FunctionDecl *functionDecl );
-                void premutate( ReturnStmt * returnStmt );
-
-                at_cleanup_t at_cleanup;
-                std::list< Statement * > stmtsToAddBefore;
+                typedef GenPoly::PolyMutator Parent;
+                using Parent::mutate;
+                virtual DeclarationWithType * mutate( FunctionDecl *functionDecl ) override;
+                virtual Statement * mutate( ReturnStmt * returnStmt ) override;
 
           protected:
@@ -137 +129 @@
 
         void ReturnFixer::makeReturnTemp( std::list< Declaration * > & translationUnit ) {
-                PassVisitor<ReturnFixer> fixer;
+                ReturnFixer fixer;
                 mutateAll( translationUnit, fixer );
         }
 
-        void ReturnFixer::premutate( ReturnStmt *returnStmt ) {
+        Statement *ReturnFixer::mutate( ReturnStmt *returnStmt ) {
                 std::list< DeclarationWithType * > & returnVals = ftype->get_returnVals();
                 assert( returnVals.size() == 0 || returnVals.size() == 1 );
@@ -152 +144 @@
                         construct->get_args().push_back( new AddressExpr( new VariableExpr( returnVals.front() ) ) );
                         construct->get_args().push_back( returnStmt->get_expr() );
-                        stmtsToAddBefore.push_back(new ExprStmt(noLabels, construct));
+                        stmtsToAdd.push_back(new ExprStmt(noLabels, construct));
 
                         // return the retVal object
                         returnStmt->set_expr( new VariableExpr( returnVals.front() ) );
                 } // if
-        }
-
-        void ReturnFixer::premutate( FunctionDecl *functionDecl ) {
-                GuardValue( this, ftype );
-                GuardValue( this, funcName );
+                return returnStmt;
+        }
+
+        DeclarationWithType* ReturnFixer::mutate( FunctionDecl *functionDecl ) {
+                ValueGuard< FunctionType * > oldFtype( ftype );
+                ValueGuard< std::string > oldFuncName( funcName );
 
                 ftype = functionDecl->get_functionType();
                 funcName = functionDecl->get_name();
+                return Parent::mutate( functionDecl );
         }