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.