On Fri, May 24, 2013 at 9:23 AM, Jordan Rose <[email protected]> wrote:
> Your test case is named .cc (rather than .cpp) and as such is not getting > run...which is important because it's actually broken. > 9_9 Fixed, thanks. > All of va_start is implementation-defined behavior anyway, yes? I guess > it's still a good warning, though. > http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2005/n1905.pdf 18.7p3 explicitly calls this (and some other things) out as undefined, not implementation-defined. > > 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 > > > _______________________________________________ > > cfe-commits mailing list > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits > > >
clang-vastart-ref.patch
Description: Binary data
_______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
