Changes in / [61accc5:f184ca3]

Location:

src

Files:

• CodeGen/CodeGenerator.cc (modified) (1 diff)
• CodeGen/CodeGenerator.h (modified) (1 diff)
• Common/PassVisitor.h (modified) (3 diffs)
• Common/PassVisitor.impl.h (modified) (5 diffs)
• ResolvExpr/AlternativeFinder.cc (modified) (6 diffs)
• ResolvExpr/Resolver.h (modified) (1 diff)
• SynTree/Expression.cc (modified) (1 diff)
• SynTree/Expression.h (modified) (1 diff)
• SynTree/Mutator.h (modified) (1 diff)
• SynTree/SynTree.h (modified) (1 diff)
• SynTree/Visitor.h (modified) (1 diff)

src/CodeGen/CodeGenerator.cc

@@ -836 +836 @@
                 assertf( ! genC, "Deleted expressions should not reach code generation." );
                 expr->expr->accept( *visitor );
-        }
-
-        void CodeGenerator::postvisit( DefaultArgExpr * arg ) {
-                assertf( ! genC, "Default argument expressions should not reach code generation." );
-                arg->expr->accept( *visitor );
         }
 

src/CodeGen/CodeGenerator.h

@@ -94 +94 @@
                 void postvisit( ConstructorExpr * );
                 void postvisit( DeletedExpr * );
-                void postvisit( DefaultArgExpr * );
                 void postvisit( GenericExpr * );
 

src/Common/PassVisitor.h

@@ -125 +125 @@
         virtual void visit( InitExpr *  initExpr ) override final;
         virtual void visit( DeletedExpr *  delExpr ) override final;
-        virtual void visit( DefaultArgExpr * argExpr ) override final;
         virtual void visit( GenericExpr * genExpr ) override final;
 
@@ -225 +224 @@
         virtual Expression * mutate( InitExpr *  initExpr ) override final;
         virtual Expression * mutate( DeletedExpr *  delExpr ) override final;
-        virtual Expression * mutate( DefaultArgExpr * argExpr ) override final;
         virtual Expression * mutate( GenericExpr * genExpr ) override final;
 
@@ -260 +258 @@
 
 private:
-        bool inFunction = false;
-
         template<typename pass_t> friend void acceptAll( std::list< Declaration* > &decls, PassVisitor< pass_t >& visitor );
         template<typename pass_t> friend void mutateAll( std::list< Declaration* > &decls, PassVisitor< pass_t >& visitor );

src/Common/PassVisitor.impl.h

@@ -404 +404 @@
                         indexerAddId( &func );
                         maybeAccept_impl( node->type, *this );
-                        // function body needs to have the same scope as parameters - CompoundStmt will not enter
-                        // a new scope if inFunction is true
-                        ValueGuard< bool > oldInFunction( inFunction );
-                        inFunction = true;
                         maybeAccept_impl( node->statements, *this );
                         maybeAccept_impl( node->attributes, *this );
@@ -438 +434 @@
                         indexerAddId( &func );
                         maybeMutate_impl( node->type, *this );
-                        // function body needs to have the same scope as parameters - CompoundStmt will not enter
-                        // a new scope if inFunction is true
-                        ValueGuard< bool > oldInFunction( inFunction );
-                        inFunction = true;
                         maybeMutate_impl( node->statements, *this );
                         maybeMutate_impl( node->attributes, *this );
@@ -720 +712 @@
         VISIT_START( node );
         {
-                // do not enter a new scope if inFunction is true - needs to check old state before the assignment
-                ValueGuard< bool > oldInFunction( inFunction );
-                auto guard1 = makeFuncGuard( [this, &oldInFunction]() { if ( ! oldInFunction.old ) indexerScopeEnter(); }, [this, &oldInFunction]() { if ( ! oldInFunction.old ) indexerScopeLeave(); } );
+                auto guard1 = makeFuncGuard( [this]() { indexerScopeEnter(); }, [this]() { indexerScopeLeave(); } );
                 auto guard2 = makeFuncGuard( [this]() { call_beginScope();   }, [this]() { call_endScope();     } );
-                inFunction = false;
                 visitStatementList( node->kids );
         }
