This is an archive of the discontinued LLVM Phabricator instance.

[sanitizer] [SystemZ] Enable TLS functionality.
AbandonedPublic

Authored by • koriakin on Apr 8 2016, 2:53 AM.

Details

Summary

This enables TLS knowledge for s390 in sanitizers.

Diff Detail

Repository
rL LLVM

Event Timeline

• koriakin retitled this revision from to [sanitizer] [SystemZ] Enable TLS functionality..
• koriakin updated this object.
• koriakin set the repository for this revision to rL LLVM.
• koriakin added a project: Restricted Project.
• koriakin added a subscriber: llvm-commits.

I'm not too happy about the inline assembly in ThreadSelf. gcc supports __builtin_thread_pointer that returns exactly what's needed - maybe a better idea would be to add support for that in clang and use it here. WDYT?

• koriakin planned changes to this revision.Apr 14 2016, 1:38 AM

I'm not too happy about the inline assembly in ThreadSelf. gcc supports __builtin_thread_pointer that returns exactly what's needed - maybe a better idea would be to add support for that in clang and use it here. WDYT?

Let's go with __builtin_thread_pointer - I have submitted the llvm bits in D19054 and will soon submit clang bits.

• koriakin abandoned this revision.Apr 27 2016, 7:25 AM

Whoops, accidentally created a new revision (D19594) instead of updating this one; abandoning.