public inbox for newlib-cvs@sourceware.org
help / color / mirror / Atom feed
* [newlib-cygwin] Remove unused __gnu_inline() attribute.
@ 2017-04-04  9:44 Corinna Vinschen
  0 siblings, 0 replies; only message in thread
From: Corinna Vinschen @ 2017-04-04  9:44 UTC (permalink / raw)
  To: newlib-cvs

https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;h=e1563a05b714f42502496466f12df7b461f40dd1

commit e1563a05b714f42502496466f12df7b461f40dd1
Author: pfg <pfg@FreeBSD.org>
Date:   Tue Apr 4 09:04:56 2017 +0200

    Remove unused __gnu_inline() attribute.
    
    This was meant to be used by a future FORTIFY_SOURCE implementation.
    Probably for good, FORTIFY_SOURCE and this particular GCCism were never
    well supported by clang or other compilers. Furthermore, the technology
    has long since been replaced by either static checkers, sanitizers, or
    even just the strong stack protector that was enabled by default.
    
    Drop __gnu_inline to avoid cluttering the headers.
    
    MFC after:	5 days

Diff:
---
 newlib/libc/include/sys/cdefs.h | 16 ----------------
 1 file changed, 16 deletions(-)

diff --git a/newlib/libc/include/sys/cdefs.h b/newlib/libc/include/sys/cdefs.h
index ad23232..679425a 100644
--- a/newlib/libc/include/sys/cdefs.h
+++ b/newlib/libc/include/sys/cdefs.h
@@ -537,22 +537,6 @@
 	    __attribute__((__format__ (__strftime__, fmtarg, firstvararg)))
 #endif
 
-/*
- * FORTIFY_SOURCE, and perhaps other compiler-specific features, require
- * the use of non-standard inlining.  In general we should try to avoid
- * using these but GCC-compatible compilers tend to support the extensions
- * well enough to use them in limited cases.
- */ 
-#if defined(__GNUC_GNU_INLINE__) || defined(__GNUC_STDC_INLINE__)
-#if __GNUC_PREREQ__(4, 3) || __has_attribute(__artificial__)
-#define	__gnu_inline	__attribute__((__gnu_inline__, __artificial__))
-#else
-#define	__gnu_inline	__attribute__((__gnu_inline__))
-#endif /* artificial */
-#else
-#define	__gnu_inline
-#endif
-
 /* Compiler-dependent macros that rely on FreeBSD-specific extensions. */
 #if defined(__FreeBSD_cc_version) && __FreeBSD_cc_version >= 300001 && \
     defined(__GNUC__) && !defined(__INTEL_COMPILER)


^ permalink raw reply	[flat|nested] only message in thread

only message in thread, other threads:[~2017-04-04  9:44 UTC | newest]

Thread overview: (only message) (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2017-04-04  9:44 [newlib-cygwin] Remove unused __gnu_inline() attribute Corinna Vinschen

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).