@@ -734 +723 @@
         MUTATE_START( node );
         {
-                // do not enter a new scope if inFunction is true - needs to check old state before the assignment
-                ValueGuard< bool > oldInFunction( inFunction );
-                auto guard1 = makeFuncGuard( [this, &oldInFunction]() { if ( ! oldInFunction.old ) indexerScopeEnter(); }, [this, &oldInFunction]() { if ( ! oldInFunction.old ) indexerScopeLeave(); } );
+                auto guard1 = makeFuncGuard( [this]() { indexerScopeEnter(); }, [this]() { indexerScopeLeave(); } );
                 auto guard2 = makeFuncGuard( [this]() { call_beginScope();   }, [this]() { call_endScope();     } );
-                inFunction = false;
                 mutateStatementList( node->kids );
         }
@@ -2088 +2074 @@
 template< typename pass_type >
 Expression * PassVisitor< pass_type >::mutate( DeletedExpr * node ) {
-        MUTATE_START( node );
-
-        indexerScopedMutate( node->env, *this );
-        indexerScopedMutate( node->result, *this );
-        maybeMutate_impl( node->expr, *this );
-
-        MUTATE_END( Expression, node );
-}
-
-//--------------------------------------------------------------------------
-// DefaultArgExpr
-template< typename pass_type >
-void PassVisitor< pass_type >::visit( DefaultArgExpr * node ) {
-        VISIT_START( node );
-
-        indexerScopedAccept( node->result, *this );
-        maybeAccept_impl( node->expr, *this );
-
-        VISIT_END( node );
-}
-
-template< typename pass_type >
-Expression * PassVisitor< pass_type >::mutate( DefaultArgExpr * node ) {
         MUTATE_START( node );
 

src/ResolvExpr/AlternativeFinder.cc

@@ -176 +176 @@
                                                 selected[ mangleName ] = current;
                                         } else if ( candidate->cost == mapPlace->second.candidate->cost ) {
-                                                // if one of the candidates contains a deleted identifier, can pick the other, since
-                                                // deleted expressions should not be ambiguous if there is another option that is at least as good
-                                                if ( findDeletedExpr( candidate->expr ) ) {
-                                                        // do nothing
-                                                        PRINT( std::cerr << "candidate is deleted" << std::endl; )
-                                                } else if ( findDeletedExpr( mapPlace->second.candidate->expr ) ) {
-                                                        PRINT( std::cerr << "current is deleted" << std::endl; )
-                                                        selected[ mangleName ] = current;
-                                                } else {
-                                                        PRINT(
-                                                                std::cerr << "marking ambiguous" << std::endl;
-                                                        )
-                                                        mapPlace->second.isAmbiguous = true;
-                                                }
+                                                PRINT(
+                                                        std::cerr << "marking ambiguous" << std::endl;
+                                                )
+                                                mapPlace->second.isAmbiguous = true;
                                         } else {
                                                 PRINT(
@@ -345 +335 @@
                 if ( ConstantExpr * constantExpr = dynamic_cast< ConstantExpr * >( member ) ) {
                         // get the value of the constant expression as an int, must be between 0 and the length of the tuple type to have meaning
-                        auto val = constantExpr->intValue();
+                        // xxx - this should be improved by memoizing the value of constant exprs
+                        // during parsing and reusing that information here.
+                        std::stringstream ss( constantExpr->get_constant()->get_value() );
+                        int val = 0;
                         std::string tmp;
-                        if ( val >= 0 && (unsigned long long)val < tupleType->size() ) {
+                        if ( ss >> val && ! (ss >> tmp) ) {
+                                if ( val >= 0 && (unsigned int)val < tupleType->size() ) {
+                                        alternatives.push_back( Alternative( new TupleIndexExpr( expr->clone(), val ), env, newCost ) );
+                                } // if
+                        } // if
+                } else if ( NameExpr * nameExpr = dynamic_cast< NameExpr * >( member ) ) {
+                        // xxx - temporary hack until 0/1 are int constants
+                        if ( nameExpr->get_name() == "0" || nameExpr->get_name() == "1" ) {
+                                std::stringstream ss( nameExpr->get_name() );
+                                int val;
+                                ss >> val;
                                 alternatives.push_back( Alternative( new TupleIndexExpr( expr->clone(), val ), env, newCost ) );
-                        } // if
+                        }
                 } // if
         }
@@ -436 +439 @@
                                         return Cost::infinity;
                                 }
-                        }
-                        if ( DefaultArgExpr * def = dynamic_cast< DefaultArgExpr * >( *actualExpr ) ) {
-                                // default arguments should be free - don't include conversion cost.
-                                // Unwrap them here because they are not relevant to the rest of the system.
-                                *actualExpr = def->expr;
-                                ++formal;
-                                continue;
                         }
                         Type * formalType = (*formal)->get_type();
