[PATCH] Add a way to check if a method declaration is pure virtual to libclang

Seth Fowler mark.seth.fowler at gmail.com
Sun May 12 18:30:21 PDT 2013


Hi Dmitri,

You got it. An updated patch is attached. I wasn't precisely sure what you intended with regards to the member function template test, since those can't be pure virtual; I assumed you wanted to test that they _weren't_ marked as pure.

Thanks,
- Seth

-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-pure-virtual-query-to-libclang.patch
Type: application/octet-stream
Size: 4305 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130512/b0343b54/attachment.obj>
-------------- next part --------------


On May 8, 2013, at 5:37 AM, Dmitri Gribenko <gribozavr at gmail.com> wrote:

> On Wed, May 8, 2013 at 9:43 AM, Seth Fowler <mark.seth.fowler at gmail.com> wrote:
>> One problem I've encountered is that there doesn't seem to be any way to determine if a method declaration is pure virtual (other than perhaps dropping to the token level). This information is readily available in the clang AST so it seems a shame not to expose it to libclang. Here's a patch to do just that.
> 
> Hi Seth,
> 
> Please add a test.  You are already extending c-index-test to print
> the relevant information.  Just add the relevant test cases (member
> function, member function template) in a separate class in
> test/Index/overrides.cpp.
> 
> Dmitri
> 
> --
> main(i,j){for(i=2;;i++){for(j=2;j<i;j++){if(!(i%j)){j=0;break;}}if
> (j){printf("%d\n",i);}}} /*Dmitri Gribenko <gribozavr at gmail.com>*/



More information about the cfe-commits mailing list