about summary refs log tree commit diff
path: root/sysdeps/powerpc/test-get_hwcap.c
diff options
context:
space:
mode:
authorPaul A. Clarke <pc@us.ibm.com>2021-10-19 09:49:12 -0500
committerPaul A. Clarke <pc@us.ibm.com>2021-11-03 09:17:28 -0500
commit9fea0f1a2a6e4f7946505c358ab99ea3ab846274 (patch)
tree2a3861a51a14782bfe652e4f4314b1d2f34d0957 /sysdeps/powerpc/test-get_hwcap.c
parentd3bf2f5927d51258a51ac7fde04f4805f8ee294a (diff)
downloadglibc-9fea0f1a2a6e4f7946505c358ab99ea3ab846274.tar.gz
glibc-9fea0f1a2a6e4f7946505c358ab99ea3ab846274.tar.xz
glibc-9fea0f1a2a6e4f7946505c358ab99ea3ab846274.zip
[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 <segher@kernel.crashing.org>
Reviewed-by: Tulio Magno Quites Machado Filho <tuliom@linux.ibm.com>
Diffstat (limited to 'sysdeps/powerpc/test-get_hwcap.c')
-rw-r--r--sysdeps/powerpc/test-get_hwcap.c8
1 files changed, 4 insertions, 4 deletions
diff --git a/sysdeps/powerpc/test-get_hwcap.c b/sysdeps/powerpc/test-get_hwcap.c
index b5cef93cdd..a64b630807 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);