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: Fix i686-linux --enable-targets=all


On Mon, Aug 06, 2007 at 07:12:55AM -0700, H.J. Lu wrote:
> Could you tell me where i386/t-crtstuff is included after
> i386/t-crtstuff64? I must have missed it.

i[34567]86-*-linux* | i[34567]86-*-kfreebsd*-gnu | i[34567]86-*-knetbsd*-gnu)
...
                if test x$enable_targets = xall; then
...
                        tmake_file="${tmake_file} i386/t-linux64 i386/t-fprules-softfp64 soft-fp/t-softfp"
...
        tmake_file="${tmake_file} i386/t-crtstuff i386/t-crtpc i386/t-crtfm t-dfprules"
	;;

I think you should be able to add it to the existing t-crtstuff, as in
my patch.  I just didn't fix the other non-Linux targets.

> > Also, I think += is preferred.
> 
> My patch is supposed to set CRTSTUFF_T_CFLAGS only once. += isn't
> very useful.

Oh, I misread this, sorry:

-CRTSTUFF_T_CFLAGS_S = -fPIC
+CRTSTUFF_T_CFLAGS_S = $(CRTSTUFF_T_CFLAGS) -fPIC

-- 
Daniel Jacobowitz
CodeSourcery


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