Kris Jurka wrote:
> Bruce Momjian wrote:
> > Great, changes attached and applied.  I removed the solaris_i386 and
> > solaris_x86_64.s files and made just one solaris_x86.s.  I updated the
> > build system to use the new file, updated the macros, and added some
> > documentation on the approach.  Thanks.
> > 
> > Would you test current CVS to make sure it still works properly?  Thanks.
> > 
> 
> The patch adds an extra else in src/template/solaris that means the 
> assembly file is not properly picked up.

Sorry, removed.

> Also the claim that Sun's cc understands C preprocessor doesn't hold 
> true for me:
> 
> /usr/local/SUNWspro/bin/cc -Xa -v -DSUNOS4_CC -g -c tas.s
> Assembler: tas.s
>          "tas.s", line 9 : Illegal mnemonic
>          "tas.s", line 9 : Syntax error
>          "tas.s", line 9 : Illegal mnemonic
>          "tas.s", line 9 : Illegal mnemonic
> 
> This is with cc -V
> cc: Sun WorkShop 6 update 2 C 5.3 Patch 111680-09 2003/05/18

Interesting.  It is probably possible to just run the *.s file through
cpp and compile the result, or just go back to having two files.

-- 
  Bruce Momjian   http://candle.pha.pa.us
  EnterpriseDB    http://www.enterprisedb.com

  + If your life is a hard drive, Christ can be your backup. +

---------------------------(end of broadcast)---------------------------
TIP 3: Have you checked our extensive FAQ?

               http://www.postgresql.org/docs/faq

Reply via email to