Messages in this thread | | | From | "Albert D. Cahalan" <> | Subject | Re: iso9660 endianness cleanup patch | Date | Tue, 1 May 2001 02:37:05 -0400 (EDT) |
| |
Linus Torvalds writes:
> Btw, please use "static inline" instead of "extern inline", as gcc may > decide not to inline the latter at all, leading to confusing link-time > errors. (Gcc may also decide not to inline "static inline", but then gcc > will output the actual body of the function out-of-line if it gets used, > so you don't get the link-time failure). > > Right now only certain broken versions of gcc will actually show this > behaviour, I think, but it's at least in theory going to be an issue.
Since the best choice depends on compiler version:
#if(GCC_VERSION_FOO) #define __inline extern inline #else #define __inline static inline #endif
(that, or _INLINE if you prefer) - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |