[clang] [clang][bytecode] Remove a Pointer::toAPValue() call (PR #217594)
Timm Baeder via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 20 04:22:55 PDT 2026
https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/217594
We can just use `computeLayoutOffset()` instead.
>From 7c5674bcbea813ed1278cf0f3d9c32468cdd5cc9 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?= <tbaeder at redhat.com>
Date: Thu, 20 Aug 2026 13:21:48 +0200
Subject: [PATCH] [clang][bytecode] Remove a Pointer::toAPValue() call
We can just use `computeLayoutOffset()` instead.
---
clang/lib/AST/ByteCode/InterpBuiltin.cpp | 12 ++++++++----
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/clang/lib/AST/ByteCode/InterpBuiltin.cpp b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
index 17a0983059bc7..0e7278a6de2c4 100644
--- a/clang/lib/AST/ByteCode/InterpBuiltin.cpp
+++ b/clang/lib/AST/ByteCode/InterpBuiltin.cpp
@@ -1374,15 +1374,16 @@ static bool interp__builtin_assume_aligned(InterpState &S, CodePtr OpPC,
return false;
const Pointer &Ptr = S.Stk.pop<Pointer>();
+ const ASTContext &ASTCtx = S.getASTContext();
CharUnits Align = CharUnits::fromQuantity(Alignment.getZExtValue());
// If there is a base object, then it must have the correct alignment.
if (Ptr.isBlockPointer()) {
CharUnits BaseAlignment;
if (const auto *VD = Ptr.getDeclDesc()->asValueDecl())
- BaseAlignment = S.getASTContext().getDeclAlign(VD);
+ BaseAlignment = ASTCtx.getDeclAlign(VD);
else if (const auto *E = Ptr.getRootExpr())
- BaseAlignment = GetAlignOfExpr(S.getASTContext(), E, UETT_AlignOf);
+ BaseAlignment = GetAlignOfExpr(ASTCtx, E, UETT_AlignOf);
if (BaseAlignment < Align) {
S.CCEDiag(Call->getArg(0),
@@ -1392,8 +1393,11 @@ static bool interp__builtin_assume_aligned(InterpState &S, CodePtr OpPC,
}
}
- APValue AV = Ptr.toAPValue(S.getASTContext());
- CharUnits AVOffset = AV.getLValueOffset();
+ std::optional<size_t> LayoutOffset = Ptr.computeLayoutOffset(ASTCtx);
+ if (!LayoutOffset)
+ return false;
+
+ CharUnits AVOffset = CharUnits::fromQuantity(*LayoutOffset);
if (ExtraOffset)
AVOffset -= CharUnits::fromQuantity(ExtraOffset->getZExtValue());
if (AVOffset.alignTo(Align) != AVOffset) {
More information about the cfe-commits
mailing list