<div dir="ltr">Understood.  I will update the patch when I have time and commit the result.  Thanks for the review!<br></div><br><div class="gmail_quote">On Wed, Feb 25, 2015 at 10:50 AM Greg Clayton <<a href="mailto:clayborg@gmail.com">clayborg@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I agree with Enrico's email comments that it would be nice to still use ProcessLaunchInfo so we can pass the environment variables down to the argdumper program so that it could us the env vars to expand command line stuff if needed.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D7805" target="_blank">http://reviews.llvm.org/D7805</a><br>
<br>
EMAIL PREFERENCES<br>
  <a href="http://reviews.llvm.org/settings/panel/emailpreferences/" target="_blank">http://reviews.llvm.org/<u></u>settings/panel/<u></u>emailpreferences/</a><br>
<br>
<br>
</blockquote></div>