Remove calls to compiler warnings after we've printed out the message. No reason to generate that code.
Merge branch 'master' of github.com:chapel-lang/chapel into remove-comp-warning
Removing debugging printf; Adding some comments
Updates based on this morning's returns
Initial attempt to optimize anonymous range iteration

When iterating over a range such as:
  for i in 1..10 do ...

we end up constructing a range and then getting it's low and high values out.
The range creation adds additional overhead that is not needed and it
obfuscates the loop from the backend compiler (it probably can't figure out how
the number of iterations even though the chapel compiler knowns it at compile
time.)

Ideally we want to generate
  for i in 1..10
as
  for(i=1; i<=10; i+=1)

This removes the construction cost, and should allow the backend compiler to
optimize/auto-vectorize better.

To accomplish this, this work replaces anonymous ranges with direct iterator
calls in for loops. Ranges are iterable because they have a these method. This
replaces the range with a direct call to an iterator. It essentially replaces
the code:
  for i in 1..10
with:
  for i in _opt_range_iter(1, 10)

the end result is that
  for i in 1..10
is now generated as:
  int64_t i;
  for (i = INT64(1); ((i <= INT64(10))); i += INT64(1)) {}
which is exactly what we wanted

This also handles anonymous ranges whose stride is known at compile time

E.G it handles
  for i in 1..10 by 2

and it will call into a more optimized iterator

if the stride is not known at compile time we don't do anything because we need
the range iterator to use chpl__diffMod

Not that this does nothing for aligned ranges, and doesn't handle cases such as
for i in 1..10 by 2 by 2 since it doesn't seem like a common case

The first attempt to do this was to replace the entire for loop with a c for
loop directly, but you can't do that for zippered iterators and figuring out
the type of the stride is really difficult and you have to insert calls to the
overflow checking and the general iterator.

A better long term solution might be to expand scalarReplacementOfAggregates to
includes ranges. That would allow all ranges to be optimized, including
non-anonymous ranges, and ranges that are used in contexts other than direct
iteration.

I still need to clean up the direct iterators, and make use of them from other
iterators. I also need to add bounds checks and the general iterator around it.
Update elimination of range constructors for anonymous range iteration

Add bounds checking (kind of hacky for now) and call to general iterator if
optimized iterators are not being used.

Should replace anonymous range iteration for most simple ranges (including
zippered)

for i in 1..10
for i in 1..10 by 10
for i in varA..varB
for i in varA..varB by paramA
for (i, j) in zip(1..10 by 2, 1..10 by paramA)

If the range is aligned, or has multiple by's applied to it or the types don't
match then a non-anonymous range is created and iterated so that there don't
have to be a ton of versions of the _direct_range_iter.
Add different versions to support  arg coercion (like is done for build_range)
Swap the order of the iterators to separate them in a more logical manner

Also adds an all int param version of the iterator because it produces the
cleanest loops.

e.g. 'for i in 1..10' turns into 'for (i=1; i<=10; i+=1)' with no temp
variables
Fix general iterator
Comment out all param iter and add TODOS
Strengthen check for bounded iterator and add zippered iter check

Need to be sure this optimization only fires for bounded iterators. Currently
it fired for bounded and unbounded. The check to do so should be strengthened
more. Also when looking at the iteratorExpr we looked for a tuple so that we
could replace ranges in a zippered iterator. However, this also fired for non
zippered iterators over tuples, which we didn't want to do.
Update range overflow detection for direct range iterators

Instead of creating a range to perform the overflow check, directly perform it
with a helper function that the original range overflow check now uses.
Add checks for bad strides
make iters smaller by removing braces and using do loops
make iters smaller by removing braces and using do loops
update comments, convert start=>low and end=>high
Add some more comments and clean up existing ones
Update to use chpl_build_bounded_range
Update brace style to match existing ones in ForLoop.cpp
Remove all param version

The all param version existed to make the cleanest loops, but it wasn't really
needed. Removing it because the code bloat in the modules doesn't seem worth
slightly cleaner generated code.

With the all param version things like:
  for i in 1..10
would turn into:
  for (i=1; i<=10; i+=1)
Now they will turn into:
  int high=10; for (i=1; i<=high; i+=1)

I'll take a look post commit if there's something that can be done to improve
copy propagation so it can achieve the same thing.