Your test case is named .cc (rather than .cpp) and as such is not getting run...which is important because it's actually broken.
All of va_start is implementation-defined behavior anyway, yes? I guess it's still a good warning, though. Jordan On May 23, 2013, at 22:07 , Nico Weber <[email protected]> wrote: > 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
clang-vastart-ref.patch
Description: Binary data
> _______________________________________________ > cfe-commits mailing list > [email protected] > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
