[PATCH] D28033: [analyzer] Treat pointers to static member functions as function pointers

Kirill Romanenkov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 10 06:10:20 PST 2017


kromanenkov accepted this revision.
kromanenkov added a comment.
This revision is now accepted and ready to land.

Looks good to me, apart from a very small cavil introduced in not even your code. Thanks for not leaving pointers to static member functions out of account!



================
Comment at: test/Analysis/pointer-to-member.cpp:80
     virtual int foo() { return 1; }
     int bar() { return 2;  }
+    int static staticMemberFunction(int p) { return p + 1; };
----------------
This line is not affected by your patch but would you please delete the extra space before the second brace?


https://reviews.llvm.org/D28033





More information about the cfe-commits mailing list