diff options
author | Anders Carlsson <andersca@mac.com> | 2010-05-03 16:05:06 +0000 |
---|---|---|
committer | Anders Carlsson <andersca@mac.com> | 2010-05-03 16:05:06 +0000 |
commit | 32897fd3bd84e96d4bfa28aca0c7a907776fb855 (patch) | |
tree | 66bd64353837e62f5b094b060c03bc681358333e /lib/CodeGen/CGExprConstant.cpp | |
parent | 7362258bb8c84cc3107993a7249cb3602ceb0bb7 (diff) |
When computing the address of a virtual member function pointer, use the pointer width instead of hardcoding for 64-bit.
git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@102921 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/CodeGen/CGExprConstant.cpp')
-rw-r--r-- | lib/CodeGen/CGExprConstant.cpp | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/lib/CodeGen/CGExprConstant.cpp b/lib/CodeGen/CGExprConstant.cpp index ab0805ee7c..2595ff0060 100644 --- a/lib/CodeGen/CGExprConstant.cpp +++ b/lib/CodeGen/CGExprConstant.cpp @@ -448,11 +448,16 @@ public: if (MD->isVirtual()) { uint64_t Index = CGM.getVTables().getMethodVTableIndex(MD); + // FIXME: We shouldn't use / 8 here. + uint64_t PointerWidthInBytes = + CGM.getContext().Target.getPointerWidth(0) / 8; + // Itanium C++ ABI 2.3: // For a non-virtual function, this field is a simple function pointer. // For a virtual function, it is 1 plus the virtual table offset // (in bytes) of the function, represented as a ptrdiff_t. - Values[0] = llvm::ConstantInt::get(PtrDiffTy, (Index * 8) + 1); + Values[0] = llvm::ConstantInt::get(PtrDiffTy, + (Index * PointerWidthInBytes) + 1); } else { const FunctionProtoType *FPT = MD->getType()->getAs<FunctionProtoType>(); const llvm::Type *Ty = |