Differential Revision: https://reviews.llvm.org/D137064
}
bool VisitCXXMemberCallExpr(CXXMemberCallExpr *CE) {
- if (isa<CXXDestructorDecl>(CE->getMethodDecl())) {
+ // getMethodDecl can return nullptr with member pointers, e.g.
+ // `(foo.*pointer_to_member_fun)(arg);`
+ if (isa_and_present<CXXDestructorDecl>(CE->getMethodDecl())) {
if (auto *ME = dyn_cast<MemberExpr>(CE->getCallee())) {
if (auto *TI = ME->getMemberNameInfo().getNamedTypeInfo()) {
H.addExtraModifier(TI->getTypeLoc().getBeginLoc(),
$TemplateParameter[[T]] $Variable_def[[x]] = {};
template <>
int $Variable_def[[x]]<int> = (int)sizeof($Class[[Base]]);
+ )cpp",
+ // no crash
+ R"cpp(
+ struct $Class_def[[Foo]] {
+ void $Method_decl[[foo]]();
+ };
+
+ void $Function_def[[s]]($Class[[Foo]] $Parameter_def[[f]]) {
+ auto $LocalVariable_def[[k]] = &$Class[[Foo]]::$Method[[foo]];
+ ($Parameter[[f]].*$LocalVariable[[k]])(); // no crash on VisitCXXMemberCallExpr
+ }
)cpp"};
for (const auto &TestCase : TestCases)
// Mask off scope modifiers to keep the tests manageable.