[PATCH] D142630: [clang][Interp] Implement virtual function calls

Timm Bäder via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 1 02:42:18 PDT 2023


tbaeder added inline comments.


================
Comment at: clang/lib/AST/Interp/Interp.h:1560
+      // is the furthest we might go up in the hierarchy.
+      ThisPtr = ThisPtr.getDeclPtr();
+    }
----------------
tbaeder wrote:
> Just so I don't forget: The assignment to `ThisPtr` here is dead.
Lies. I explained above that it's for the testcase. `ThisPtr` is a reference, so this replaces the this pointer on the stack.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142630/new/

https://reviews.llvm.org/D142630



More information about the cfe-commits mailing list