[PATCH] D53906: [ELF] Allow configuring the TLS layout for an Android executable

[Rui Ueyama via Phabricator via llvm-commits]

ruiu added a comment.

This is a well-written small patch, but it introduces a new ABI for TLS variables which makes Android's ABI different from and incompatible with any other ABIs. I don't have a good idea to avoid doing that, though, given that we want to maintain the compatibility with DSOs built for the current and previous versions of Android OSes. However, I'd still like to seek a way so that we don't need to define "Android ABI".

I'd like to keep this patch open for a while. In the meantime, could you try to get a feedback from broader audiences? I think if I approve this patch now, what this patch implements will become a part of the ABI that lives like forever, and I don't think that decision cannot/shouldn't be made only by you and me.


Repository:
  rLLD LLVM Linker

https://reviews.llvm.org/D53906