<div class="gmail_quote">On Tue, Sep 6, 2011 at 6:53 PM, Jakob Stoklund Olesen <span dir="ltr"><<a href="mailto:stoklund@2pi.dk">stoklund@2pi.dk</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<div class="im">On Sep 6, 2011, at 6:47 PM, Chandler Carruth wrote:<br>
<br>
> Author: chandlerc<br>
> Date: Tue Sep  6 20:47:09 2011<br>
> New Revision: 139217<br>
><br>
</div><div class="im">> -  void EmitCaretDiagnostic(SourceLocation Loc, CharSourceRange *Ranges,<br>
> -                           unsigned NumRanges, const SourceManager &SM,<br>
> +  void EmitCaretDiagnostic(SourceLocation Loc,<br>
> +                           SmallVectorImpl<CharSourceRange> &Ranges,<br>
<br>
</div>Should this be an ArrayRef too?<div class="yj6qo c4rCgd"><div id=":2qy" class="EtNW5c" tabindex="0" title="Show trimmed content"></div></div></blockquote></div><br><div>I *really* wanted it to be one. Sadly, we mutate the source ranges as part of the recursive walk through the macro expansions. =[ I'll look into whether we can do that on the stack as copies eventually; I'm hoping to isolate that recursive walk more before I start messing with it</div>