| Message ID | 5360d644-72d5-454b-92d4-d8298069f6b5@gmail.com |
|---|---|
| State | New |
| Headers | show |
| Series | coreutils-sum-pr108666.c: fix spurious LLP64 warnings | expand |
On 2/2/24 23:55, Jonathan Yong wrote: > Attached patch OK? Fixes the following warnings: > coreutils-sum-pr108666.c:17:1: warning: conflicting types for built-in > function ‘memcpy’; expected ‘void *(void *, const void *, long long > unsigned int)’ [-Wbuiltin-declaration-mismatch] > 17 | memcpy(void* __restrict __dest, const void* __restrict __src, > size_t __n) > | ^~~~~~ > > coreutils-sum-pr108666.c:25:1: warning: conflicting types for built-in > function ‘malloc’; expected ‘void *(long long unsigned int)’ > [-Wbuiltin-declaration-mismatch] > 25 | malloc(size_t __size) __attribute__((__nothrow__, __leaf__)) > | ^~~~~~ > > Copied for review convenience: > diff --git > a/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c > b/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c > index 5684d1b02d4..dadd27eaf41 100644 > --- a/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c > +++ b/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c > @@ -1,6 +1,6 @@ > /* Reduced from coreutils's sum.c: bsd_sum_stream */ > > -typedef long unsigned int size_t; > +typedef __SIZE_TYPE__ size_t; > typedef unsigned char __uint8_t; > typedef unsigned long int __uintmax_t; > typedef struct _IO_FILE FILE; Ping?
On Fri, 2024-02-02 at 23:55 +0000, Jonathan Yong wrote: > Attached patch OK? Fixes the following warnings: Thanks; looks good to me. Dave > coreutils-sum-pr108666.c:17:1: warning: conflicting types for built- > in function ‘memcpy’; expected ‘void *(void *, const void *, long > long unsigned int)’ [-Wbuiltin-declaration-mismatch] > 17 | memcpy(void* __restrict __dest, const void* __restrict > __src, size_t __n) > | ^~~~~~ > > coreutils-sum-pr108666.c:25:1: warning: conflicting types for built- > in function ‘malloc’; expected ‘void *(long long unsigned int)’ [- > Wbuiltin-declaration-mismatch] > 25 | malloc(size_t __size) __attribute__((__nothrow__, __leaf__)) > | ^~~~~~ > > Copied for review convenience: > diff --git a/gcc/testsuite/c-c++-common/analyzer/coreutils-sum- > pr108666.c b/gcc/testsuite/c-c++-common/analyzer/coreutils-sum- > pr108666.c > index 5684d1b02d4..dadd27eaf41 100644 > --- a/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c > +++ b/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c > @@ -1,6 +1,6 @@ > /* Reduced from coreutils's sum.c: bsd_sum_stream */ > > -typedef long unsigned int size_t; > +typedef __SIZE_TYPE__ size_t; > typedef unsigned char __uint8_t; > typedef unsigned long int __uintmax_t; > typedef struct _IO_FILE FILE;
On 2/14/24 13:55, David Malcolm wrote: > On Fri, 2024-02-02 at 23:55 +0000, Jonathan Yong wrote: >> Attached patch OK? Fixes the following warnings: > > Thanks; looks good to me. > > Dave > Thanks, pushed to master branch.
From 54731e86e4bdce03ef4a722860ea8cee931ec127 Mon Sep 17 00:00:00 2001
From: Jonathan Yong <10walls@gmail.com>
Date: Fri, 2 Feb 2024 23:47:47 +0000
Subject: [PATCH] coreutils-sum-pr108666.c: fix spurious LLP64 warnings
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit
Fixes the following warnings on x86_64-w64-mingw32:
coreutils-sum-pr108666.c:17:1: warning: conflicting types for built-in function ‘memcpy’; expected ‘void *(void *, const void *, long long unsigned int)’ [-Wbuiltin-declaration-mismatch]
17 | memcpy(void* __restrict __dest, const void* __restrict __src, size_t __n)
| ^~~~~~
coreutils-sum-pr108666.c:25:1: warning: conflicting types for built-in function ‘malloc’; expected ‘void *(long long unsigned int)’ [-Wbuiltin-declaration-mismatch]
25 | malloc(size_t __size) __attribute__((__nothrow__, __leaf__))
| ^~~~~~
gcc/testsuite:
* coreutils-sum-pr108666.c: Use __SIZE_TYPE__ instead of
long unsigned int for size_t definition.
Signed-off-by: Jonathan Yong <10walls@gmail.com>
---
gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c b/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c
index 5684d1b02d4..dadd27eaf41 100644
--- a/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c
+++ b/gcc/testsuite/c-c++-common/analyzer/coreutils-sum-pr108666.c
@@ -1,6 +1,6 @@
/* Reduced from coreutils's sum.c: bsd_sum_stream */
-typedef long unsigned int size_t;
+typedef __SIZE_TYPE__ size_t;
typedef unsigned char __uint8_t;
typedef unsigned long int __uintmax_t;
typedef struct _IO_FILE FILE;
--
2.43.0
Attached patch OK? Fixes the following warnings: coreutils-sum-pr108666.c:17:1: warning: conflicting types for built-in function ‘memcpy’; expected ‘void *(void *, const void *, long long unsigned int)’ [-Wbuiltin-declaration-mismatch] 17 | memcpy(void* __restrict __dest, const void* __restrict __src, size_t __n) | ^~~~~~ coreutils-sum-pr108666.c:25:1: warning: conflicting types for built-in function ‘malloc’; expected ‘void *(long long unsigned int)’ [-Wbuiltin-declaration-mismatch] 25 | malloc(size_t __size) __attribute__((__nothrow__, __leaf__)) | ^~~~~~ Copied for review convenience: