This is the mail archive of the
libc-alpha@sourceware.org
mailing list for the glibc project.
Re: [PATCH] Revert to defining __extern_inline only for g++-4.3+
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Siddhesh Poyarekar <siddhesh at redhat dot com>
- Cc: libc-alpha at sourceware dot org
- Date: Wed, 13 Aug 2014 20:37:46 +0200
- Subject: Re: [PATCH] Revert to defining __extern_inline only for g++-4.3+
- Authentication-results: sourceware.org; auth=none
- References: <20140813182334 dot GA3507 at spoyarek dot pnq dot redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Wed, Aug 13, 2014 at 11:53:35PM +0530, Siddhesh Poyarekar wrote:
> There is however a regression in this patch for llvm, since it reverts
> the llvm expectation that __GNUC_STDC_INLINE__ or __GNUC_GNU_INLINE__
> definition imply proper extern inline semantics.
Depends on what exactly clang does.
GCC 4.1 and 4.2 define __GNUC_GNU_INLINE__ though, and do not support
gnu_inline attribute in C++, therefore we can't let it through if
__GNUC_GNU_INLINE__ is defined, since that is not a guarantee gnu_inline
will work in C++.
Jakub