On Mar 15, 2012, at 23:53, Daniel Dunbar wrote: > On Wed, Mar 14, 2012 at 9:43 AM, Erik Verbruggen <[email protected]> wrote: >> >> >> On 14 mrt. 2012, at 17:34, Douglas Gregor <[email protected]> wrote: >> >>> >>> On Dec 14, 2011, at 7:51 AM, Erik Verbruggen wrote: >>> >>>> Attached is a patch to have the parser skip method bodies. The beef of the >>>> patch is propagating the flag down to the constructor of the Parser class. >>>> I also changed the code-completion (which also uses it) to set this flag. >>> >>> >>> This patch seems to have gotten dropped on the floor. I'm fine with this >>> patch (although I suspect it's gone rather stale). >> >> It's very stale. I'll revisit it when I have time and send a new patch. > > Out of curiosity, what kinds of improvements were you seeing with the patch?
I am using that flag to speeding up indexing declarations/definitions. As I am not looking for usages, skipping method bodies is fine. I forgot the exact numbers, but it took about half the time to index Clang, LLVM, or Qt Creator when skipping method bodies than an unpatched clang (which does parse the method bodies). -- Erik. _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
