[PATCH] D18333: [ELF/AArch64] Fix TLS IE to LE relax for local symbols

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 21 23:58:56 PDT 2016


grimar added a comment.

In http://reviews.llvm.org/D18333#379889, @zatrazz wrote:

> In fact the rebase against your patch to implement TLSDESC made the changes not necessary. The original change as basically:
>  ...
>  So maybe to rename this change to 'Add aarch64 TLS IE to LE relax for local symbol test'?
>
> [1] http://reviews.llvm.org/D18330


Yes, I think you need to change the description and commit text accordingly to what this patch actually do.


Repository:
  rL LLVM

http://reviews.llvm.org/D18333





More information about the llvm-commits mailing list