diff options
Diffstat (limited to 'gcc')
| -rw-r--r-- | gcc/c-family/c-warn.cc | 20 | ||||
| -rw-r--r-- | gcc/testsuite/c-c++-common/Wsizeof-pointer-memaccess5.c | 29 |
2 files changed, 41 insertions, 8 deletions
diff --git a/gcc/c-family/c-warn.cc b/gcc/c-family/c-warn.cc index 47e0a6b..05d6e37 100644 --- a/gcc/c-family/c-warn.cc +++ b/gcc/c-family/c-warn.cc @@ -944,8 +944,9 @@ sizeof_pointer_memaccess_warning (location_t *sizeof_arg_loc, tree callee, "argument to %<sizeof%> in %qD call is the same " "expression as the destination; did you mean to " "remove the addressof?", callee); - else if ((TYPE_PRECISION (TREE_TYPE (type)) - == TYPE_PRECISION (char_type_node)) + else if ((INTEGRAL_TYPE_P (TREE_TYPE (type)) + && (TYPE_PRECISION (TREE_TYPE (type)) + == TYPE_PRECISION (char_type_node))) || strop) warning_at (loc, OPT_Wsizeof_pointer_memaccess, "argument to %<sizeof%> in %qD call is the same " @@ -984,8 +985,9 @@ sizeof_pointer_memaccess_warning (location_t *sizeof_arg_loc, tree callee, "argument to %<sizeof%> in %qD call is the same " "expression as the source; did you mean to " "remove the addressof?", callee); - else if ((TYPE_PRECISION (TREE_TYPE (type)) - == TYPE_PRECISION (char_type_node)) + else if ((INTEGRAL_TYPE_P (TREE_TYPE (type)) + && (TYPE_PRECISION (TREE_TYPE (type)) + == TYPE_PRECISION (char_type_node))) || strop) warning_at (loc, OPT_Wsizeof_pointer_memaccess, "argument to %<sizeof%> in %qD call is the same " @@ -1024,8 +1026,9 @@ sizeof_pointer_memaccess_warning (location_t *sizeof_arg_loc, tree callee, "argument to %<sizeof%> in %qD call is the same " "expression as the first source; did you mean to " "remove the addressof?", callee); - else if ((TYPE_PRECISION (TREE_TYPE (type)) - == TYPE_PRECISION (char_type_node)) + else if ((INTEGRAL_TYPE_P (TREE_TYPE (type)) + && (TYPE_PRECISION (TREE_TYPE (type)) + == TYPE_PRECISION (char_type_node))) || strop) warning_at (loc, OPT_Wsizeof_pointer_memaccess, "argument to %<sizeof%> in %qD call is the same " @@ -1064,8 +1067,9 @@ sizeof_pointer_memaccess_warning (location_t *sizeof_arg_loc, tree callee, "argument to %<sizeof%> in %qD call is the same " "expression as the second source; did you mean to " "remove the addressof?", callee); - else if ((TYPE_PRECISION (TREE_TYPE (type)) - == TYPE_PRECISION (char_type_node)) + else if ((INTEGRAL_TYPE_P (TREE_TYPE (type)) + && (TYPE_PRECISION (TREE_TYPE (type)) + == TYPE_PRECISION (char_type_node))) || strop) warning_at (loc, OPT_Wsizeof_pointer_memaccess, "argument to %<sizeof%> in %qD call is the same " diff --git a/gcc/testsuite/c-c++-common/Wsizeof-pointer-memaccess5.c b/gcc/testsuite/c-c++-common/Wsizeof-pointer-memaccess5.c new file mode 100644 index 0000000..aaa7da0 --- /dev/null +++ b/gcc/testsuite/c-c++-common/Wsizeof-pointer-memaccess5.c @@ -0,0 +1,29 @@ +/* PR c/117230 */ +/* { dg-do compile } */ +/* { dg-options "-Wsizeof-pointer-memaccess" } */ + +typedef int V __attribute__((vector_size (sizeof (int)))); + +void +foo (V *a, char *b) +{ + __builtin_memcpy (b, a, sizeof (a)); /* { dg-warning "argument to 'sizeof' in '\[^\n\r]*__builtin_memcpy\[^\n\r]*' call is the same expression as the source; did you mean to dereference it\\\?" } */ +} + +void +bar (V *a, char *b) +{ + __builtin_memcpy (a, b, sizeof (a)); /* { dg-warning "argument to 'sizeof' in '\[^\n\r]*__builtin_memcpy\[^\n\r]*' call is the same expression as the destination; did you mean to dereference it\\\?" } */ +} + +int +baz (V *a, char *b) +{ + return __builtin_memcmp (a, b, sizeof (a)); /* { dg-warning "argument to 'sizeof' in '\[^\n\r]*__builtin_memcmp\[^\n\r]*' call is the same expression as the first source; did you mean to dereference it\\\?" } */ +} + +int +qux (V *a, char *b) +{ + return __builtin_memcmp (b, a, sizeof (a)); /* { dg-warning "argument to 'sizeof' in '\[^\n\r]*__builtin_memcmp\[^\n\r]*' call is the same expression as the second source; did you mean to dereference it\\\?" } */ +} |