Page MenuHomePhabricator

[clang][Interp] Handle PtrMemOps
Needs ReviewPublic

Authored by tbaeder on Feb 15 2023, 11:57 PM.

Details

Summary

This is a result of the discussions in https://reviews.llvm.org/D141472 and https://reviews.llvm.org/D142630.

We need to consider virtual functions when calling function pointers as well.

Diff Detail

Event Timeline

tbaeder created this revision.Feb 15 2023, 11:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 15 2023, 11:57 PM
tbaeder requested review of this revision.Feb 15 2023, 11:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 15 2023, 11:57 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
aaron.ballman added inline comments.Wed, May 17, 4:57 AM
clang/lib/AST/Interp/Context.cpp
122

Do you have test coverage for the bound member case?

tbaeder added inline comments.Thu, Jun 1, 2:58 AM
clang/lib/AST/Interp/Context.cpp
122

I was sure I had, I added this to make some test work... but now I removed it and nothing breaks, so I guess I don't.

tbaeder updated this revision to Diff 527346.Thu, Jun 1, 2:58 AM