defaults.h (TARGET_DECLSPEC): New macro.

	* defaults.h (TARGET_DECLSPEC): New macro.
	* c-cppbuiltin.c (c_cpp_builtins): Handle TARGET_DECLSPEC.
	* config/arm/pe.h (SUBTARGET_CPP_SPEC): Remove __declspec support.
	* config/arm/symbian. (TARGET_DLLIMPORT_DECL_ATTRIBUTES): Define.
	* config/i386/beof-elf.h (TARGET_OS_CPP_BUILTINS): Remove
	__declspec support.
	(TARGET_DECLSPEC): Define.
	* config/i386/cygming.h (TARGET_OS_CPP_BUILTINS): Remove
	__declspec support.
	* config/i386/i386-interix.h (TARGET_OS_CPP_BUILTINS): Remove
	__declspec support.
	(TARGET_DECLSPEC): Define.
	* config/mcore/mcore.h (TARGET_CPU_CPP_BUILTINS): Remove
	__declspec support.
	* config/sh/symbian-pre.h (SUBTARGET_CPP_SPEC): Remove __declspec
	support.
	* doc/tm.texi (TARGET_DECLSPEC): Document.

	* doc/extend.texi: Fix typo in comment.

From-SVN: r85571

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 64fff1d6140e381c20e3b01fa6a2345f4f0cfd24..0461d107dffc12320d3a0bd84ef2398bdfc84fbf 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,25 @@
+2004-08-04  Mark Mitchell  <mark@codesourcery.com>
+
+       * defaults.h (TARGET_DECLSPEC): New macro.
+       * c-cppbuiltin.c (c_cpp_builtins): Handle TARGET_DECLSPEC.
+       * config/arm/pe.h (SUBTARGET_CPP_SPEC): Remove __declspec support.
+       * config/arm/symbian. (TARGET_DLLIMPORT_DECL_ATTRIBUTES): Define.
+       * config/i386/beof-elf.h (TARGET_OS_CPP_BUILTINS): Remove
+       __declspec support.
+       (TARGET_DECLSPEC): Define.
+       * config/i386/cygming.h (TARGET_OS_CPP_BUILTINS): Remove
+       __declspec support.
+       * config/i386/i386-interix.h (TARGET_OS_CPP_BUILTINS): Remove
+       __declspec support.
+       (TARGET_DECLSPEC): Define.
+       * config/mcore/mcore.h (TARGET_CPU_CPP_BUILTINS): Remove
+       __declspec support.
+       * config/sh/symbian-pre.h (SUBTARGET_CPP_SPEC): Remove __declspec
+       support.
+       * doc/tm.texi (TARGET_DECLSPEC): Document.
+
+       * doc/extend.texi: Fix typo in comment.
+
 2004-08-04  Paul Brook  <paul@codesourcery.com>
 
        * target-def.h (TARGET_CXX_CDTOR_RETURNS_THIS): Define.

diff --git a/gcc/c-cppbuiltin.c b/gcc/c-cppbuiltin.c
index 3f0f688861d0ed72fb83212f112ac87400bf0ea6..441a38e3b8c1f58d656eb4b4d96e91f749e83db4 100644 (file)
--- a/gcc/c-cppbuiltin.c
+++ b/gcc/c-cppbuiltin.c
@@ -426,6 +426,15 @@ c_cpp_builtins (cpp_reader *pfile)
   TARGET_CPU_CPP_BUILTINS ();
   TARGET_OS_CPP_BUILTINS ();
   TARGET_OBJFMT_CPP_BUILTINS ();
