gc_regions: cleanup & fixes for deallocation (#11920)

* gc_regions: withRegion nows return the modified MemRegion

* gc_regions: make withScratchRegion dealloc correctly

* tests/gc: add tregionleak test

This test checks if memory within regions are freed properly.
This commit is contained in:
alaviss 2020-08-17 18:20:48 +00:00 • committed by GitHub
commit e9df8ebcfd
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 27 additions and 5 deletions

View file

@ -87,13 +87,13 @@ var
tlRegion {.threadVar.}: MemRegion
# tempStrRegion {.threadVar.}: MemRegion # not yet used
template withRegion*(r: MemRegion; body: untyped) =
template withRegion*(r: var MemRegion; body: untyped) =
let oldRegion = tlRegion
tlRegion = r
try:
body
finally:
#r = tlRegion
r = tlRegion
tlRegion = oldRegion
template inc(p: pointer, s: int) =
@ -262,14 +262,13 @@ when false:
setObstackPtr(obs)
template withScratchRegion*(body: untyped) =
var scratch: MemRegion
let oldRegion = tlRegion
tlRegion = scratch
tlRegion = MemRegion()
try:
body
finally:
deallocAll()
tlRegion = oldRegion
deallocAll(scratch)
when false:
proc joinRegion*(dest: var MemRegion; src: MemRegion) =