Changeset 22bc276 for src/CodeGen

Timestamp:

Sep 21, 2017, 3:23:33 PM (7 years ago)

Author:

Rob Schluntz <rschlunt@…>

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:

05807e9, 29bc63e

Parents:

e4d6335

git-author:

Rob Schluntz <rschlunt@…> (09/21/17 15:19:47)

git-committer:

Rob Schluntz <rschlunt@…> (09/21/17 15:23:33)

Message:

Convert HoistArrayDimension? to PassVisitor?, cleanup in InitTweak? and CodeGen?

File:

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

src/CodeGen/CodeGenerator.cc

@@ -443 +443 @@
         void CodeGenerator::postvisit( UntypedExpr * untypedExpr ) {
                 extension( untypedExpr );
-                if ( NameExpr * nameExpr = dynamic_cast< NameExpr* >( untypedExpr->get_function() ) ) {
+                if ( NameExpr * nameExpr = dynamic_cast< NameExpr* >( untypedExpr->function ) ) {
                         OperatorInfo opInfo;
-                        if ( operatorLookup( nameExpr->get_name(), opInfo ) ) {
-                                std::list< Expression* >::iterator arg = untypedExpr->get_args().begin();
+                        if ( operatorLookup( nameExpr->name, opInfo ) ) {
+                                std::list< Expression* >::iterator arg = untypedExpr->args.begin();
                                 switch ( opInfo.type ) {
                                   case OT_INDEX:
-                                        assert( untypedExpr->get_args().size() == 2 );
+                                        assert( untypedExpr->args.size() == 2 );
                                         (*arg++)->accept( *visitor );
                                         output << "[";
@@ -461 +461 @@
                                   case OT_CTOR:
                                   case OT_DTOR:
-                                        if ( untypedExpr->get_args().size() == 1 ) {
+                                        if ( untypedExpr->args.size() == 1 ) {
                                                 // the expression fed into a single parameter constructor or destructor may contain side
                                                 // effects, so must still output this expression
@@ -480 +480 @@
                                                 (*arg++)->accept( *visitor );
                                                 output << opInfo.symbol << "{ ";
-                                                genCommaList( arg, untypedExpr->get_args().end() );
+                                                genCommaList( arg, untypedExpr->args.end() );
                                                 output << "}) /* " << opInfo.inputName << " */";
                                         } // if
@@ -488 +488 @@
                                   case OT_PREFIXASSIGN:
                                   case OT_LABELADDRESS:
-                                        assert( untypedExpr->get_args().size() == 1 );
+                                        assert( untypedExpr->args.size() == 1 );
                                         output << "(";
                                         output << opInfo.symbol;
@@ -497 +497 @@
                                   case OT_POSTFIX:
                                   case OT_POSTFIXASSIGN:
-                                        assert( untypedExpr->get_args().size() == 1 );
+                                        assert( untypedExpr->args.size() == 1 );
                                         (*arg)->accept( *visitor );
                                         output << opInfo.symbol;
@@ -504 +504 @@
                                   case OT_INFIX:
                                   case OT_INFIXASSIGN:
-                                        assert( untypedExpr->get_args().size() == 2 );
+                                        assert( untypedExpr->args.size() == 2 );
                                         output << "(";
                                         (*arg++)->accept( *visitor );
@@ -517 +517 @@
                                 } // switch
                         } else {
-                                if ( nameExpr->get_name() == "..." ) { // case V1 ... V2 or case V1~V2
-                                        assert( untypedExpr->get_args().size() == 2 );
-                                        (*untypedExpr->get_args().begin())->accept( *visitor );
-                                        output << " ... ";
-                                        (*--untypedExpr->get_args().end())->accept( *visitor );
-                                } else {                                                                // builtin routines
-                                        nameExpr->accept( *visitor );
-                                        output << "(";
-                                        genCommaList( untypedExpr->get_args().begin(), untypedExpr->get_args().end() );
-                                        output << ")";
-                                } // if
+                                // builtin routines
+                                nameExpr->accept( *visitor );
+                                output << "(";
+                                genCommaList( untypedExpr->args.begin(), untypedExpr->args.end() );
+                                output << ")";
                         } // if
                 } else {
-                        untypedExpr->get_function()->accept( *visitor );
+                        untypedExpr->function->accept( *visitor );
                         output << "(";
-                        genCommaList( untypedExpr->get_args().begin(), untypedExpr->get_args().end() );
+                        genCommaList( untypedExpr->args.begin(), untypedExpr->args.end() );
                         output << ")";
                 } // if
@@ -538 +532 @@
 
         void CodeGenerator::postvisit( RangeExpr * rangeExpr ) {
-                rangeExpr->get_low()->accept( *visitor );
+                rangeExpr->low->accept( *visitor );
                 output << " ... ";
-                rangeExpr->get_high()->accept( *visitor );
+                rangeExpr->high->accept( *visitor );
         }