Changeset 397c101a for src/ControlStruct/MLEMutator.cc

Timestamp:

Nov 27, 2019, 4:43:51 PM (4 years ago)

Author:

Thierry Delisle <tdelisle@…>

Branches:

ADT, arm-eh, ast-experimental, enum, forall-pointer-decay, jacob/cs343-translation, jenkins-sandbox, master, new-ast, new-ast-unique-expr, pthread-emulation, qualifiedEnum

Children:

524627e, 78cdb06, d4f1521

Parents:

152c2b2

Message:

Fix bug where 'continue' would incorrectly claim to skip initialization Fixes #150?

File:

• src/ControlStruct/MLEMutator.cc (modified) (1 diff)

src/ControlStruct/MLEMutator.cc

@@ -231 +231 @@
 
         Statement *MLEMutator::mutateLoop( Statement *bodyLoop, Entry &e ) {
+                // only generate these when needed
+                if( !e.isContUsed() && !e.isBreakUsed() ) return bodyLoop;
+
                 // ensure loop body is a block
-                CompoundStmt *newBody;
-                if ( ! (newBody = dynamic_cast<CompoundStmt *>( bodyLoop )) ) {
-                        newBody = new CompoundStmt();
-                        newBody->get_kids().push_back( bodyLoop );
-                } // if
-
-                // only generate these when needed
+                CompoundStmt * newBody = new CompoundStmt();
+                newBody->get_kids().push_back( bodyLoop );
 
                 if ( e.isContUsed() ) {