[PATCH] [ELF/AArch64] Fix correct TCB aligment calculation

Adhemerval Zanella adhemerval.zanella at linaro.org
Thu May 28 07:12:17 PDT 2015


Hi ruiu, shankar.easwaran,

This patch fixes the TLS local relocations alignment done by @238258.
As pointed out, the TLS size should not be considered, but rather the
TCB size based on maximum output segment alignment.  Although it has
not shown in the TLS simple cases for test-suite, more comprehensible
tests with more local TLS variable showed wrong relocations values
being generated.

The local TLS testcase is expanded to add more tls variable (both
exported and static) initialized or not.

http://reviews.llvm.org/D10088

Files:
  lib/ReaderWriter/ELF/AArch64/AArch64RelocationHandler.cpp
  lib/ReaderWriter/ELF/AArch64/AArch64TargetHandler.h
  test/elf/AArch64/local-tls.test

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10088.26682.patch
Type: text/x-patch
Size: 10590 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150528/3c7c438d/attachment.bin>


More information about the llvm-commits mailing list