I have fixed the command line parsing problem for cfd-mini-c
Merge remote-tracking branch 'upstream/master'

Conflicts:
	test/studies/colostate/cfd-mini-c.execopts
Iterators: Revise the test for determining if a SymExpr is in a loop header to handle new representation
Fix stdin support when multiple lines in .execopts.

Previously, a .execopts file like the one below would successfully run with
green.txt, but would fail when running the second configuration for red.txt:

```
< green.txt  # cond.if.green.good
< red.txt    # cond.if.red.good
< yellow.txt # cond.if.yellow.good
```

Error message:

```
...
[Executing program ./cond.if  < green.txt]
[Executing diff cond.if.green.good cond.if.0-1.exec.out.tmp]
[Success matching program output for /Users/tvandoren/src/pytochpl/docs/source/examples/cond.if]
[Error: a redirection file already exists: green.txt]
```

The error is coming from the second iteration of the .execopts loop because the
stdin file is not reset with each loop.

Update sub_test to track when stdin file is modified by .execopts and reset at
the beginning of each .execopts iteration.
Merge pull request #954 from noakesmichael/baselineFix

The code for lowering iterators included a subtle check to determine whether
a symExpr is contained in the header of a loop.  Unlike similar tests it relied
on the overall shape of the code above the SymExpr rather than the more
common reliance on the call-type of the blockInfo.

For the short term I have updated the test to (a) match the new
shape (b) be slightly more robust (c) added a tagged comment
so that I can come back and improve this test when the Loop
work is more sound.

Compiled: STD clang/darwin, STD gcc/linux64, LLVM gcc/linux64
Tested: paratest --baseline and standard paratest
Review: None
Merge pull request #956 from thomasvandoren/fix-sub-test-stdin-multiple-execopts

Fix stdin support when multiple lines in .execopts.

Previously, a .execopts file like the one below would successfully run with
green.txt, but would fail when running the second configuration for red.txt:

```
< green.txt  # cond.if.green.good
< red.txt    # cond.if.red.good
< yellow.txt # cond.if.yellow.good
```

Error message:

```
...
[Executing program ./cond.if  < green.txt]
[Executing diff cond.if.green.good cond.if.0-1.exec.out.tmp]
[Success matching program output for /Users/tvandoren/src/pytochpl/docs/source/examples/cond.if]
[Error: a redirection file already exists: green.txt]
```

The error is coming from the second iteration of the .execopts loop because the
stdin file is not reset with each loop.

Update sub_test to track when stdin file is modified by .execopts and reset at
the beginning of each .execopts iteration.

r: @benharsh
Merge pull request #952 from cathieO/master

Command line parsing in cfd-mini-c fixed

The command line parsing wasn't working because it did not check for unwanted command arguments.
Turn of warning when stdin redirection isn't supported

start_test auto detects when stdin redirection isn't supported and warned the
user that it was going to skip tests with stdin. The warning proved move
annoying than useful especially since it made things like 'make check' fail.
This removes the warning and just turns it into an 'Info' message. The user
won't see it reported at the end of test summary but that seems fine.
Merge pull request #957 from ronawho/multi-locale-test-improvements

Turn of warning when stdin redirection isn't supported

[ok'd by @bradcray @thomasvandoren]

start_test auto detects when stdin redirection isn't supported and warned the
user that it was going to skip tests with stdin. The warning proved move
annoying than useful especially since it made things like 'make check' fail.
This removes the warning and just turns it into an 'Info' message. The user
won't see it reported at the end of test summary but that seems fine.