Hi Ted, this adds 265 compile errors to the chrome build (we build with -Werror). I went through the first 40 or so, and they're all false positives. The two most common cases are
http://llvm.org/bugs/show_bug.cgi?id=9063 http://llvm.org/bugs/show_bug.cgi?id=9064 (a real bug) I also filed http://llvm.org/bugs/show_bug.cgi?id=9061 http://llvm.org/bugs/show_bug.cgi?id=9062 At the moment, we will have to disable -Wuninitialized in chrome if we want to update clang (which we _always_ want to do :-) ). Nico On Tue, Jan 25, 2011 at 8:49 PM, Ted Kremenek <[email protected]> wrote: > Author: kremenek > Date: Tue Jan 25 22:49:48 2011 > New Revision: 124279 > > URL: http://llvm.org/viewvc/llvm-project?rev=124279&view=rev > Log: > Merge -Wuninitialized-experimental into -Wuninitialized. > > Modified: > cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td > cfe/trunk/test/Preprocessor/pragma_diagnostic_sections.cpp > cfe/trunk/test/Sema/uninit-variables.c > cfe/trunk/test/SemaCXX/uninit-variables.cpp > > Modified: cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td?rev=124279&r1=124278&r2=124279&view=diff > ============================================================================== > --- cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td (original) > +++ cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td Tue Jan 25 22:49:48 > 2011 > @@ -821,9 +821,9 @@ > def note_uninit_reference_member : Note< > "uninitialized reference member is here">; > def warn_field_is_uninit : Warning<"field is uninitialized when used here">, > - InGroup<DiagGroup<"uninitialized">>; > + InGroup<Uninitialized>; > def warn_uninit_var : Warning<"use of uninitialized variable %0">, > - InGroup<DiagGroup<"uninitialized-experimental">>, DefaultIgnore; > + InGroup<Uninitialized>, DefaultIgnore; > def note_uninit_var : Note< > "variable %0 is possibly uninitialized when used here">; > def note_uninit_var_captured_by_block : Note< > > Modified: cfe/trunk/test/Preprocessor/pragma_diagnostic_sections.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Preprocessor/pragma_diagnostic_sections.cpp?rev=124279&r1=124278&r2=124279&view=diff > ============================================================================== > --- cfe/trunk/test/Preprocessor/pragma_diagnostic_sections.cpp (original) > +++ cfe/trunk/test/Preprocessor/pragma_diagnostic_sections.cpp Tue Jan 25 > 22:49:48 2011 > @@ -1,4 +1,4 @@ > -// RUN: %clang_cc1 -fsyntax-only -Wall -Wunused-macros -Wunused-parameter > -verify %s > +// RUN: %clang_cc1 -fsyntax-only -Wall -Wunused-macros -Wunused-parameter > -Wno-uninitialized -verify %s > > // rdar://8365684 > struct S { > > Modified: cfe/trunk/test/Sema/uninit-variables.c > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/Sema/uninit-variables.c?rev=124279&r1=124278&r2=124279&view=diff > ============================================================================== > --- cfe/trunk/test/Sema/uninit-variables.c (original) > +++ cfe/trunk/test/Sema/uninit-variables.c Tue Jan 25 22:49:48 2011 > @@ -1,4 +1,4 @@ > -// RUN: %clang_cc1 -fsyntax-only -Wuninitialized-experimental -fsyntax-only > -fblocks %s -verify > +// RUN: %clang_cc1 -fsyntax-only -Wuninitialized -fsyntax-only -fblocks %s > -verify > > int test1() { > int x; // expected-warning{{use of uninitialized variable 'x'}} > expected-note{{add initialization to silence this warning}} > > Modified: cfe/trunk/test/SemaCXX/uninit-variables.cpp > URL: > http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/uninit-variables.cpp?rev=124279&r1=124278&r2=124279&view=diff > ============================================================================== > --- cfe/trunk/test/SemaCXX/uninit-variables.cpp (original) > +++ cfe/trunk/test/SemaCXX/uninit-variables.cpp Tue Jan 25 22:49:48 2011 > @@ -1,4 +1,4 @@ > -// RUN: %clang_cc1 -fsyntax-only -Wuninitialized-experimental -fsyntax-only > %s -verify > +// RUN: %clang_cc1 -fsyntax-only -Wuninitialized -fsyntax-only %s -verify > > int test1_aux(int &x); > int test1() { > > > _______________________________________________ > 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
