Fix sign-compare warnings arising from port/simd.h
authorJohn Naylor <[email protected]>
Sun, 4 Sep 2022 02:23:57 +0000 (09:23 +0700)
committerJohn Naylor <[email protected]>
Sun, 4 Sep 2022 02:23:57 +0000 (09:23 +0700)
Noted while building an extension using -Wsign-compare.

Per gripe from Pavel Stehule
Discussion: https://www.postgresql.org/message-id/CAFj8pRAagKQHfw71aQbL8PbL0S_360M61V0_vPqJXbpUFvqnRA%40mail.gmail.com

src/include/port/simd.h

index 74da6e51e03bdd0fca1e01174c9e2ef821ba16d6..61ae4ecf600f6f94c4d122db3369cbe0fb7dc90b 100644 (file)
@@ -165,7 +165,7 @@ vector8_has(const Vector8 v, const uint8 c)
 #ifdef USE_ASSERT_CHECKING
    bool        assert_result = false;
 
-   for (int i = 0; i < sizeof(Vector8); i++)
+   for (Size i = 0; i < sizeof(Vector8); i++)
    {
        if (((const uint8 *) &v)[i] == c)
        {
@@ -216,7 +216,7 @@ vector8_has_le(const Vector8 v, const uint8 c)
 #ifdef USE_ASSERT_CHECKING
    bool        assert_result = false;
 
-   for (int i = 0; i < sizeof(Vector8); i++)
+   for (Size i = 0; i < sizeof(Vector8); i++)
    {
        if (((const uint8 *) &v)[i] <= c)
        {
@@ -239,7 +239,7 @@ vector8_has_le(const Vector8 v, const uint8 c)
    else
    {
        /* one byte at a time */
-       for (int i = 0; i < sizeof(Vector8); i++)
+       for (Size i = 0; i < sizeof(Vector8); i++)
        {
            if (((const uint8 *) &v)[i] <= c)
            {