[PATCH] Variable templates w/ partial support for static data members

Larisse Voufo lvoufo at google.com
Tue Aug 6 00:43:43 PDT 2013


On Mon, Aug 5, 2013 at 5:04 PM, Richard Smith <richard at metafoo.co.uk> wrote:
>
>   OK, please go ahead and commit this, all the FIXMEs look fine to handle in subsequent commits. Many thanks!
>

Done. Thank you!

>
> ================
> Comment at: test/SemaCXX/cxx1y-variable-templates_in_class.cpp:1-2
> @@ +1,3 @@
> +// RUN: %clang_cc1 -verify -fsyntax-only %s -w -DPRECXX11
> +// RUN: %clang_cc1 -std=c++11 -verify -fsyntax-only -w %s
> +// RUN: %clang_cc1 -std=c++1y -verify -fsyntax-only %s
> ----------------
> Please use -Wno-c++11-extensions -Wno-c++1y-extensions here, rather than -w.
>
> ================
> Comment at: test/SemaCXX/cxx1y-variable-templates_top_level.cpp:1-2
> @@ +1,3 @@
> +// RUN: %clang_cc1 -verify -fsyntax-only -w %s
> +// RUN: %clang_cc1 -std=c++11 -verify -fsyntax-only -w %s -DCXX11
> +// RUN: %clang_cc1 -std=c++1y -verify -fsyntax-only %s -DCXX11
> ----------------
> Likewise.
>
>
> http://llvm-reviews.chandlerc.com/D1067



More information about the cfe-commits mailing list