Skip to content
This repository has been archived by the owner on Oct 12, 2022. It is now read-only.

Commit

Permalink
rt.sections_android: Don't bother excluding static TLS data from GC s…
Browse files Browse the repository at this point in the history
…canning

As it apparently doesn't always end up at the start of the data segment,
e.g., with the gold linker, see
ldc-developers/ldc#3350 (comment).

It should be rather small in most cases, especially in comparison to the
ranges on the heap, so just don't bother.
  • Loading branch information
kinke authored and Geod24 committed Mar 24, 2020
1 parent daf326a commit 475c85f
Showing 1 changed file with 0 additions and 8 deletions.
8 changes: 0 additions & 8 deletions src/rt/sections_android.d
Original file line number Diff line number Diff line change
Expand Up @@ -95,14 +95,6 @@ void initSections() nothrow @nogc
void* end = start + phdr.p_memsz;
debug(PRINTF) printf("data segment: %p - %p\n", start, end);

// exclude static TLS range
if (_staticTLSRange.length)
{
safeAssert(start == _staticTLSRange.ptr,
"static TLS range expected to be at start of data segment");
start += _staticTLSRange.length;
}

// pointer-align up
enum mask = size_t.sizeof - 1;
start = cast(void*) ((cast(size_t)start + mask) & ~mask);
Expand Down

0 comments on commit 475c85f

Please sign in to comment.