Re: gcc 4.0 build failures on m68k, mips, and mipsel

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Martin Pitt <martin(at)piware(dot)de>
Cc: pgsql-ports(at)postgresql(dot)org
Subject: Re: gcc 4.0 build failures on m68k, mips, and mipsel
Date: 2005-08-25 17:23:49
Message-ID: 331.1124990629@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-ports

Martin Pitt <martin(at)piware(dot)de> writes:
>>> However, Thiemo Seufer dealt with this and created a proper tas()
>>> function for PostgreSQL on mips, which also works with gcc 4. Of
>>> course this is much better, so I applied his patch in Debian.
>>> I attach the patch. Do you consider applying it upstream?
>>
>> Hm, does this patch actually compile as-is? I'd think you'd need to
>> have the MIPS definition of slock_t placed earlier than the assembly
>> code fragment.

> I attach the updated patch against 8.0.

OK, better, but I think not quite there yet. AFAICS the s_lock.c code
should simply go away, as there are no platforms for which it'd be used.
Also, the constraints for the inline assembler ought to include memory
and cc clobbers, according to the comments near the head of s_lock.h.
I've applied the attached modified patch to CVS HEAD, but can't test it
without a MIPS machine ... could you double-check it?

regards, tom lane

Attachment Content-Type Size
unknown_filename text/plain 2.3 KB

In response to

Browse pgsql-ports by date

  From Date Subject
Next Message Anye M. Sellers 2005-08-30 16:12:06 Can a pg_dump file be loaded into a different OS?
Previous Message Martin Pitt 2005-08-25 07:43:12 Re: gcc 4.0 build failures on m68k, mips, and mipsel