[cfe-commits] r133039 - /cfe/trunk/test/SemaCXX/warn-memset-bad-sizeof.cpp

Nico Weber nicolasweber at gmx.de
Wed Jun 15 00:00:58 PDT 2011


On 14.06.2011, at 22:00, Chandler Carruth wrote:

> On Tue, Jun 14, 2011 at 9:50 PM, Nico Weber <nicolasweber at gmx.de> wrote:
> -// RUN: %clang_cc1 -fsyntax-only -verify %s
> +// RUN: %clang_cc1 -fsyntax-only -verify -Wno-sizeof-array-argument %s
> 
> Why not just fold the tests array-based warnings on bad sizeof into this test, and mark them appropriately? The seem closely related anyways...

Dunno, one of them is sizeof a function parameter, the other is memset/memcpy used with a sizeof, which doesn't seem that similar to me. I don't feel very strongly either way though.

Nico



More information about the cfe-commits mailing list