<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Feb 22, 2016 at 4:44 PM, Aaron Ballman <span dir="ltr"><<a href="mailto:aaron@aaronballman.com" target="_blank" class="cremed">aaron@aaronballman.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class=""><div class="h5">On Mon, Feb 22, 2016 at 4:43 PM, Samuel Benzaquen <<a href="mailto:sbenza@google.com" class="cremed">sbenza@google.com</a>> wrote:<br>
><br>
> On Mon, Feb 22, 2016 at 4:19 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com" class="cremed">aaron@aaronballman.com</a>><br>
> wrote:<br>
>><br>
>> On Mon, Feb 22, 2016 at 4:13 PM, Samuel Benzaquen via cfe-commits<br>
>> <<a href="mailto:cfe-commits@lists.llvm.org" class="cremed">cfe-commits@lists.llvm.org</a>> wrote:<br>
>> > Author: sbenza<br>
>> > Date: Mon Feb 22 15:13:02 2016<br>
>> > New Revision: 261574<br>
>> ><br>
>> > URL: <a href="http://llvm.org/viewvc/llvm-project?rev=261574&view=rev" rel="noreferrer" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=261574&view=rev</a><br>
>> > Log:<br>
>> > [ASTMatchers] Add matcher hasAnyName.<br>
>> ><br>
>> > Summary: Add matcher hasAnyName as an optimization over<br>
>> > anyOf(hasName(),...)<br>
>><br>
>> Does this mean we can get a clang-tidy check to convert<br>
>> anyOf(hasName(), ...) into hasAnyName()? ;-)<br>
>><br>
>> ~Aaron<br>
><br>
><br>
> I would be simple, but I don't think it the cost/benefit is there. =)<br>
> I changed all the checks manually in 5 minutes.<br>
> I'll be sending that change soon.<br>
<br>
</div></div>Haha, I was joking about the new check, but am really glad to hear<br>
we'll be using the new AST matcher right away. Thank you for working<br>
on this!<br>
<span class=""><font color="#888888"><br>
~Aaron<br></font></span></blockquote><div><br></div><div class="gmail_extra">Matching the name of the nodes is a notable part of the CPU time of clang-tidy.</div><div class="gmail_extra">This is why I spent time making it more efficient more than once already.</div><div><br></div><div>_Sam </div></div><br></div></div>