jyapayne/Nim
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

sizeof for empty objects/tuples should be 1; fixes #14690 (#14751)

Browse source
This commit is contained in:
Andreas Rumpf 2020-06-21 19:58:37 +02:00 • committed by GitHub
commit 3ba0c30758
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 11 additions and 3 deletions
Download patch file Download diff file Expand all files Collapse all files

4
compiler/sizealignoffsetimpl.nim
View file

@ -330,7 +330,7 @@ proc computeSizeAlign(conf: ConfigRef; typ: PType) =
sym.offset = accum.offset
accum.inc(int(child.size))
typ.paddingAtEnd = int16(accum.finish())
typ.size = accum.offset
typ.size = if accum.offset == 0: 1 else: accum.offset
typ.align = int16(accum.maxAlign)
except IllegalTypeRecursionError:
typ.paddingAtEnd = szIllegalRecursion
@ -388,7 +388,7 @@ proc computeSizeAlign(conf: ConfigRef; typ: PType) =
typ.align = szUnknownSize
typ.paddingAtEnd = szUnknownSize
else:
typ.size = accum.offset
typ.size = if accum.offset == 0: 1 else: accum.offset
typ.align = int16(accum.maxAlign)
typ.paddingAtEnd = paddingAtEnd
except IllegalTypeRecursionError:

10
tests/misc/tsizeof.nim
View file

@ -78,7 +78,7 @@ macro c_offsetof(fieldAccess: typed): int32 =
## Bullet proof implementation that works on actual offsetof operator
## in the c backend. Assuming of course this implementation is
## correct.
let s = if fieldAccess.kind == nnkCheckedFieldExpr: fieldAccess[0]
let s = if fieldAccess.kind == nnkCheckedFieldExpr: fieldAccess[0]
else: fieldAccess
let a = s[0].getTypeInst
let b = s[1]
@ -686,3 +686,11 @@ reject:
reject:
const off8 = offsetof(MyPackedCaseObject, val5)
type
O0 = object
T0 = tuple[]
doAssert sizeof(O0) == 1
doAssert sizeof(T0) == 1