Skip to content

Commit 653281c

Browse files
committed
Add comments.
1 parent 5262935 commit 653281c

File tree

1 file changed

+3
-0
lines changed

1 file changed

+3
-0
lines changed

lld/ELF/Relocations.cpp

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1400,6 +1400,9 @@ unsigned RelocationScanner::handleTlsRelocation(RelExpr expr, RelType type,
14001400
// LoongArch does not support transition from TLSDESC to LE/IE in the extreme
14011401
// code model, in which NEEDS_TLSDESC should set, rather than NEEDS_TLSGD. So
14021402
// we check independently.
1403+
// FIXME: The use of NEEDS_TLSDESC originates from previous implementation.
1404+
// Using NEEDS_TLSGD may improve the dynamic linker's loading speed, which
1405+
// will be optimized in the future.
14031406
if (ctx.arg.emachine == EM_LOONGARCH &&
14041407
oneof<RE_LOONGARCH_TLSDESC_PAGE_PC, R_TLSDESC, R_TLSDESC_PC,
14051408
R_TLSDESC_CALL>(expr) &&

0 commit comments

Comments
 (0)