Changeset 321a2481


Ignore:
Timestamp:
Jun 6, 2016, 1:11:34 PM (6 years ago)
Author:
Rob Schluntz <rschlunt@…>
Branches:
aaron-thesis, arm-eh, cleanup-dtors, ctor, deferred_resn, demangler, enum, forall-pointer-decay, gc_noraii, jacob/cs343-translation, jenkins-sandbox, master, memory, new-ast, new-ast-unique-expr, new-env, no_list, persistent-indexer, pthread-emulation, qualifiedEnum, resolv-new, with_gc
Children:
64071c2
Parents:
b8387fc
git-author:
Rob Schluntz <rschlunt@…> (06/06/16 13:08:28)
git-committer:
Rob Schluntz <rschlunt@…> (06/06/16 13:11:34)
Message:

add void casts to code generator to reduce gcc warnings

Location:
src
Files:
5 edited

Legend:

Unmodified
Added
Removed
  • src/CodeGen/CodeGenerator.cc

    rb8387fc r321a2481  
    638638
    639639        void CodeGenerator::visit( ExprStmt *exprStmt ) {
    640                 // I don't see why this check is necessary.
    641                 // If this starts to cause problems then put it back in,
    642                 // with an explanation
    643640                assert( exprStmt );
    644 
    645                 // if ( exprStmt != 0 ) {
    646                 exprStmt->get_expr()->accept( *this );
    647                 output << ";" ;
    648                 // } // if
     641                // cast the top-level expression to void to reduce gcc warnings.
     642                Expression * expr = new CastExpr( exprStmt->get_expr() );
     643                expr->accept( *this );
     644                output << ";";
    649645        }
    650646
     
    755751
    756752        void CodeGenerator::visit( WhileStmt *whileStmt ) {
    757                 if ( whileStmt->get_isDoWhile() )
     753                if ( whileStmt->get_isDoWhile() ) {
    758754                        output << "do" ;
    759                 else {
     755                } else {
    760756                        output << "while (" ;
    761757                        whileStmt->get_condition()->accept( *this );
     
    781777                output << "for (;";
    782778
    783                 if ( forStmt->get_condition() != 0 )
     779                if ( forStmt->get_condition() != 0 ) {
    784780                        forStmt->get_condition()->accept( *this );
     781                }
    785782                output << ";";
    786783
    787                 if ( forStmt->get_increment() != 0 )
    788                         forStmt->get_increment()->accept( *this );
     784                if ( forStmt->get_increment() != 0 ) {
     785                        // cast the top-level expression to void to reduce gcc warnings.
     786                        Expression * expr = new CastExpr( forStmt->get_increment() );
     787                        expr->accept( *this );
     788                }
    789789                output << ") ";
    790790
  • src/Makefile.in

    rb8387fc r321a2481  
    414414          esac; \
    415415        done; \
    416         echo ' cd $(top_srcdir) && $(AUTOMAKE) --gnu src/Makefile'; \
     416        echo ' cd $(top_srcdir) && $(AUTOMAKE) --foreign src/Makefile'; \
    417417        $(am__cd) $(top_srcdir) && \
    418           $(AUTOMAKE) --gnu src/Makefile
     418          $(AUTOMAKE) --foreign src/Makefile
    419419.PRECIOUS: Makefile
    420420Makefile: $(srcdir)/Makefile.in $(top_builddir)/config.status
  • src/driver/Makefile.in

    rb8387fc r321a2481  
    196196          esac; \
    197197        done; \
    198         echo ' cd $(top_srcdir) && $(AUTOMAKE) --gnu src/driver/Makefile'; \
     198        echo ' cd $(top_srcdir) && $(AUTOMAKE) --foreign src/driver/Makefile'; \
    199199        $(am__cd) $(top_srcdir) && \
    200           $(AUTOMAKE) --gnu src/driver/Makefile
     200          $(AUTOMAKE) --foreign src/driver/Makefile
    201201.PRECIOUS: Makefile
    202202Makefile: $(srcdir)/Makefile.in $(top_builddir)/config.status
  • src/examples/Makefile.in

    rb8387fc r321a2481  
    198198          esac; \
    199199        done; \
    200         echo ' cd $(top_srcdir) && $(AUTOMAKE) --gnu src/examples/Makefile'; \
     200        echo ' cd $(top_srcdir) && $(AUTOMAKE) --foreign src/examples/Makefile'; \
    201201        $(am__cd) $(top_srcdir) && \
    202           $(AUTOMAKE) --gnu src/examples/Makefile
     202          $(AUTOMAKE) --foreign src/examples/Makefile
    203203.PRECIOUS: Makefile
    204204Makefile: $(srcdir)/Makefile.in $(top_builddir)/config.status
  • src/libcfa/Makefile.in

    rb8387fc r321a2481  
    231231          esac; \
    232232        done; \
    233         echo ' cd $(top_srcdir) && $(AUTOMAKE) --gnu src/libcfa/Makefile'; \
     233        echo ' cd $(top_srcdir) && $(AUTOMAKE) --foreign src/libcfa/Makefile'; \
    234234        $(am__cd) $(top_srcdir) && \
    235           $(AUTOMAKE) --gnu src/libcfa/Makefile
     235          $(AUTOMAKE) --foreign src/libcfa/Makefile
    236236.PRECIOUS: Makefile
    237237Makefile: $(srcdir)/Makefile.in $(top_builddir)/config.status
Note: See TracChangeset for help on using the changeset viewer.