Changes in / [aad5a48:10e81ac]

Location:

src

Files:

• InitTweak/FixInit.cc (modified) (2 diffs)
• SymTab/Autogen.cc (modified) (2 diffs)
• SymTab/Validate.cc (modified) (3 diffs)
• SynTree/ApplicationExpr.cc (modified) (1 diff)
• main.cc (modified) (1 diff)

src/InitTweak/FixInit.cc

@@ -131 +131 @@
                                 // optimization: don't need to copy construct in order to call intrinsic functions
                                 return appExpr;
-                        } else if ( FunctionDecl * funcDecl = dynamic_cast< FunctionDecl * > ( function->get_var() ) ) {
-                                FunctionType * ftype = funcDecl->get_functionType();
+                        } else if ( DeclarationWithType * funcDecl = dynamic_cast< DeclarationWithType * > ( function->get_var() ) ) {
+                                // FunctionType * ftype = funcDecl->get_functionType();
+                                FunctionType * ftype = dynamic_cast< FunctionType * >( GenPoly::getFunctionType( funcDecl->get_type() ) );
+                                assert( ftype );
                                 if ( (funcDecl->get_name() == "?{}" || funcDecl->get_name() == "?=?") && ftype->get_parameters().size() == 2 ) {
                                         Type * t1 = ftype->get_parameters().front()->get_type();
@@ -138 +140 @@
                                         PointerType * ptrType = dynamic_cast< PointerType * > ( t1 );
                                         assert( ptrType );
+
                                         if ( ResolvExpr::typesCompatible( ptrType->get_base(), t2, SymTab::Indexer() ) ) {
                                                 // optimization: don't need to copy construct in order to call a copy constructor or

src/SymTab/Autogen.cc

@@ -10 +10 @@
 // Created On       : Thu Mar 03 15:45:56 2016
 // Last Modified By : Rob Schluntz
-// Last Modified On : Wed May 11 13:22:03 2016
+// Last Modified On : Thu May 26 14:14:09 2016
 // Update Count     : 1
 //
@@ -163 +163 @@
         /// Creates a new type decl that's the same as src, but renamed and with only the ?=?, ?{} (default and copy), and ^?{} assertions (for complete types only)
         TypeDecl *cloneAndRename( TypeDecl *src, const std::string &name ) {
+                // TypeDecl *dst = new TypeDecl( name, src->get_storageClass(), 0, src->get_kind() );
+
+                // if ( src->get_kind() == TypeDecl::Any ) {
+                //      TypeInstType *opParamType = new TypeInstType( Type::Qualifiers(), name, dst );
+                //      FunctionType *opFunctionType = new FunctionType( Type::Qualifiers(), false );
+                //      opFunctionType->get_parameters().push_back(
+                //              new ObjectDecl( "", DeclarationNode::NoStorageClass, LinkageSpec::Cforall, 0, new PointerType( Type::Qualifiers(), opParamType->clone() ), 0 ) );
+                //      FunctionDecl *ctorAssert = new FunctionDecl( "?{}", DeclarationNode::NoStorageClass, LinkageSpec::Cforall, opFunctionType->clone(), 0, false, false );
+                //      FunctionDecl *dtorAssert = new FunctionDecl( "^?{}", DeclarationNode::NoStorageClass, LinkageSpec::Cforall, opFunctionType->clone(), 0, false, false );
+
+                //      opFunctionType->get_parameters().push_back(
+                //              new ObjectDecl( "", DeclarationNode::NoStorageClass, LinkageSpec::Cforall, 0, opParamType, 0 ) );
+                //      FunctionDecl *copyCtorAssert = new FunctionDecl( "?{}", DeclarationNode::NoStorageClass, LinkageSpec::Cforall, opFunctionType->clone(), 0, false, false );
+
+                //      opFunctionType->get_returnVals().push_back(
+                //              new ObjectDecl( "", DeclarationNode::NoStorageClass, LinkageSpec::Cforall, 0, opParamType->clone(), 0 ) );
+                //      FunctionDecl *assignAssert = new FunctionDecl( "?=?", DeclarationNode::NoStorageClass, LinkageSpec::Cforall, opFunctionType, 0, false, false );
+
+
+                //      dst->get_assertions().push_back( assignAssert );
+                //      dst->get_assertions().push_back( ctorAssert );
+                //      dst->get_assertions().push_back( dtorAssert );
+                //      dst->get_assertions().push_back( copyCtorAssert );
+                // }
+
                 TypeDecl *dst = new TypeDecl( src->get_name(), src->get_storageClass(), 0, src->get_kind() );
                 cloneAll(src->get_assertions(), dst->get_assertions());

src/SymTab/Validate.cc

@@ -385 +385 @@
 
         void Pass2::visit( StructDecl *structDecl ) {
+                // visit struct members first so that the types of self-referencing members are updated properly
+                Parent::visit( structDecl );
                 if ( ! structDecl->get_members().empty() ) {
                         ForwardStructsType::iterator fwds = forwardStructs.find( structDecl->get_name() );
@@ -394 +396 @@
                         } // if
                 } // if
-                Indexer::visit( structDecl );
         }
 
         void Pass2::visit( UnionDecl *unionDecl ) {
+                Parent::visit( unionDecl );
                 if ( ! unionDecl->get_members().empty() ) {
                         ForwardUnionsType::iterator fwds = forwardUnions.find( unionDecl->get_name() );
@@ -407 +409 @@
                         } // if
                 } // if
-                Indexer::visit( unionDecl );
         }
 

src/SynTree/ApplicationExpr.cc

@@ -64 +64 @@
 
 void ApplicationExpr::print( std::ostream &os, int indent ) const {
-        os << "Application of" << std::endl << std::string(indent, ' ');
+        os << "Application of" << std::endl << std::string(indent+2, ' ');
         function->print( os, indent+2 );
         if ( ! args.empty() ) {

src/main.cc

@@ -203 +203 @@
                 } else {
                         input = stdin;
+                        // if running cfa-cpp directly, might forget to pass -F option. Since this takes from stdin, pass
+                        // a fake name along
+                        if ( filename == NULL ) filename = "stdin";
                 } // if