[cfe-commits] [patch] Warn on sizeof(array_fun_arg)

[Nico Weber]

Hi,

the attached patch produces this diagnostic

test/Sema/warn-sizeof-arrayarg.c:14:15: warning: sizeof on array
function parameter will return size of 'int *' instead of 'int [10]'
[-Wsizeof-array-argument]
  (void)sizeof(a);  // \
              ^
test/Sema/warn-sizeof-arrayarg.c:7:12: note: declared here
void f(int a[10], Arr arr) {  // \
           ^

for this code:

  void f(int a[10]) {
    compute_hash(a, sizeof(a));
  }

It finds 0 bugs and 0 false positives in llvm/clang code and 2 bugs
and 0 false positives in chrome/webkit.

Ok?

Nico
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-sizeof-arrary-arg.patch
Type: application/octet-stream
Size: 3468 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110614/97d97d0c/attachment.obj>