Merge branch 'master' of https://github.com/chapel-lang/chapel
Fixed a minor glitch in each of sub_test and paratest.server
Snapshot of where I left things on Friday
Change sub_test to always use the compileline from CHPL_HOME

compileline can behave poorly if run from a CHPL_TEST_UTIL_DIR that is
not inside of CHPL_HOME when using a module.
Merge pull request #669 from Kyle-B/kb-subtest-compileline

Change sub_test to always use the compileline from CHPL_HOME

compileline can behave poorly if run from a CHPL_TEST_UTIL_DIR that is
not inside of CHPL_HOME when using a module.
Introduce While/For loop as a thin veneer over BlockStmt
Convert While/For build functions to class methods on the new classes
Convert remaining uses of ForLoop from BlockStmt to ForLoop veneer in build.cpp
Update a function name to suppress a GCC warning
Updates to REGRESSIONS based on yesterday's runs
Merge pull request #670 from noakesmichael/loopWrap-1

While/For loop veneer

This is the first step in introducing new AST nodes for WhileDo, DoWhile, For, and CFor loops.
The first implementation is a very thin wrapping over BlockStmt.  There is no new state only
new constructors and a few simple methods.  This is the anchor point for adding a reasonably
robust interface layer to these AST nodes across the code base.  Once that is done it will be
practical to refine the internal representation.

There is little "new code" here.  Most code has been migrated from other places e.g.
copying simple code from BlockStmt and from build.cpp

Elliot has performed a sanity check on this code but did not perform a
careful code review given that is largely refactoring of existing code.

Compiled: Std clang on Mac, Std gcc on Liinux, LLVM gcc on Linux
Testing: Full regression suite  std+verify, llvm+verify
Fix NUMA build failure.

Application of FLAG_COMPILER_GENERATED caused locale.name to be selected in favor of
NumaDomain.name.  Assignment to this in the constructor then looked like an attempt to
write to an rvalue, so the Chapel compiler flagged it as an error.

I decided to back out the change.  It seems like in general a user wants the local field
name to bind at least as tightly as an inherited getter.  The compiler generates getter
and setter functions only if they are not provided by the user, so the "compiler
generated" flag is not really needed in this case.

I also backed out the change to classes/diten/nearestMutualParentClass, as the "compiler
generated" pragma is no longer needed there.
Migrate the codegen code from BlockStmt to the new Loop ASTs
Add a test case to show that getters in derived classes dominate.
Merge pull request #671 from hildeth/master

Fix NUMA build failure.

Application of FLAG_COMPILER_GENERATED caused locale.name to be selected in favor of
NumaDomain.name. Assignment to this in the constructor then looked like an attempt to
write to an rvalue, so the Chapel compiler flagged it as an error.

I decided to back out the change. It seems like in general a user wants the local field
name to bind at least as tightly as an inherited getter. The compiler generates getter
and setter functions only if they are not provided by the user, so the "compiler
generated" flag is not really needed in this case.

I also backed out the change to classes/diten/nearestMutualParentClass, as the "compiler
generated" pragma is no longer needed there.

This patch passes all but 6 multilocale tests with CHPL_LOCALE_MODEL=numa and CHPL_GASNET_SEGMENT=fast.  There are known comm-count differences in distributions/robust/arithmetic/performance/multilocale/
  reduceAlias
  reduceSlice
  reduce
There are also known sporadic timeouts in distributions/robust/arithmetic/
  modules/test_module_Sort
  slicing/test_array_slicing3
The remaining test 
  basics/test_zipper_default
now times out consistently on the test machine I was using (chap01).
Add some printfs to verify that the LLVM BlockStmt::codegen() isn't being used for Loops
Merge pull request #672 from noakesmichael/loopWrap-2

Move loop codegen from BlockStmt to new AST nodes

Compiled: STD clang on mac, STD gnu on linux, LLVM gnu on linux
Tested: Full paratest with STD --verify and LLVM --verify
Reviewed: None
Call default constructor by name.

In the compiler-generated _defaultOf() function, set up the call to the zero-args
constructor using its name rather than pre-resolving it to the all-fields constructor.
This will allow binding a user-defined default (zero-args) constructor if there is one.

Also, simplified the logic and flattened the loop structure.
1) Improve the interface for constructing ForLoops  2) Remove the legacy elements of codegen from BlockStmt
Translate PRIM_INIT to _defaultOf() uniformly.

The existing code determines if there is a user-defined zero-args constructor and uses
that if there is, otherwise uses _defaultOf().

The compiler-supplied version of _defaultOf() now calls the user-defined default
constructor, so the test and extra code can be removed.