[patch] Warn on va_start() when called with a reference parameter

Nico Weber thakis at chromium.org
Thu May 23 22:07:55 PDT 2013


Hi,

it's undefined behavior to call va_start() with a reference parameter. It
happens to work fine with clang, but blows up with other compilers (e.g.
msvs), so this patch adds a warning for it. It also moves 2 other existing
warnings behind the new -Wvarargs flag.

Ok?

Nico
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130523/d1bea381/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clang-vastart-ref.patch
Type: application/octet-stream
Size: 4697 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130523/d1bea381/attachment.obj>


More information about the cfe-commits mailing list