OSDN Git Service

make thread-pointer-loading asm non-volatile
authorRich Felker <dalias@aerifal.cx>
Tue, 16 Oct 2018 18:08:01 +0000 (14:08 -0400)
committerRich Felker <dalias@aerifal.cx>
Tue, 16 Oct 2018 18:11:46 +0000 (14:11 -0400)
this will allow the compiler to cache and reuse the result, meaning we
no longer have to take care not to load it more than once for the sake
of archs where the load may be expensive.

depends on commit 1c84c99913bf1cd47b866ed31e665848a0da84a2 for
correctness, since otherwise the compiler could hoist loads during
stage 3 of dynamic linking before the initial thread-pointer setup.

14 files changed:
arch/aarch64/pthread_arch.h
arch/arm/pthread_arch.h
arch/i386/pthread_arch.h
arch/microblaze/pthread_arch.h
arch/mips/pthread_arch.h
arch/mips64/pthread_arch.h
arch/mipsn32/pthread_arch.h
arch/or1k/pthread_arch.h
arch/powerpc/pthread_arch.h
arch/powerpc64/pthread_arch.h
arch/s390x/pthread_arch.h
arch/sh/pthread_arch.h
arch/x32/pthread_arch.h
arch/x86_64/pthread_arch.h

index e8499d8..e64b126 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        char *self;
-       __asm__ __volatile__ ("mrs %0,tpidr_el0" : "=r"(self));
+       __asm__ ("mrs %0,tpidr_el0" : "=r"(self));
        return (void*)(self - sizeof(struct pthread));
 }
 
index 5c6aff2..e689ea2 100644 (file)
@@ -4,7 +4,7 @@
 static inline pthread_t __pthread_self()
 {
        char *p;
-       __asm__ __volatile__ ( "mrc p15,0,%0,c13,c0,3" : "=r"(p) );
+       __asm__ ( "mrc p15,0,%0,c13,c0,3" : "=r"(p) );
        return (void *)(p-sizeof(struct pthread));
 }
 
@@ -20,7 +20,7 @@ static inline pthread_t __pthread_self()
 {
        extern hidden uintptr_t __a_gettp_ptr;
        register uintptr_t p __asm__("r0");
-       __asm__ __volatile__ ( BLX " %1" : "=r"(p) : "r"(__a_gettp_ptr) : "cc", "lr" );
+       __asm__ ( BLX " %1" : "=r"(p) : "r"(__a_gettp_ptr) : "cc", "lr" );
        return (void *)(p-sizeof(struct pthread));
 }
 
index 7f38a56..6f600b9 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        struct pthread *self;
-       __asm__ __volatile__ ("movl %%gs:0,%0" : "=r" (self) );
+       __asm__ ("movl %%gs:0,%0" : "=r" (self) );
        return self;
 }
 
index 08d1ba7..f6ba8de 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        struct pthread *self;
-       __asm__ __volatile__ ("ori %0, r21, 0" : "=r" (self) );
+       __asm__ ("ori %0, r21, 0" : "=r" (self) );
        return self;
 }
 
index 5fea15a..1e7839e 100644 (file)
@@ -2,10 +2,10 @@ static inline struct pthread *__pthread_self()
 {
 #if __mips_isa_rev < 2
        register char *tp __asm__("$3");
-       __asm__ __volatile__ (".word 0x7c03e83b" : "=r" (tp) );
+       __asm__ (".word 0x7c03e83b" : "=r" (tp) );
 #else
        char *tp;
-       __asm__ __volatile__ ("rdhwr %0, $29" : "=r" (tp) );
+       __asm__ ("rdhwr %0, $29" : "=r" (tp) );
 #endif
        return (pthread_t)(tp - 0x7000 - sizeof(struct pthread));
 }
index 5fea15a..1e7839e 100644 (file)
@@ -2,10 +2,10 @@ static inline struct pthread *__pthread_self()
 {
 #if __mips_isa_rev < 2
        register char *tp __asm__("$3");
-       __asm__ __volatile__ (".word 0x7c03e83b" : "=r" (tp) );
+       __asm__ (".word 0x7c03e83b" : "=r" (tp) );
 #else
        char *tp;
-       __asm__ __volatile__ ("rdhwr %0, $29" : "=r" (tp) );
+       __asm__ ("rdhwr %0, $29" : "=r" (tp) );
 #endif
        return (pthread_t)(tp - 0x7000 - sizeof(struct pthread));
 }
index 5fea15a..1e7839e 100644 (file)
@@ -2,10 +2,10 @@ static inline struct pthread *__pthread_self()
 {
 #if __mips_isa_rev < 2
        register char *tp __asm__("$3");
-       __asm__ __volatile__ (".word 0x7c03e83b" : "=r" (tp) );
+       __asm__ (".word 0x7c03e83b" : "=r" (tp) );
 #else
        char *tp;
-       __asm__ __volatile__ ("rdhwr %0, $29" : "=r" (tp) );
+       __asm__ ("rdhwr %0, $29" : "=r" (tp) );
 #endif
        return (pthread_t)(tp - 0x7000 - sizeof(struct pthread));
 }
index 521b9c5..1b806f8 100644 (file)
@@ -3,10 +3,10 @@ static inline struct pthread *__pthread_self()
 {
 #ifdef __clang__
        char *tp;
-       __asm__ __volatile__ ("l.ori %0, r10, 0" : "=r" (tp) );
+       __asm__ ("l.ori %0, r10, 0" : "=r" (tp) );
 #else
        register char *tp __asm__("r10");
-       __asm__ __volatile__ ("" : "=r" (tp) );
+       __asm__ ("" : "=r" (tp) );
 #endif
        return (struct pthread *) (tp - sizeof(struct pthread));
 }
index bab2e6c..ae0f28d 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        register char *tp __asm__("r2");
-       __asm__ __volatile__ ("" : "=r" (tp) );
+       __asm__ ("" : "=r" (tp) );
        return (pthread_t)(tp - 0x7000 - sizeof(struct pthread));
 }
                         
index 37b75e2..79c3ecd 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        register char *tp __asm__("r13");
-       __asm__ __volatile__ ("" : "=r" (tp) );
+       __asm__ ("" : "=r" (tp) );
        return (pthread_t)(tp - 0x7000 - sizeof(struct pthread));
 }
 
index bd90016..e2251f1 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        struct pthread *self;
-       __asm__ __volatile__ (
+       __asm__ (
                "ear  %0, %%a0\n"
                "sllg %0, %0, 32\n"
                "ear  %0, %%a1\n"
index a7dd27a..3ee9c1a 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        char *self;
-       __asm__ __volatile__ ("stc gbr,%0" : "=r" (self) );
+       __asm__ ("stc gbr,%0" : "=r" (self) );
        return (struct pthread *) (self - sizeof(struct pthread));
 }
 
index 267ad07..f640a1a 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        struct pthread *self;
-       __asm__ __volatile__ ("mov %%fs:0,%0" : "=r" (self) );
+       __asm__ ("mov %%fs:0,%0" : "=r" (self) );
        return self;
 }
 
index c61509c..65e880c 100644 (file)
@@ -1,7 +1,7 @@
 static inline struct pthread *__pthread_self()
 {
        struct pthread *self;
-       __asm__ __volatile__ ("mov %%fs:0,%0" : "=r" (self) );
+       __asm__ ("mov %%fs:0,%0" : "=r" (self) );
        return self;
 }