Extern/Beef
1
0
Fork 0
mirror of https://github.com/beefytech/Beef.git synced 2025-06-08 11:38:21 +02:00
Code Activity

Fix unions with base types

Browse source
This commit is contained in:
Brian Fiete 2022-08-24 16:54:22 -07:00
parent e6cbf0509b
commit f9d5b58102
3 changed files with 17 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

2
IDEHelper/Compiler/BfIRBuilder.cpp
View file

@ -3179,7 +3179,7 @@ void BfIRBuilder::CreateDbgTypeDefinition(BfType* type)
auto resolvedFieldDIType = DbgGetType(fieldType);
String fieldName = "$bfunion";
auto memberType = DbgCreateMemberType(diForwardDecl, fieldName, fileDIScope, lineNum,
fieldType->mSize * 8, fieldType->mAlign * 8, 0,
fieldType->mSize * 8, fieldType->mAlign * 8, typeInstance->mBaseType->mInstSize * 8,
flags, resolvedFieldDIType);
diFieldTypes.push_back(memberType);
}

6
IDEHelper/Compiler/BfModuleTypeUtils.cpp
View file

@ -5433,8 +5433,8 @@ void BfModule::DoPopulateType(BfType* resolvedTypeRef, BfPopulateType populateTy
{
BF_ASSERT(resolvedFieldType->mSize >= 0);
if (alignSize > 1)
dataPos = (dataPos + (alignSize - 1)) & ~(alignSize - 1);
// if (alignSize > 1)
// dataPos = (dataPos + (alignSize - 1)) & ~(alignSize - 1);
fieldInstance->mDataOffset = dataPos;
typeInstance->mInstAlign = std::max(typeInstance->mInstAlign, alignSize);
@ -5623,7 +5623,7 @@ void BfModule::DoPopulateType(BfType* resolvedTypeRef, BfPopulateType populateTy
if (unionInnerType != NULL)
{
dataPos = unionInnerType->mSize;
dataPos = startDataPos + unionInnerType->mSize;
typeInstance->mInstAlign = BF_MAX(unionInnerType->mAlign, typeInstance->mInstAlign);
}

13
IDEHelper/Tests/src/Unions.bf
View file

@ -29,6 +29,12 @@ namespace Tests
}
}
[Union]
struct UnionD : UnionC
{
public int16 mInt16;
}
[Test]
static void TestBasics()
{
@ -41,6 +47,13 @@ namespace Tests
*((float*)&ub.mInt32) = 1.2f;
Test.Assert(ub.mFloat == 1.2f);
Test.Assert(sizeof(UnionB) == 4);
UnionD ud = .();
ud.mInt32 = 123;
ud.mInt16 = 234;
Test.Assert(sizeof(UnionD) == 6);
Test.Assert(alignof(UnionD) == 4);
Test.Assert(((int16*)&ud)[3] == 234);
}
}
}