wangxindsb added a comment.

> Oh, I think I see how it works now. How about:

> 
> 
>   struct A;
>   struct X {
>      void callFooOfA(A*);
>   };
>   struct A {
>     A() {
>       X x;
>       x.virtualMethod(); // this virtual call is ok
>       x.callFooOfA(this)
>     }
>     virtual foo();
>   };
>   void X::callFooOfA(A* a) {
>      a->foo(); // Would be good to warn here. 
>   }
>   int main() {
>     A a;
>   }

Yes,  you are right, the checker doesn't warn on the //a->foo();//. I will fix 
this error soon.

Thanks,
Xin


https://reviews.llvm.org/D34275



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to