summaryrefslogtreecommitdiff
path: root/lib/builtins/emutls.c
diff options
context:
space:
mode:
authorRyan Prichard <rprichard@google.com>2018-06-12 01:32:25 +0000
committerRyan Prichard <rprichard@google.com>2018-06-12 01:32:25 +0000
commit1c4de81020af3bf7203944aefe7202937b199eba (patch)
treea60ddf8075cc312bd4ac9862d668b1fbce5880c9 /lib/builtins/emutls.c
parent38dc573ae09a1c405594c8cc257d56ec495da8bd (diff)
downloadcompiler-rt-1c4de81020af3bf7203944aefe7202937b199eba.tar.gz
[builtins] emutls cleanup: determine header size using sizeof
Summary: Also add a few post-#else/#endif comments Reviewers: echristo, srhines Reviewed By: echristo Subscribers: chh, delcypher, llvm-commits, #sanitizers Differential Revision: https://reviews.llvm.org/D47861 git-svn-id: https://llvm.org/svn/llvm-project/compiler-rt/trunk@334462 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'lib/builtins/emutls.c')
-rw-r--r--lib/builtins/emutls.c13
1 files changed, 7 insertions, 6 deletions
diff --git a/lib/builtins/emutls.c b/lib/builtins/emutls.c
index 5dd8dd154..063ddfa06 100644
--- a/lib/builtins/emutls.c
+++ b/lib/builtins/emutls.c
@@ -96,7 +96,7 @@ static __inline emutls_address_array* emutls_getspecific() {
return (emutls_address_array*) pthread_getspecific(emutls_pthread_key);
}
-#else
+#else /* _WIN32 */
#include <windows.h>
#include <malloc.h>
@@ -222,11 +222,11 @@ static __inline void __atomic_store_n(void *ptr, uintptr_t val, unsigned type) {
InterlockedExchangePointer((void *volatile *)ptr, (void *)val);
}
-#endif
+#endif /* __ATOMIC_RELEASE */
#pragma warning (pop)
-#endif
+#endif /* _WIN32 */
static size_t emutls_num_object = 0; /* number of allocated TLS objects */
@@ -314,11 +314,12 @@ static __inline void emutls_check_array_set_size(emutls_address_array *array,
* which must be no smaller than the given index.
*/
static __inline uintptr_t emutls_new_data_array_size(uintptr_t index) {
- /* Need to allocate emutls_address_array with one extra slot
- * to store the data array size.
+ /* Need to allocate emutls_address_array with extra slots
+ * to store the header.
* Round up the emutls_address_array size to multiple of 16.
*/
- return ((index + 1 + 15) & ~((uintptr_t)15)) - 1;
+ uintptr_t header_words = sizeof(emutls_address_array) / sizeof(void *);
+ return ((index + header_words + 15) & ~((uintptr_t)15)) - header_words;
}
/* Returns the size in bytes required for an emutls_address_array with