From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <gcc-bugs-return-455719-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Received: (qmail 24075 invoked by alias); 6 Jul 2014 19:00:54 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Received: (qmail 24006 invoked by uid 55); 6 Jul 2014 19:00:45 -0000
From: "mpolacek at gcc dot gnu.org" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug c/6940] taking sizeof array parameter should trigger a warning
Date: Sun, 06 Jul 2014 19:00:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: c
X-Bugzilla-Version: 3.1
X-Bugzilla-Keywords: diagnostic
X-Bugzilla-Severity: enhancement
X-Bugzilla-Who: mpolacek at gcc dot gnu.org
X-Bugzilla-Status: ASSIGNED
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: mpolacek at gcc dot gnu.org
X-Bugzilla-Target-Milestone: 4.10.0
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields:
Message-ID: <bug-6940-4-OSFyk0EJvY@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-6940-4@http.gcc.gnu.org/bugzilla/>
References: <bug-6940-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014-07/txt/msg00310.txt.bz2

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=6940

--- Comment #3 from Marek Polacek <mpolacek at gcc dot gnu.org> ---
Author: mpolacek
Date: Sun Jul  6 19:00:10 2014
New Revision: 212312

URL: https://gcc.gnu.org/viewcvs?rev=212312&root=gcc&view=rev
Log:
    PR c/6940
    * doc/invoke.texi: Document -Wsizeof-array-argument.
c-family/
    * c.opt (Wsizeof-array-argument): New option.
c/
    * c-decl.c (grokdeclarator): Set C_ARRAY_PARAMETER.
    * c-tree.h (C_ARRAY_PARAMETER): Define.
    * c-typeck.c (c_expr_sizeof_expr): Warn when using sizeof on an array
    function parameter.
cp/
    * cp-tree.h (DECL_ARRAY_PARAMETER_P): Define.
    * decl.c (grokdeclarator): Set DECL_ARRAY_PARAMETER_P.
    * typeck.c (cxx_sizeof_expr): Warn when using sizeof on an array
    function parameter.
testsuite/
    * c-c++-common/Wsizeof-pointer-memaccess1.c: Use
    -Wno-sizeof-array-argument.
    * c-c++-common/Wsizeof-pointer-memaccess2.c: Likewise.
    * g++.dg/warn/Wsizeof-pointer-memaccess-1.C: Likewise.
    * gcc.dg/Wsizeof-pointer-memaccess1.c: Likewise.
    * g++.dg/torture/Wsizeof-pointer-memaccess1.C: Likewise.
    * g++.dg/torture/Wsizeof-pointer-memaccess2.C: Likewise.
    * gcc.dg/torture/Wsizeof-pointer-memaccess1.c: Likewise.
    * c-c++-common/sizeof-array-argument.c: New test.
    * gcc.dg/vla-5.c: Add dg-warnings.
../libgomp/
    * testsuite/libgomp.c/appendix-a/a.29.1.c (f): Add dg-warnings.

Added:
    trunk/gcc/testsuite/c-c++-common/sizeof-array-argument.c
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/c-family/ChangeLog
    trunk/gcc/c-family/c.opt
    trunk/gcc/c/ChangeLog
    trunk/gcc/c/c-decl.c
    trunk/gcc/c/c-tree.h
    trunk/gcc/c/c-typeck.c
    trunk/gcc/cp/ChangeLog
    trunk/gcc/cp/cp-tree.h
    trunk/gcc/cp/decl.c
    trunk/gcc/cp/typeck.c
    trunk/gcc/doc/invoke.texi
    trunk/gcc/testsuite/ChangeLog
    trunk/gcc/testsuite/c-c++-common/Wsizeof-pointer-memaccess1.c
    trunk/gcc/testsuite/c-c++-common/Wsizeof-pointer-memaccess2.c
    trunk/gcc/testsuite/g++.dg/torture/Wsizeof-pointer-memaccess1.C
    trunk/gcc/testsuite/g++.dg/torture/Wsizeof-pointer-memaccess2.C
    trunk/gcc/testsuite/g++.dg/warn/Wsizeof-pointer-memaccess-1.C
    trunk/gcc/testsuite/gcc.dg/Wsizeof-pointer-memaccess1.c
    trunk/gcc/testsuite/gcc.dg/torture/Wsizeof-pointer-memaccess1.c
    trunk/gcc/testsuite/gcc.dg/vla-5.c
    trunk/libgomp/ChangeLog
    trunk/libgomp/testsuite/libgomp.c/appendix-a/a.29.1.c