HA: [lld] r254428 - [ELF] - Refactor of tls_index implementation for tls local dynamic model.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 3 02:01:11 PST 2015


>addLocalModuleTlsIndex was the correct name. It is the TLS Index
>object for the local module.
>
>- Michael Spencer

Ok. My point was not about it is incorrect, but to make it a bit close to specs.
Urlich has next description "to generate a tls index entry for symbol x1 this code uses x1 at tlsld(%rip) which creates a special kind of index which refers to the current module" so
module is called as "current" there. At the same time he talks about Local Dynamic TLS Model. "local model" is used often in whole spec, but "module" is always "current" I think.
So for me addLocalModelTlsIndex is a little closer to what spec uses.

George.


More information about the llvm-commits mailing list