Changes in src/CodeGen/CodeGenerator.cc [f32c7f4:1840193]

File:

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

src/CodeGen/CodeGenerator.cc

@@ -10 +10 @@
 // Created On       : Mon May 18 07:44:20 2015
 // Last Modified By : Rob Schluntz
-// Last Modified On : Wed Aug 12 14:33:52 2015
-// Update Count     : 222
+// Last Modified On : Thu Sep 17 15:25:58 2015
+// Update Count     : 233
 //
 
@@ -190 +190 @@
                 if ( designators.size() == 0 ) return;
                 for ( DesignatorList::iterator iter = designators.begin(); iter != designators.end(); ++iter ) {
-                        if ( NameExpr * nm = dynamic_cast< NameExpr * >( *iter ) ) {
+                        if ( dynamic_cast< NameExpr * >( *iter ) ) {
                                 // if expression is a name, then initializing aggregate member
                                 output << ".";
@@ -258 +258 @@
               
                                   case OT_CALL:
-                                        // there are no intrinsic definitions of the function call operator
+                                  case OT_CTOR:
+                                  case OT_DTOR:
+                                        // there are no intrinsic definitions of the function call operator or constructors or destructors
                                         assert( false );
                                         break;
@@ -322 +324 @@
               
                                   case OT_CALL:
+                                        case OT_CTOR:
+                                        case OT_DTOR:
                                         assert( false );
                                         break;
@@ -392 +396 @@
 
         void CodeGenerator::visit( CastExpr *castExpr ) {
-                output << "((";
-                if ( castExpr->get_results().empty() ) {
-                        output << "void" ;
-                } else {
-                        output << genType( castExpr->get_results().front(), "" );
-                } // if
-                output << ")";
-                castExpr->get_arg()->accept( *this );
-                output << ")";
+                // if the cast is to an lvalue type, then the cast
+                // should be dropped, since the result of a cast is
+                // never an lvalue in C
+                if ( castExpr->get_results().front()->get_isLvalue() ) {
+                        castExpr->get_arg()->accept( *this );
+                } else {
+                        output << "((";
+                        if ( castExpr->get_results().empty() ) {
+                                output << "void" ;
+                        } else {
+                                output << genType( castExpr->get_results().front(), "" );
+                        } // if
+                        output << ")";
+                        castExpr->get_arg()->accept( *this );
+                        output << ")";                  
+                }
         }