From: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | John Naylor <john(dot)naylor(at)enterprisedb(dot)com>, PostgreSQL Hackers <pgsql-hackers(at)lists(dot)postgresql(dot)org> |
Subject: | Re: warning: comparison of integer expressions of different signedness related to simd.h |
Date: | 2022-09-03 06:02:02 |
Message-ID: | CAFj8pRB1Ouv4kBBxayoRUswnbpsFsOPQdzUPNSEqLgOnjCOvfw@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
so 3. 9. 2022 v 7:57 odesílatel Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> napsal:
> John Naylor <john(dot)naylor(at)enterprisedb(dot)com> writes:
> > On Sat, Sep 3, 2022 at 12:30 PM Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
> wrote:
> >> /usr/local/pgsql/master/include/server/port/simd.h: In function
> ‘vector8_has’:
> >> /usr/local/pgsql/master/include/server/port/simd.h:168:27: warning:
> comparison of integer expressions of different signedness: ‘int’ and ‘long
> unsigned int’ [-Wsign-compare]
> >> 168 | for (int i = 0; i < sizeof(Vector8); i++)
>
> > "int" should probably be "Size" -- does that remove the warning?
>
> Agreed, should be Size or size_t, or else cast the sizeof() result.
> But I wonder why none of the buildfarm is showing such a warning.
>
I got this warning when I compiled plgsql_check against master with enabled
asserts
https://github.com/okbob/plpgsql_check
In file included from
/usr/local/pgsql/master/include/server/mb/pg_wchar.h:22,
from src/format.c:17:
/usr/local/pgsql/master/include/server/port/simd.h: In function
‘vector8_has_le’:
/usr/local/pgsql/master/include/server/port/simd.h:219:27: warning:
comparison of integer expressions of different signedness: ‘int’ and ‘long
unsigned int’ [-Wsign-compare]
219 | for (int i = 0; i < sizeof(Vector8); i++)
| ^
> regards, tom lane
>
From | Date | Subject | |
---|---|---|---|
Next Message | Julien Rouhaud | 2022-09-03 07:09:03 | Re: pgbench: using prepared BEGIN statement in a pipeline could cause an error |
Previous Message | Tom Lane | 2022-09-03 05:57:32 | Re: warning: comparison of integer expressions of different signedness related to simd.h |