1) Convert result for buildFoallLoopStmt from SCOPELESS to NORMAL BlockStmt 2) Reformatting for readability
Merge pull request #693 from noakesmichael/forallScoped

When the parser/build functions were creating AST for a ForallLoopStmt, the return value was a
scopeless BlockStmt.  However the top-level of BlockStmt contained definitions for "internal state"
of the underlying iterator.  These variables are therefore implicitly part of the parent scope even though
they are "owned" by the Forall loop.  This in turn tricked some other transformations esp. for --baseline.

Converted to a normal BlockStmt.

compiled:  STD on mac/clang and linux/gcc
tested: paratest --verify and paratest --baseline --inline --verify
reviewed: none