[prev in list] [next in list] [prev in thread] [next in thread] 

List:       cfe-dev
Subject:    Re: [cfe-dev] Allow to selectively skip function bodies while parsing.
From:       Richard Smith <richard () metafoo ! co ! uk>
Date:       2012-11-27 21:32:06
Message-ID: CAOfiQq=nw0nCy5jNtPjRmmZYQqJwr5d3+T7Sa1c=4L_BvboqTA () mail ! gmail ! com
[Download RAW message or body]

[Attachment #2 (multipart/alternative)]


Committed with some minor style tweaks as r168726.

On Tue, Nov 27, 2012 at 10:11 AM, Argyrios Kyrtzidis <kyrtzidis@apple.com>wrote:

> LGTM.
> 
> On Nov 27, 2012, at 9:34 AM, Jordan Rose <jordan_rose@apple.com> wrote:
> 
> > +Argyrios, who I believe added the current skip support.
> > 
> > 
> > On Nov 26, 2012, at 4:03 , Olivier Goffart <ogoffart@kde.org> wrote:
> > 
> > > Hi,
> > > 
> > > I would like to upstream the attached patch which allows ASTConsumer to
> select
> > > which function to skip while parsing.
> > > 
> > > I have been using it to do a online code browser:
> http://code.woboq.org
> > > 
> > > Most of the time of my tool is spent in parsing.
> > > And I have to parse the same includes again and again, and some
> includes have
> > > a lot of inline functions.
> > > By skipping body in already seen includes, I was able to reduce the
> parsing
> > > time by 30%.
> > > 
> > > This might also be useful for other tools like refactoring or so.
> > > 
> > > Regards,
> > > --
> > > 
> Olivier<0001-Add-a-hook-in-the-ASTConsumer-to-be-able-to-skip-fun.patch>_______________________________________________
> 
> > > cfe-dev mailing list
> > > cfe-dev@cs.uiuc.edu
> > > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
> > 
> > _______________________________________________
> > cfe-dev mailing list
> > cfe-dev@cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
> 
> _______________________________________________
> cfe-dev mailing list
> cfe-dev@cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
> 


[Attachment #5 (text/html)]

Committed with some minor style tweaks as r168726.<br><br><div class="gmail_quote">On \
Tue, Nov 27, 2012 at 10:11 AM, Argyrios Kyrtzidis <span dir="ltr">&lt;<a \
href="mailto:kyrtzidis@apple.com" target="_blank">kyrtzidis@apple.com</a>&gt;</span> \
wrote:<br> <blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px \
#ccc solid;padding-left:1ex">LGTM.<br> <div class="HOEnZb"><div class="h5"><br>
On Nov 27, 2012, at 9:34 AM, Jordan Rose &lt;<a \
href="mailto:jordan_rose@apple.com">jordan_rose@apple.com</a>&gt; wrote:<br> <br>
&gt; +Argyrios, who I believe added the current skip support.<br>
&gt;<br>
&gt;<br>
&gt; On Nov 26, 2012, at 4:03 , Olivier Goffart &lt;<a \
href="mailto:ogoffart@kde.org">ogoffart@kde.org</a>&gt; wrote:<br> &gt;<br>
&gt;&gt; Hi,<br>
&gt;&gt;<br>
&gt;&gt; I would like to upstream the attached patch which allows ASTConsumer to \
select<br> &gt;&gt; which function to skip while parsing.<br>
&gt;&gt;<br>
&gt;&gt; I have been using it to do a online code browser:  <a \
href="http://code.woboq.org" target="_blank">http://code.woboq.org</a><br> \
&gt;&gt;<br> &gt;&gt; Most of the time of my tool is spent in parsing.<br>
&gt;&gt; And I have to parse the same includes again and again, and some includes \
have<br> &gt;&gt; a lot of inline functions.<br>
&gt;&gt; By skipping body in already seen includes, I was able to reduce the \
parsing<br> &gt;&gt; time by 30%.<br>
&gt;&gt;<br>
&gt;&gt; This might also be useful for other tools like refactoring or so.<br>
&gt;&gt;<br>
&gt;&gt; Regards,<br>
&gt;&gt; --<br>
&gt;&gt; Olivier&lt;0001-Add-a-hook-in-the-ASTConsumer-to-be-able-to-skip-fun.patch&gt;_______________________________________________<br>
 &gt;&gt; cfe-dev mailing list<br>
&gt;&gt; <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
&gt;&gt; <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" \
target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br> &gt;<br>
&gt; _______________________________________________<br>
&gt; cfe-dev mailing list<br>
&gt; <a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
&gt; <a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" \
target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br> <br>
_______________________________________________<br>
cfe-dev mailing list<br>
<a href="mailto:cfe-dev@cs.uiuc.edu">cfe-dev@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev" \
target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev</a><br> \
</div></div></blockquote></div><br>



_______________________________________________
cfe-dev mailing list
cfe-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev


[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic