diff options
| author | Richard Henderson <rth@twiddle.net> | 2013-07-11 20:47:45 +0400 |
|---|---|---|
| committer | Matt Turner <mattst88@gmail.com> | 2013-11-17 04:33:09 +0400 |
| commit | a47e5bb5764f029f989a182b0dd2d4cce69f8b14 (patch) | |
| tree | d2abfd2919b84ab3917303f5683e982be09f8568 /arch/alpha/lib/memset.S | |
| parent | 673fdfe3f0630b03f3854d0361b1232f2e5ef7fb (diff) | |
| download | linux-a47e5bb5764f029f989a182b0dd2d4cce69f8b14.tar.xz | |
alpha: Eliminate compiler warning from memset macro
Compiling with GCC 4.8 yields several instances of
crypto/vmac.c: In function ‘vmac_final’:
crypto/vmac.c:616:9: warning: value computed is not used [-Wunused-value]
memset(&mac, 0, sizeof(vmac_t));
^
arch/alpha/include/asm/string.h:31:25: note: in definition of macro ‘memset’
? __builtin_memset((s),0,(n)) \
^
Converting the macro to an inline function eliminates this problem.
However, doing only that causes problems with the GCC 3.x series. The
inline function cannot be named "memset", as otherwise we wind up with
recursion via __builtin_memset. Solve this by adjusting the symbols
such that __memset is the inline, and ___memset is the real function.
Signed-off-by: Richard Henderson <rth@twiddle.net>
Diffstat (limited to 'arch/alpha/lib/memset.S')
| -rw-r--r-- | arch/alpha/lib/memset.S | 11 |
1 files changed, 7 insertions, 4 deletions
diff --git a/arch/alpha/lib/memset.S b/arch/alpha/lib/memset.S index 311b8cfc6914..76ccc6d1f364 100644 --- a/arch/alpha/lib/memset.S +++ b/arch/alpha/lib/memset.S @@ -19,11 +19,13 @@ .text .globl memset .globl __memset + .globl ___memset .globl __memsetw .globl __constant_c_memset - .ent __memset + + .ent ___memset .align 5 -__memset: +___memset: .frame $30,0,$26,0 .prologue 0 @@ -103,7 +105,7 @@ within_one_quad: end: ret $31,($26),1 /* E1 */ - .end __memset + .end ___memset .align 5 .ent __memsetw @@ -121,4 +123,5 @@ __memsetw: .end __memsetw -memset = __memset +memset = ___memset +__memset = ___memset |