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

Nico Weber thakis at chromium.org
Tue Jun 14 18:04:20 PDT 2011


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]'
  (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.


-------------- 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>

More information about the cfe-commits mailing list