From 6da31e65f87031889e2e00fc71656baacdde3fdb Mon Sep 17 00:00:00 2001 From: markm Date: Sat, 14 Apr 2001 20:42:04 +0000 Subject: Back out wrapping the asm ... ; bits in #ifndef lint macros. There are some good reasons for not doing this, even if the linting of the code breaks. 1) If lint were ever to understand the stuff inside the macros, that would break the checks. 2) There are ways to use __GNUC__ to exclude overly specific code. 3) (Not yet practical) Lint(1) needs to properlyu understand all of te code we actually run. Complained about by: bde Education by: jake, jhb, eivind --- sys/i386/include/endian.h | 6 ------ 1 file changed, 6 deletions(-) diff --git a/sys/i386/include/endian.h b/sys/i386/include/endian.h index 2764e69..41ceda4 100644 --- a/sys/i386/include/endian.h +++ b/sys/i386/include/endian.h @@ -71,9 +71,7 @@ __END_DECLS static __inline unsigned long __uint16_swap_uint32(unsigned long __x) { -#ifndef lint __asm ("rorl $16, %1" : "=r" (__x) : "0" (__x)); -#endif return __x; } @@ -81,23 +79,19 @@ __uint16_swap_uint32(unsigned long __x) static __inline unsigned long __uint8_swap_uint32(unsigned long __x) { -#ifndef lint #if defined(_KERNEL) && (defined(I486_CPU) || defined(I586_CPU) || defined(I686_CPU)) && !defined(I386_CPU) __asm ("bswap %0" : "=r" (__x) : "0" (__x)); #else __asm ("xchgb %h1, %b1\n\trorl $16, %1\n\txchgb %h1, %b1" : "=q" (__x) : "0" (__x)); #endif -#endif return __x; } static __inline unsigned short __uint8_swap_uint16(unsigned short __x) { -#ifndef lint __asm ("xchgb %h1, %b1" : "=q" (__x) : "0" (__x)); -#endif return __x; } -- cgit v1.1