+
+  /* Support the __declspec keyword by turning them into attributes.
+     Note that the current way we do this may result in a collision
+     with predefined attributes later on.  This can be solved by using
+     one attribute, say __declspec__, and passing args to it.  The
+     problem with that approach is that args are not accumulated: each
+     new appearance would clobber any existing args.  */
+  if (TARGET_DECLSPEC)
+    builtin_define ("__declspec(x)=__attribute__((x))");
 }
 
 /* Pass an object-like macro.  If it doesn't lie in the user's

diff --git a/gcc/config/arm/pe.h b/gcc/config/arm/pe.h
index 8eea54ce1f2a8538ccddd793b540ee56cfa16a1a..7db549e68f31390106d3baac92f798b0540422f8 100644 (file)
--- a/gcc/config/arm/pe.h
+++ b/gcc/config/arm/pe.h
@@ -42,15 +42,8 @@
    merge_decl_attributes.  */
 #define TARGET_DLLIMPORT_DECL_ATTRIBUTES
 
-/* Support the __declspec keyword by turning them into attributes.
-   We currently only support: naked, dllimport, and dllexport.
-   Note that the current way we do this may result in a collision with
-   predefined attributes later on.  This can be solved by using one attribute,
-   say __declspec__, and passing args to it.  The problem with that approach
-   is that args are not accumulated: each new appearance would clobber any
-   existing args.  */
 #undef  SUBTARGET_CPP_SPEC
-#define SUBTARGET_CPP_SPEC "-D__pe__ -D__declspec(x)=__attribute__((x))"
+#define SUBTARGET_CPP_SPEC "-D__pe__"
 
 
 /* Experimental addition for pr 7885.

diff --git a/gcc/config/arm/symbian.h b/gcc/config/arm/symbian.h
index 05930d4548d4842f2ccbcc270f4d644568270bd8..2f8320de99e2d35bf8ed486882edcd3399459163 100644 (file)
--- a/gcc/config/arm/symbian.h
+++ b/gcc/config/arm/symbian.h
@@ -30,7 +30,7 @@
    "memmove".  We cannot simply set flag_no_builtin in arm.c because
    (a) flag_no_builtin is not declared in language-independent code,
    and (b) that would prevent users from explicitly overriding the
-   default with -fno-builtin, which may sometimes be useful.
+   default with -fbuiltin, which may sometimes be useful.
 
    Make all symbols hidden by default.  Symbian OS expects that all
    exported symbols will be explicitly marked with
@@ -48,3 +48,6 @@
 #define LINK_SPEC "%{mbig-endian:-EB} %{mlittle-endian:-EL} "          \
   "%{static:-Bstatic} %{shared:-shared} %{symbolic:-Bsymbolic} "       \
   "-X"
+
+/* Support the "dllimport" attribute.  */
+#define TARGET_DLLIMPORT_DECL_ATTRIBUTES 1

diff --git a/gcc/config/i386/beos-elf.h b/gcc/config/i386/beos-elf.h
index 50c39264bc0c82a924e326fc106c83d9bfece18b..b0de9bba834c40cfa48a78af4018978c77bc7508 100644 (file)
--- a/gcc/config/i386/beos-elf.h
+++ b/gcc/config/i386/beos-elf.h
@@ -55,6 +55,8 @@ Boston, MA 02111-1307, USA.  */
 #undef WCHAR_TYPE_SIZE
 #define WCHAR_TYPE_SIZE 16
 
