<p dir="ltr">As a note since I raised a (completely) question about your implementation here you should have waited for an ack from me. </p>
<div class="gmail_quote">On Feb 5, 2014 2:31 PM, "Ben Langmuir" <<a href="mailto:blangmuir@apple.com">blangmuir@apple.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
On Feb 5, 2014, at 8:11 AM, Argyrios Kyrtzidis <<a href="mailto:kyrtzidis@apple.com">kyrtzidis@apple.com</a>> wrote:<br>
<br>
> Sorry for the delay!<br>
><br>
> On Jan 24, 2014, at 9:58 AM, Ben Langmuir <<a href="mailto:blangmuir@apple.com">blangmuir@apple.com</a>> wrote:<br>
><br>
>> Based on an off-list discussion, I’ve updated this patch to not check for configuration mismatches (e.g. different langopts), since checking for staleness is the interesting case to check for and adding all the requisite langopts to the command line increases the complexity of using the feature.<br>

><br>
> +  /// for configuration differences between the AST file and the invokation.<br>
><br>
> typo: ‘invocation'<br>
<br>
I don’t know why, but I always spell invocation incorrectly the first time.  Good catch.<br>
<br>
><br>
> Also this doesn't verify system headers, do you plan to do this after you commit this patch ?<br>
><br>
<br>
Yep, I’ll commit that separately.<br>
<br>
><br>
> Otherwise, LGTM!<br>
<br>
<br>
Great, thank!  r200884<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div>