@@ -615 +611 @@
         ConstantExpr* getDefaultValue( Initializer* init ) {
                 if ( SingleInit* si = dynamic_cast<SingleInit*>( init ) ) {
-                        if ( CastExpr* ce = dynamic_cast<CastExpr*>( si->value ) ) {
-                                return dynamic_cast<ConstantExpr*>( ce->arg );
-                        } else {
-                                return dynamic_cast<ConstantExpr*>( si->value );
+                        if ( CastExpr* ce = dynamic_cast<CastExpr*>( si->get_value() ) ) {
+                                return dynamic_cast<ConstantExpr*>( ce->get_arg() );
                         }
                 }
@@ -879 +873 @@
                                                                 indexer ) ) {
                                                         results.emplace_back(
-                                                                i, new DefaultArgExpr( cnstExpr ), move(env), move(need), move(have),
+                                                                i, cnstExpr, move(env), move(need), move(have),
                                                                 move(openVars), nextArg, nTuples );
                                                 }
@@ -1071 +1065 @@
                 funcFinder.findWithAdjustment( untypedExpr->function );
                 // if there are no function alternatives, then proceeding is a waste of time.
-                // xxx - findWithAdjustment throws, so this check and others like it shouldn't be necessary.
                 if ( funcFinder.alternatives.empty() ) return;
 

src/ResolvExpr/Resolver.h

@@ -36 +36 @@
         void resolveCtorInit( ConstructorInit * ctorInit, const SymTab::Indexer & indexer );
         void resolveStmtExpr( StmtExpr * stmtExpr, const SymTab::Indexer & indexer );
-        /// Searches expr and returns the first DeletedExpr found, otherwise nullptr
-        DeletedExpr * findDeletedExpr( Expression * expr );
 } // namespace ResolvExpr
 

src/SynTree/Expression.cc

@@ -746 +746 @@
         os << std::endl << indent+1 << "... deleted by: ";
         deleteStmt->print( os, indent+1 );
-}
-
-
-DefaultArgExpr::DefaultArgExpr( Expression * expr ) : expr( expr ) {
-        assert( expr->result );
-        result = expr->result->clone();
-}
-DefaultArgExpr::DefaultArgExpr( const DefaultArgExpr & other ) : Expression( other ), expr( maybeClone( other.expr ) ) {}
-DefaultArgExpr::~DefaultArgExpr() {
-        delete expr;
-}
-
-void DefaultArgExpr::print( std::ostream & os, Indenter indent ) const {
-        os << "Default Argument Expression" << std::endl << indent+1;
-        expr->print( os, indent+1 );
 }
 

src/SynTree/Expression.h

@@ -865 +865 @@
 };
 
-/// expression wrapping the use of a default argument - should never make it past the resolver.
-class DefaultArgExpr : public Expression {
-public:
-        Expression * expr;
-
-        DefaultArgExpr( Expression * expr );
-        DefaultArgExpr( const DefaultArgExpr & other );
-        ~DefaultArgExpr();
-
-        virtual DefaultArgExpr * clone() const { return new DefaultArgExpr( * this ); }
-        virtual void accept( Visitor & v ) { v.visit( this ); }
-        virtual Expression * acceptMutator( Mutator & m ) { return m.mutate( this ); }
-        virtual void print( std::ostream & os, Indenter indent = {} ) const;
-};
-
 /// C11 _Generic expression
 class GenericExpr : public Expression {

src/SynTree/Mutator.h

@@ -93 +93 @@
         virtual Expression * mutate( InitExpr  * initExpr ) = 0;
         virtual Expression * mutate( DeletedExpr * delExpr ) = 0;
-        virtual Expression * mutate( DefaultArgExpr * argExpr ) = 0;
         virtual Expression * mutate( GenericExpr * genExpr ) = 0;
 

src/SynTree/SynTree.h

@@ -101 +101 @@
 class InitExpr;
 class DeletedExpr;
-class DefaultArgExpr;
 class GenericExpr;
 

src/SynTree/Visitor.h

@@ -95 +95 @@
         virtual void visit( InitExpr *  initExpr ) = 0;
         virtual void visit( DeletedExpr * delExpr ) = 0;
-        virtual void visit( DefaultArgExpr * argExpr ) = 0;
         virtual void visit( GenericExpr * genExpr ) = 0;