+#define TARGET_DECLSPEC 1
+
 #define TARGET_OS_CPP_BUILTINS()                                       \
   do                                                                   \
     {                                                                  \
@@ -63,7 +65,6 @@ Boston, MA 02111-1307, USA.  */
        builtin_define ("_X86_");                                       \
        builtin_define ("__stdcall=__attribute__((__stdcall__))");      \
        builtin_define ("__cdecl=__attribute__((__cdecl__))");          \
-       builtin_define ("__declspec(x)=__attribute__((x))");            \
        builtin_assert ("system=beos");                                 \
        if (flag_pic)                                                   \
          {                                                             \

diff --git a/gcc/config/i386/cygming.h b/gcc/config/i386/cygming.h
index 69d7f7920ae755499e85ce83a1b94a91415ae5dc..9c2e22aa706cf3dd0587b08795e9b7d25b7a5785 100644 (file)
--- a/gcc/config/i386/cygming.h
+++ b/gcc/config/i386/cygming.h
@@ -72,14 +72,6 @@ Boston, MA 02111-1307, USA.  */
 
 #define MAYBE_UWIN_CPP_BUILTINS() /* Nothing.  */
 
-/* Support the __declspec keyword by turning them into attributes.
-   We currently only support: dllimport and dllexport.
-   Note that the current way we do this may result in a collision with
-   predefined attributes later on.  This can be solved by using one attribute,
-   say __declspec__, and passing args to it.  The problem with that approach
-   is that args are not accumulated: each new appearance would clobber any
-   existing args.  */
-
 #define TARGET_OS_CPP_BUILTINS()                                       \
   do                                                                   \
     {                                                                  \
@@ -88,7 +80,6 @@ Boston, MA 02111-1307, USA.  */
        builtin_define ("__stdcall=__attribute__((__stdcall__))");      \
        builtin_define ("__fastcall=__attribute__((__fastcall__))");    \
        builtin_define ("__cdecl=__attribute__((__cdecl__))");          \
-       builtin_define ("__declspec(x)=__attribute__((x))");            \
        if (!flag_iso)                                                  \
          {                                                             \
            builtin_define ("_stdcall=__attribute__((__stdcall__))");   \

diff --git a/gcc/config/i386/i386-interix.h b/gcc/config/i386/i386-interix.h
index 2a99ce549a7c406dcba85fed3976fd2ac9ce75d7..4119aa2d184946e5423e68350a4eb5e12e0a7696 100644 (file)
--- a/gcc/config/i386/i386-interix.h
+++ b/gcc/config/i386/i386-interix.h
@@ -54,6 +54,8 @@ Boston, MA 02111-1307, USA.  */
 
 #define ASM_LOAD_ADDR(loc, reg)   "     leal " #loc "," #reg "\n"
 
+#define TARGET_DECLSPEC 1
+
 /* cpp handles __STDC__ */
 #define TARGET_OS_CPP_BUILTINS()                                       \
   do                                                                   \
@@ -64,7 +66,6 @@ Boston, MA 02111-1307, USA.  */
        builtin_define ("_X86_=1");                                     \
        builtin_define ("__stdcall=__attribute__((__stdcall__))");      \
        builtin_define ("__cdecl=__attribute__((__cdecl__))");          \
-       builtin_define ("__declspec(x)=__attribute__((x))");            \
        builtin_assert ("system=unix");                                 \
        builtin_assert ("system=interix");                              \
        if (preprocessing_asm_p ())                                     \

diff --git a/gcc/config/mcore/mcore.h b/gcc/config/mcore/mcore.h
index b800a9326590ef4828e7028cea0e6b27155d032f..dde4e093c50a9c94de22ebbe3814c34aae50c2a8 100644 (file)
--- a/gcc/config/mcore/mcore.h
+++ b/gcc/config/mcore/mcore.h
@@ -35,19 +35,11 @@
    merge_decl_attributes.  */
 #define TARGET_DLLIMPORT_DECL_ATTRIBUTES
 
-/* Support the __declspec keyword by turning them into attributes.
-   We currently only support: dllexport and dllimport.
-   Note that the current way we do this may result in a collision with
-   predefined attributes later on.  This can be solved by using one attribute,
-   say __declspec__, and passing args to it.  The problem with that approach
-   is that args are not accumulated: each new appearance would clobber any
-   existing args.  */
 #define TARGET_CPU_CPP_BUILTINS()                                        \
   do                                                                     \
     {                                                                    \
       builtin_define ("__mcore__");                                      \
       builtin_define ("__MCORE__");                                      \
-      builtin_define ("__declspec(x)=__attribute__((x))");               \
       if (TARGET_LITTLE_END)                                             \
         builtin_define ("__MCORELE__");                                          \
       else                                                               \

diff --git a/gcc/config/sh/symbian-pre.h b/gcc/config/sh/symbian-pre.h
index aa23535bda1bf7fd69276fd4c6d5e54126b3f7b0..44fb4a939e288b494c2307bff54cecf8da30e9bb 100644 (file)
--- a/gcc/config/sh/symbian-pre.h
+++ b/gcc/config/sh/symbian-pre.h
@@ -27,14 +27,7 @@
 /* Default to using the Renesas ABI.  */
 #define TARGET_ABI_DEFAULT     RENESAS_BIT
 
-/* Support the __declspec keyword by turning them into attributes.
-   We currently only support: naked, dllimport, and dllexport.
-   Note that the current way we do this may result in a collision with
-   predefined attributes later on.  This can be solved by using one attribute,
-   say __declspec__, and passing args to it.  The problem with that approach
-   is that args are not accumulated: each new appearance would clobber any
-   existing args.  */
-#define SUBTARGET_CPP_SPEC "-D__declspec(x)=__attribute__((x))"
+#define SUBTARGET_CPP_SPEC ""
 
 /* Get tree.c to declare merge_dllimport_decl_attributes().  */
 #define TARGET_DLLIMPORT_DECL_ATTRIBUTES

diff --git a/gcc/defaults.h b/gcc/defaults.h
index cd45f9180ec5b500a741927f77a73cf3b7c10982..3cd779ca29dccde83aa1b5dbb261720e269ab460 100644 (file)
--- a/gcc/defaults.h
+++ b/gcc/defaults.h
@@ -415,6 +415,16 @@ do { fputs (integer_asm_op (POINTER_SIZE / BITS_PER_UNIT, TRUE), FILE); \
 #define PIC_OFFSET_TABLE_REGNUM INVALID_REGNUM
 #endif
 
+#ifndef TARGET_DECLSPEC
+#ifdef TARGET_DLLIMPORT_DECL_ATTRIBUTES
+/* If the target supports the "dllimport" attribute, users are
+   probably used to the "__declspec" syntax.  */
+#define TARGET_DECLSPEC 1
+#else
+#define TARGET_DECLSPEC 0
+#endif
+#endif
+
 /* By default, the preprocessor should be invoked the same way in C++
    as in C.  */
 #ifndef CPLUSPLUS_CPP_SPEC

diff --git a/gcc/doc/extend.texi b/gcc/doc/extend.texi
index 291e4fd44c8768a0ce110a5aaf9fa2ff87f969f2..1b4b76b569f9efd6e3a13dc8d74d31d5f8c49a87 100644 (file)
--- a/gcc/doc/extend.texi
+++ b/gcc/doc/extend.texi
@@ -2177,7 +2177,7 @@ int i __attribute__ ((visibility ("hidden")));
 See the ELF gABI for complete details, but the short story is:
 
 @table @dfn
-@c keep this list of visibilies in alphabetical order.
+@c keep this list of visibilities in alphabetical order.
 
 @item default
 Default visibility is the normal case for ELF.  This value is

diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
index cc3cfeebee420c97465704f61804e3318289fa66..2a0147cfd4de0054b161cf625cd6e20aae21b504 100644 (file)
--- a/gcc/doc/tm.texi
+++ b/gcc/doc/tm.texi
@@ -8346,6 +8346,15 @@ as the expansion of @code{TARGET_MERGE_DECL_ATTRIBUTES}.  This is done
 in @file{i386/cygwin.h} and @file{i386/i386.c}, for example.
 @end deftypefn
 
+@defmac TARGET_DECLSPEC
+Define this macro to a non-zero value if you want to treat
+@code{__declspec(X)} as equivalent to @code{__attribute((X))}.  By
+default, this behavior is enabled only for targets that define
+@code{TARGET_DLLIMPORT_DECL_ATTRIBUTES}.  The current implementation
+of @code{__declspec} is via a built-in macro, but you should not rely
+on this implementation detail.
+@end defmac
+
 @deftypefn {Target Hook} void TARGET_INSERT_ATTRIBUTES (tree @var{node}, tree *@var{attr_ptr})
 Define this target hook if you want to be able to add attributes to a decl
 when it is being created.  This is normally useful for back ends which