[patch] Make "C++98 requires an accessible copy constructor" warning DefaultIgnore

Nico Weber thakis at chromium.org
Wed Sep 17 19:19:52 PDT 2014


On Wed, Sep 17, 2014 at 3:43 PM, Richard Smith <richard at metafoo.co.uk>
wrote:

> Index: test/SemaCXX/cxx98-compat.cpp
> ===================================================================
> --- test/SemaCXX/cxx98-compat.cpp (revision 217313)
> +++ test/SemaCXX/cxx98-compat.cpp (working copy)
> @@ -1,5 +1,5 @@
> -// RUN: %clang_cc1 -fsyntax-only -std=c++11 -Wc++98-compat -verify %s
> -// RUN: %clang_cc1 -fsyntax-only -std=c++1y -Wc++98-compat -verify %s
> -DCXX14COMPAT
> +// RUN: %clang_cc1 -fsyntax-only -std=c++11 -Wc++98-compat
> -Wc++98-compat-bind-to-temporary-copy -verify %s
> +// RUN: %clang_cc1 -fsyntax-only -std=c++14 -Wc++98-compat
> -Wc++98-compat-bind-to-temporary-copy -verify %s -DCXX14COMPAT
>
> Please move the relevant tests to test/SemaCXX/cxx98-compat-pedantic.cpp
> instead of adding the warning flag here.
>
> Otherwise, LGTM, thanks!
>

r218008, thanks!
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20140917/60024bf8/attachment.html>


More information about the cfe-commits mailing list