Thanks, I'll fix it up before submitting <br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 3, 2016 at 2:40 PM Jim Ingham <<a href="mailto:jingham@apple.com">jingham@apple.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">jingham added a comment.<br class="gmail_msg">
<br class="gmail_msg">
You messed up the meaning of one comment (noted inline).  Otherwise this looks fine to me too.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
> Args.cpp:97-98<br class="gmail_msg">
> +  // Argument can be split into multiple discontiguous pieces, for example:<br class="gmail_msg">
> +  //  "Hello " "World"<br class="gmail_msg">
> +  // this would result in a single argument "Hello World" (without the quotes)<br class="gmail_msg">
> +  // since the quotes would be removed and there is not space between the<br class="gmail_msg">
<br class="gmail_msg">
The example needs to be "Hello ""World" or it doesn't make sense.  "Hello " "World" with a space in between would be two arguments.<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D25099" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D25099</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>