[cfe-commits] [Patch] Adding flag to libclang to skip parsing method bodies

[Erik Verbruggen]

On Mar 15, 2012, at 23:53, Daniel Dunbar wrote:

> On Wed, Mar 14, 2012 at 9:43 AM, Erik Verbruggen <erikjv at me.com> wrote:
>> 
>> 
>> On 14 mrt. 2012, at 17:34, Douglas Gregor <dgregor at apple.com> 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.