d/freebsd-dev
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Back out wrapping the asm ... ; bits in #ifndef lint macros. There

Browse Source 
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
This commit is contained in:
markm 2001-04-14 20:42:04 +00:00
parent dac860bd9e
commit 0746dc354f
1 changed files with 0 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
sys/i386/include/endian.h
View File

@ -71,9 +71,7 @@ __END_DECLS
static __inline unsigned long static __inline unsigned long
__uint16_swap_uint32(unsigned long __x) __uint16_swap_uint32(unsigned long __x)
{ {
#ifndef lint
__asm ("rorl $16, %1" : "=r" (__x) : "0" (__x)); __asm ("rorl $16, %1" : "=r" (__x) : "0" (__x));
#endif
return __x; return __x;
} }
@ -81,13 +79,11 @@ __uint16_swap_uint32(unsigned long __x)
static __inline unsigned long static __inline unsigned long
__uint8_swap_uint32(unsigned long __x) __uint8_swap_uint32(unsigned long __x)
{ {
#ifndef lint
#if defined(_KERNEL) && (defined(I486_CPU) || defined(I586_CPU) || defined(I686_CPU)) && !defined(I386_CPU) #if defined(_KERNEL) && (defined(I486_CPU) || defined(I586_CPU) || defined(I686_CPU)) && !defined(I386_CPU)
__asm ("bswap %0" : "=r" (__x) : "0" (__x)); __asm ("bswap %0" : "=r" (__x) : "0" (__x));
#else #else
__asm ("xchgb %h1, %b1\n\trorl $16, %1\n\txchgb %h1, %b1" __asm ("xchgb %h1, %b1\n\trorl $16, %1\n\txchgb %h1, %b1"
: "=q" (__x) : "0" (__x)); : "=q" (__x) : "0" (__x));
#endif
#endif #endif
return __x; return __x;
} }
@ -95,9 +91,7 @@ __uint8_swap_uint32(unsigned long __x)
static __inline unsigned short static __inline unsigned short
__uint8_swap_uint16(unsigned short __x) __uint8_swap_uint16(unsigned short __x)
{ {
#ifndef lint
__asm ("xchgb %h1, %b1" : "=q" (__x) : "0" (__x)); __asm ("xchgb %h1, %b1" : "=q" (__x) : "0" (__x));
#endif
return __x; return __x;
} }