From 9fea0f1a2a6e4f7946505c358ab99ea3ab846274 Mon Sep 17 00:00:00 2001 From: "Paul A. Clarke" Date: Tue, 19 Oct 2021 09:49:12 -0500 Subject: [powerpc] Tighten contraints for asm constant parameters There are a few places where only known numeric values are acceptable for `asm` parameters, yet the constraint "i" is used. "i" can include "symbolic constants whose values will be known only at assembly time or later." Use "n" instead of "i" where known numeric values are required. Suggested-by: Segher Boessenkool Reviewed-by: Tulio Magno Quites Machado Filho --- sysdeps/powerpc/test-get_hwcap.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) (limited to 'sysdeps/powerpc/test-get_hwcap.c') diff --git a/sysdeps/powerpc/test-get_hwcap.c b/sysdeps/powerpc/test-get_hwcap.c index b5cef93..a64b630 100644 --- a/sysdeps/powerpc/test-get_hwcap.c +++ b/sysdeps/powerpc/test-get_hwcap.c @@ -63,16 +63,16 @@ uint64_t check_tcbhwcap (long tid) #ifdef __powerpc64__ __asm__ ("ld %0,%1(%2)\n" : "=r" (tcb_hwcap) - : "i" (__HWCAPOFF), "b" (__tp)); + : "n" (__HWCAPOFF), "b" (__tp)); #else uint64_t h1, h2; __asm__ ("lwz %0,%1(%2)\n" : "=r" (h1) - : "i" (__HWCAPOFF), "b" (__tp)); + : "n" (__HWCAPOFF), "b" (__tp)); __asm__ ("lwz %0,%1(%2)\n" : "=r" (h2) - : "i" (__HWCAP2OFF), "b" (__tp)); + : "n" (__HWCAP2OFF), "b" (__tp)); tcb_hwcap = (h1 >> 32) << 32 | (h2 >> 32); #endif @@ -117,7 +117,7 @@ uint64_t check_tcbhwcap (long tid) /* Same test for the platform number. */ __asm__ ("lwz %0,%1(%2)\n" : "=r" (tcb_at_platform) - : "i" (__ATPLATOFF), "b" (__tp)); + : "n" (__ATPLATOFF), "b" (__tp)); at_platform_string = (const char *) getauxval (AT_PLATFORM); at_platform = _dl_string_platform (at_platform_string); -- cgit v1.1