This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: RS6000 Changes to support WindISS


	The only piece that concerns me is:

*** 1177,1183 ****
    -Acpu=powerpc -Amachine=powerpc"

  #define	STARTFILE_FREEBSD_SPEC	FBSD_STARTFILE_SPEC
! #define ENDFILE_FREEBSD_SPEC	FBSD_ENDFILE_SPEC
  #define LIB_FREEBSD_SPEC	FBSD_LIB_SPEC
  #define LINK_START_FREEBSD_SPEC	""

--- 1196,1204 ----
    -Acpu=powerpc -Amachine=powerpc"

  #define	STARTFILE_FREEBSD_SPEC	FBSD_STARTFILE_SPEC
! #define ENDFILE_FREEBSD_SPEC    "\
! crtsaveres.o%s %{!shared:crtend.o%s} %{shared:crtendS.o%s} crtn.o%s"
!
  #define LIB_FREEBSD_SPEC	FBSD_LIB_SPEC
  #define LINK_START_FREEBSD_SPEC	""


FreeBSD created a common spec file for all targets and now you want to
duplicate the spec into svr4.h.  This will mean that FreeBSD developers
will need to make any future changes in both places.  I think you need to
discuss this with David O'Brien before that patch of the patch can be
committed.  The rest looks fine.

Thanks, David


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]