[clang] [clang][PAC] Add __builtin_get_vtable_pointer (PR #139790)

Oliver Hunt via cfe-commits cfe-commits at lists.llvm.org
Tue May 13 23:33:43 PDT 2025


================
@@ -1825,6 +1825,37 @@ static ExprResult PointerAuthStringDiscriminator(Sema &S, CallExpr *Call) {
   return Call;
 }
 
+static ExprResult GetVTablePointer(Sema &S, CallExpr *Call) {
+  if (S.checkArgCount(Call, 1))
+    return ExprError();
+  ExprResult ThisArg = S.DefaultFunctionArrayLvalueConversion(Call->getArg(0));
+  if (ThisArg.isInvalid())
+    return ExprError();
+  Call->setArg(0, ThisArg.get());
+  const Expr *Subject = Call->getArg(0);
+  QualType SubjectType = Subject->getType();
+  const CXXRecordDecl *SubjectRecord = SubjectType->getPointeeCXXRecordDecl();
+  if (!SubjectType->isPointerType() || !SubjectRecord) {
+    S.Diag(Subject->getBeginLoc(), diag::err_get_vtable_pointer_incorrect_type)
+        << 0 << SubjectType;
----------------
ojhunt wrote:

@cor3ntin for both of these (/*Class*/ and /*Polymorpic*/) should that be using `=*/`? and also I wonder if we can make clang format aware of the `/*ParameterName=*/` idiom for parameters of bool and (maybe) integer types? (recognizing that in the context of `operator <<` in diags this is a different problem)

https://github.com/llvm/llvm-project/pull/139790


More information about the cfe-commits mailing list