[cfe-commits] r124279 - in /cfe/trunk: include/clang/Basic/DiagnosticSemaKinds.td test/Preprocessor/pragma_diagnostic_sections.cpp test/Sema/uninit-variables.c test/SemaCXX/uninit-variables.cpp

Nico Weber thakis at chromium.org
Wed Jan 26 17:17:15 PST 2011


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 <kremenek at apple.com> 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
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>




More information about the cfe-commits mailing list