Remove unnecessary ref temp insertion.
Factor changeRetToArgAndClone().

Further factoring of changeRetToArgAndClone() to get the main routine down to less than a
page.  Also, removed insertions of reference temps which are unnecessary due to the
insertReferenceTemps mini-pass that occurs at the end of callDestructors.
Remove another ref temp insertion.
Update Chapel help message for --log

The old version said the definition of the letters to use lived in log.h.  That
is no longer true, they live in runpasses.cpp.  Update the help message to point
to the right file.
Correct verification problem in 60 test cases.

Removal of the code that inserts ret_to_arg_ref_tmp_ was incorrect.  The code is inserted
when FLAG_REF is absent, not present.

Removed the ancient (and apparently not very accurate) test for premature frees.
Merge pull request #742 from lydia-duncan/updateHelpMessage

Update Chapel help message for --log

The old version said the definition of the letters to use lived in log.h. That
is no longer true as of June, they live in runpasses.cpp. Update the help
message to point to the right file.
Update comments per Lydia's suggestions.
Merge branch 'master' into factorCRTAAC
Merge branch 'factorCRTAAC' of github.com:hildeth/chapel into factorCRTAAC
Merge pull request #744 from hildeth/factorCRTAAC

Factor changeRetToArgAndClone().

Remove insertion of reference temps done later in insertReferenceTemps().
Fix the additional string leaks detected with --baseline

Skipping dead code elimination leaves the array debugging calls to chpl__testPar
in the code (while the contents of this call would never be exercised because of
the appropriate boolean flag, the call is still made).  Since these calls send
in a message in the form of a string, additional memory leaks are generated.
To combat that, we now cast the message to our new string record - this seemed
fine because 1) only developers use chpl__testPar or baseline to our knowledge,
and 2) ChapelStandard already includes the NewString module so the cast was the
only thing that needed to be added.  When the string record is our default, this
cast will no longer be necessary and should be removed.
Merge pull request #747 from lydia-duncan/baselineStringLeaks

Fix the additional string leaks detected with --baseline

Skipping dead code elimination leaves the array debugging calls to chpl__testPar
in the code (while the contents of this call would never be exercised because of
the appropriate boolean flag, the call is still made). Since these calls send
in a message in the form of a string, additional memory leaks are generated.
To combat that, we now cast the message to our new string record - this seemed
fine because 1) only developers use chpl__testPar or baseline to our knowledge,
and 2) ChapelStandard already includes the NewString module so the cast was the
only thing that needed to be added. When the string record is our default, this
cast will no longer be necessary and should be removed.