Make change suggested by Greg in review.

This add a check to see if we need to create a parent directory and returns
the first error we find that is either mkdir generated, or found by stat but not
a "file or directory does not exist".
A comment was out of date.
Eliminate a duplicate ugly error upon 'myNonRefIter() = 2;'
git-added a file missed before
Adding perf annotations for tcmalloc as default
Also squash .bad file lines containing the phrase "In file included from".
Also update .bad file with "In file included from" lines removed.
Merge pull request #421 from hildeth/squashCLineNumbers

Squash .bad file lines containing "In file included from" in genericArrayInClass-otharrs-inline-iterators.
Rework chpl_getNum{Cores,PUs}OnThisNode().

This standardizes naming, making the following changes:
  chpl_getNumCoresOnThisNode() --> chpl_getNumPhysCpusOnThisNode()
  chpl_getNumPUsOnThisNode() --> chpl_getNumLogCpusOnThisNode()

It also adds an argument to both of these, for callers to indicate
whether they want information about all the CPU resources that are
present on the node, or only the ones to which the process has access
(due to kernel affinity restrictions, etc.).

I ran the maxTaskPar test on this change with CHPL_TASKS=qthreads,
CHPL_TASKS=qthreads and CHPL_LOCALE_MODEL=numa, CHPL_TASKS=fifo,
CHPL_TASKS=massivethreads, and CHPL_TASKS=muxed.
Update .bad file for test_distribution_syntax2
Merge pull request #402 from lydia-duncan/mkdir

Take Greg's comment on mkdir into account [reviewed by gbtitus]

This adds a check to see if we need to create a parent directory and returns
the first error we find that is either mkdir generated, or found by stat but not
a "file or directory does not exist" error (which we would've solved by calling
mkdir afterwards).
Merge pull request #423 from hildeth/master

Update .bad file for test_distribution_syntax2

This test now gets past the point where the original error was encountered, but the original fault remains.  So the .future file was left as-is.
Merge pull request #411 from vasslitvinov/disable-certain-lvalue-errors

Disable certain lvalue errors.

MOTIVATION

The following code:

iter iRef() ref {...}
iRef() = 2;

apparently is well-defined and means
"assign 2 to each thing yielded by iRef".

Analogously, the following code should generate an "lvalue"
error because things yielded by a non-ref iterator should not
be assignable:

// from test/functions/deitz/iterators/test_var_iterator5_error.chpl
iter iVal() {...}
iVal() = 2;

With my PR #324, the code generates two errors - one pointing
to the user code, i.e. the line "iVal() = 2", the other pointing
into our internal modules.

This second error is (a) unnecessary, (b) confusing
"hey your internal modules have an error", and (c)
has a floating line number (Elliot updated .good
three times in the last couple of days).

This commit suppresses this second error.

DETAILS

The AST generated for both snippets above is the same.
Here is AST for iVal(), with embelishments:

def call_tmp:_ir_iVal
move( call_tmp call( fn iVal ) )
call( fn = call_tmp 2 ) <-------------------- [1]

where fn = is this:

// ChapelArray.chpl
proc =(ref ic: _iteratorRecord, x: iteratorIndexType(ic)) {
for e in ic do
e = x;
}

with this, slightly edited, AST:

fn =:void
def arg ic:_ref(_ir_iVal) <-- intent: ref
def arg x:int(64)
{
def _iterator:_ic_iVal
move( _iterator call( fn _getIterator deref( arg ic ) ) )
def _indexOfInterest:int(64)
move( _indexOfInterest call( fn iteratorIndex _iterator ) )
{
def e:int(64)
move( e _indexOfInterest )
call( fn = e arg x ) <-------------------- [2]
for loop( _indexOfInterest _iterator )
}
...
}

where fn = is our standard integer assignment (ref a: int, b: int).

[1]: lvalueCheck() in resolution marks this call an error
because 'call_tmp' has the flag "expr temp". I don't know
the meaning of "expr temp" and how it got there. Evidently
it works for our constness checking.

Aside: In both iVal and iRef cases, 'call_tmp' has also the flags
"temp" "maybe param" "maybe type". In iRef case, there is no "expr temp".

[2]: lvalueCheck() flags this an error because 'e' has the flag "const".

Aside: 'e' also has the flags "insert auto destroy" "index var"
in both cases.
In the iRef case, the type of 'e' and _indexOfInterest is _ref(int(64)),
happily reflecting ref-ness of the underlying iterator.

The error in [2] makes a good sense to me and [1] doesn't.
However, [1] has the right line number, and [2] points into
ChapelArray.chpl, which may confuse for the user ("this is not my error").

After consulting with David and Tom, I feel that we do not need
to report the error in [2] because the one in [1] will have been
reported already.

So how to suppress the error in [2]? I found it the simplest
to mark [2]'s enclosing function with a new flag and have
lvalueCheck() not report errors in functions with this flag.
Alternative suggestions are welcome.

Aside: I thought that when an error is in an internal module,
we climb up the call chain until we hit user code. However
I did not see code to do that. Maybe we only do this for warnings?
Remove a stray semicolon from walk.execopts

This caused testing to fail in any configurations that added additional
execution time flags... ugh.
Merge pull request #425 from bradcray/filerator

Remove a stray semicolon from walk.execopts

This caused testing to fail in any configurations that added additional
execution time flags... ugh.
Renaming Tim's glob/wordexp -> my_glob/my_wordexp

When testing with --baseline, the C compiler complained because
Tim's glob()/wordexp() conflicted with those of the standard C
headers.  Since this is a standalone test for experimental
purposes, I took the easy approach of renaming his iterators
rather than adding more symbols to the reservedCSymbols list.
Change the function names again.

This time it's:
chpl_getNumPhysCpusOnThisNode() --> chpl_getNumPhysicalCpus()
chpl_getNumLogCpusOnThisNode() --> chpl_getNumLogicalCpus().
Merge pull request #424 from benharsh/annotations

Adding annotations to the perf graphs for the time periods where tcmalloc was the default allocator during nightly testing.
Merge pull request #422 from gbtitus/logical-physical-cpus

Rework chpl_getNum{Cores,PUs}OnThisNode().

This standardizes naming, making the following changes:
chpl_getNumCoresOnThisNode() --> chpl_getNumPhysicalCpus()
chpl_getNumPUsOnThisNode() --> chpl_getNumLogicalCpus()

It also adds an argument to both of these, for callers to indicate
whether they want information about all the CPU resources that are
present on the node, or only the ones to which the process has access
(due to kernel affinity restrictions, etc.).

I ran the maxTaskPar test on this change with CHPL_TASKS=qthreads,
CHPL_TASKS=qthreads and CHPL_LOCALE_MODEL=numa, CHPL_TASKS=fifo,
CHPL_TASKS=massivethreads, and CHPL_TASKS=muxed.
Write our own versions of op= for associative domain set operations instead of relying on the default ones