mirrors/ldc
1
0
Fork 0
mirror of https://github.com/ldc-developers/ldc.git synced 2025-05-09 04:15:58 +03:00
Code Issues Projects Releases Packages Wiki Activity Actions

[fix: handle unsized IR field types]

Browse source
This commit is contained in:
Martin Kinkelin 2022-08-03 19:24:09 +02:00
parent bd5fab1f06
commit 372d6b4983
3 changed files with 7 additions and 24 deletions
Download patch file Download diff file Expand all files Collapse all files

18
gen/llvmhelpers.cpp
View file

@ -1321,24 +1321,6 @@ void DtoSetFuncDeclIntrinsicName(TemplateInstance *ti, TemplateDeclaration *td,
////////////////////////////////////////////////////////////////////////////////
size_t getMemberSize(Type *type) {
const dinteger_t dSize = type->size();
llvm::Type *const llType = DtoType(type);
if (!llType->isSized()) {
// Forward reference in a cycle or similar, we need to trust the D type.
return dSize;
}
const uint64_t llSize = gDataLayout->getTypeAllocSize(llType);
assert(llSize <= dSize &&
"LLVM type is bigger than the corresponding D type, "
"might lead to aggregate layout mismatch.");
return llSize;
}
////////////////////////////////////////////////////////////////////////////////
Type *stripModifiers(Type *type, bool transitive) {
if (type->ty == TY::Tfunction) {
return type;

4
gen/llvmhelpers.h
View file

@ -152,10 +152,6 @@ llvm::CallInst *DtoInlineAsmExpr(const Loc &loc, llvm::StringRef code,
llvm::ArrayRef<llvm::Value *> operands,
llvm::Type *returnType);
/// Returns the size the LLVM type for a member variable of the given type will
/// take up in a struct (in bytes). This does not include padding in any way.
size_t getMemberSize(Type *type);
/// Returns the llvm::Value of the passed DValue, making sure that it is an
/// lvalue (has a memory address), so it can be passed to the D runtime
/// functions without problems.

9
ir/irtypeaggr.cpp
View file

@ -86,7 +86,7 @@ void AggrTypeBuilder::addAggregate(
LLSmallVector<BitFieldDeclaration *, 16> allBitFieldDecls;
for (VarDeclaration *field : ad->fields) {
if (auto bf = field->isBitFieldDeclaration()) {
printf(".: %s: byte offset %d, bit offset %d, type size %d\n", bf->toChars(), bf->offset, bf->bitOffset, (int) bf->type->size());
//printf(".: %s: byte offset %d, bit offset %d, type size %d\n", bf->toChars(), bf->offset, bf->bitOffset, (int) bf->type->size());
allBitFieldDecls.push_back(bf);
}
}
@ -188,7 +188,12 @@ void AggrTypeBuilder::addAggregate(
m_defaultTypes.push_back(fieldType);
// advance offset to right past this field
m_offset += getTypeAllocSize(fieldType); //getMemberSize(vd->type);
if (!fieldType->isSized()) {
// forward reference in a cycle or similar, we need to trust the D type
m_offset += vd->type->size();
} else {
m_offset += getTypeAllocSize(fieldType);
}
// set the field index
m_varGEPIndices[vd] = m_fieldIndex;