[PATCH] D14780: Add calling convention for C++ TLS on Darwin

Manman Ren via llvm-commits llvm-commits at lists.llvm.org
Fri Dec 4 09:49:33 PST 2015


manmanren added a comment.

In http://reviews.llvm.org/D14780#301744, @lhames wrote:

> Perhaps a less generic name would be good, since not all platforms will use cxx_tlscc for C++ TLS. Maybe 'darwin_cxx_tlscc' as Sean suggested, or 'cxx_fast_tlscc'?


Thanks for reviewing! I changed the name to cxx_fast_tlscc as suggested and committed r254737.

Cheer,
Manman

> Otherwise this looks good to me. :)



Repository:
  rL LLVM

http://reviews.llvm.org/D14780





More information about the llvm-commits mailing list