Replace all uses of magic numbers when operating on OPENSSL_ia32_P[] by

meaningful constants in a private header file, so that reviewers can actually
get a chance to figure out what the code is attempting to do without knowing
all cpuid bits.

While there, turn it from an array of two 32-bit ints into a properly aligned
64-bit int.

Use of OPENSSL_ia32_P is now restricted to the assembler parts. C code will
now always use OPENSSL_cpu_caps() and check for the proper bits in the
whole 64-bit word it returns.

i386 tests and ok jsing@

1 files changed, 5 insertions, 4 deletions

diff --git a/src/lib/libcrypto/rc4/asm/rc4-x86_64.pl b/src/lib/libcrypto/rc4/asm/rc4-x86_64.pl
index 197749dda7..2135b38ef8 100755
--- a/src/lib/libcrypto/rc4/asm/rc4-x86_64.pl
+++ b/src/lib/libcrypto/rc4/asm/rc4-x86_64.pl
@@ -122,6 +122,7 @@ $out="%rcx";	    # arg4
 $code=<<___;
 .text
 .extern OPENSSL_ia32cap_P
+.hidden OPENSSL_ia32cap_P
 
 .globl  RC4
 .type   RC4,\@function,4
@@ -164,7 +165,7 @@ $code.=<<___;
         movl    ($dat,$XX[0],4),$TX[0]#d
         test    \$-16,$len
         jz      .Lloop1
-        bt      \$30,%r8d       # Intel CPU?
+        bt      \$IA32CAP_BIT0_INTEL,%r8d       # Intel CPU?
         jc      .Lintel
         and     \$7,$TX[1]
         lea     1($XX[0]),$XX[1]
@@ -442,7 +443,7 @@ RC4_set_key:
         xor     %r11,%r11
 
         mov     OPENSSL_ia32cap_P(%rip),$idx#d
-        bt      \$20,$idx#d     # RC4_CHAR?
+        bt      \$IA32CAP_BIT0_INTELP4,$idx#d   # RC4_CHAR?
         jc      .Lc1stloop
         jmp     .Lw1stloop
 
@@ -506,9 +507,9 @@ RC4_set_key:
 RC4_options:
         lea     .Lopts(%rip),%rax
         mov     OPENSSL_ia32cap_P(%rip),%edx
-        bt      \$20,%edx
+        bt      \$IA32CAP_BIT0_INTELP4,%edx
         jc      .L8xchar
-        bt      \$30,%edx
+        bt      \$IA32CAP_BIT0_INTEL,%edx
         jnc     .Ldone
         add     \$25,%rax
         ret