[PATCH] VAX/Linux: Correct tmake_file setting

View: New views
3 Messages — Rating Filter:   Alert me  

[PATCH] VAX/Linux: Correct tmake_file setting

by Maciej W. Rozycki :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Hello,

 This change:

2009-07-01  Maciej W. Rozycki  <macro@...>

        * config.gcc (vax-*-linux*): Set tmake_file to vax/t-linux.

inadvertently removed generic Linux settings from the vax-linux target
Makefile fragment.  Probably the most notable impact from this is that
crt{begin,end}S.o files are built without -fPIC, which is obviously not
what we want.

 The following fix works for me for --target=vax-linux, emitting correct
relocations in the files mentioned and removing shared library static
link-time warnings that resulted from the incorrect change.

 The configuration is not capable enough for regression testing yet.

2009-10-29  Maciej W. Rozycki  <macro@...>

        * config.gcc (vax-*-linux*): Keep the original contents of
        tmake_file while adding vax/t-linux.

 OK to apply?

  Maciej

gcc-4.5.0-20091029-vax-linux-tmake.patch
Index: gcc-4.5.0-20091029-vax-linux/gcc/config.gcc
===================================================================
--- gcc-4.5.0-20091029-vax-linux.orig/gcc/config.gcc
+++ gcc-4.5.0-20091029-vax-linux/gcc/config.gcc
@@ -2503,7 +2503,7 @@ v850-*-*)
  ;;
 vax-*-linux*)
  tm_file="${tm_file} dbxelf.h elfos.h svr4.h linux.h vax/elf.h vax/linux.h"
- tmake_file=vax/t-linux
+ tmake_file="${tmake_file} vax/t-linux"
  ;;
 vax-*-netbsdelf*)
  tm_file="${tm_file} elfos.h netbsd.h netbsd-elf.h vax/elf.h vax/netbsd-elf.h"

Re: [PATCH] VAX/Linux: Correct tmake_file setting

by Richard Henderson-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On 10/28/2009 07:39 PM, Maciej W. Rozycki wrote:
> * config.gcc (vax-*-linux*): Keep the original contents of
> tmake_file while adding vax/t-linux.

Ok.


r~

Re: [PATCH] VAX/Linux: Correct tmake_file setting

by Maciej W. Rozycki :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

On Tue, 3 Nov 2009, Richard Henderson wrote:

> On 10/28/2009 07:39 PM, Maciej W. Rozycki wrote:
> > * config.gcc (vax-*-linux*): Keep the original contents of
> > tmake_file while adding vax/t-linux.
>
> Ok.

 Both changes applied -- thanks for your review!

  Maciej