Re: SunOS patch for memcmp()

Поиск
Список
Период
Сортировка
От Peter Eisentraut
Тема Re: SunOS patch for memcmp()
Дата
Msg-id Pine.LNX.4.30.0112191843250.635-100000@peter.localdomain
обсуждение исходный текст
Ответ на SunOS patch for memcmp()  (Bruce Momjian <pgman@candle.pha.pa.us>)
Ответы Re: SunOS patch for memcmp()  (Bruce Momjian <pgman@candle.pha.pa.us>)
Re: SunOS patch for memcmp()  (Tom Lane <tgl@sss.pgh.pa.us>)
Список pgsql-patches
Bruce Momjian writes:

> > What should we do? The only remaining issue is a non-8-bit-clean
> > memcmp, which seems pretty easy to fix it.
>
> OK, here is a patch to allow SunOS to pass the regression tests.

At a glance, this patch looks okay.  However, since this issue does not
represent a regression from 7.1 I'm not exactly in favour of installing it
now.  We want to get a release out, so I think we need to get stricter in
those matters.  And this patch is not exactly trivial.

> One interesting item is that I had to compile with
> backend/utils/adt/varbit.c with -fno-builtin because my gcc 2.X manual
> says:

Yeah, if you use GCC with optimization on SunOS 4 then the issue should be
moot because the GCC version is used.  However, I don't think that's the
setup in question.

This is actually a good situation to show that configuring with one kind
of compiler flag and building with another is not reliable, even if it's
only the optimization level.

--
Peter Eisentraut   peter_e@gmx.net


В списке pgsql-patches по дате отправления:

Предыдущее
От: Tom Lane
Дата:
Сообщение: Re: Patch to fix float8 test on FreeBSD/alpha
Следующее
От: Bruce Momjian
Дата:
Сообщение: Re: SunOS patch for memcmp()