wangxindsb added a comment.

> I do not see any test cases for this patch. Could you add them as well?

I add the test case just now.

> How do you handle the following case?
> 
>   struct A {
>     A() {
>       X x;
>       x.virtualMethod(); // this virtual call is ok
>     }
>   }
>   int main() {
>     A a;
>   }

I use the checker to check the code above, the checker works as expect and 
doesn't throw the warning.


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