r199443 - Move away from 'general' / 'generalized' as a way of identifying C++11
Richard Smith
richard-llvm at metafoo.co.uk
Thu Jan 16 16:11:48 PST 2014
Author: rsmith
Date: Thu Jan 16 18:11:48 2014
New Revision: 199443
URL: http://llvm.org/viewvc/llvm-project?rev=199443&view=rev
Log:
Move away from 'general' / 'generalized' as a way of identifying C++11
attribute syntax. There's nothing generalized about this; it's one of
several first-class attribute syntaxes we support, all of which are
more-or-less equally general.
As discussed on cfe-commits, we may want to revisit this if we start allowing
this syntax as an extension in C (or if C adopts the syntax), but hopefully
this diagnostic wording will be crystal clear to everyone in the mean time.
Modified:
cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
cfe/trunk/test/SemaCXX/cxx98-compat.cpp
cfe/trunk/www/cxx_status.html
Modified: cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td?rev=199443&r1=199442&r2=199443&view=diff
==============================================================================
--- cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td (original)
+++ cfe/trunk/include/clang/Basic/DiagnosticParseKinds.td Thu Jan 16 18:11:48 2014
@@ -517,7 +517,7 @@ def warn_cxx98_compat_nullptr : Warning<
def warn_cxx98_compat_alignas : Warning<"'alignas' is incompatible with C++98">,
InGroup<CXX98Compat>, DefaultIgnore;
def warn_cxx98_compat_attribute : Warning<
- "generalized attributes are incompatible with C++98">,
+ "C++11 attribute syntax is incompatible with C++98">,
InGroup<CXX98Compat>, DefaultIgnore;
def err_cxx11_attribute_forbids_arguments : Error<
"attribute '%0' cannot have an argument list">;
Modified: cfe/trunk/test/SemaCXX/cxx98-compat.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/test/SemaCXX/cxx98-compat.cpp?rev=199443&r1=199442&r2=199443&view=diff
==============================================================================
--- cfe/trunk/test/SemaCXX/cxx98-compat.cpp (original)
+++ cfe/trunk/test/SemaCXX/cxx98-compat.cpp Thu Jan 16 18:11:48 2014
@@ -23,7 +23,7 @@ template<int ...I> // expected-warning
class Variadic3 {};
alignas(8) int with_alignas; // expected-warning {{'alignas' is incompatible with C++98}}
-int with_attribute [[ ]]; // expected-warning {{generalized attributes are incompatible with C++98}}
+int with_attribute [[ ]]; // expected-warning {{C++11 attribute syntax is incompatible with C++98}}
void Literals() {
(void)u8"str"; // expected-warning {{unicode literals are incompatible with C++98}}
Modified: cfe/trunk/www/cxx_status.html
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/www/cxx_status.html?rev=199443&r1=199442&r2=199443&view=diff
==============================================================================
--- cfe/trunk/www/cxx_status.html (original)
+++ cfe/trunk/www/cxx_status.html Thu Jan 16 18:11:48 2014
@@ -191,7 +191,7 @@ currently requires the C++ runtime libra
<td class="full" align="center">Clang 3.1</td>
</tr>
<tr>
- <td>General attributes</td>
+ <td>Standardized attribute syntax</td>
<td><a href="http://www.open-std.org/jtc1/sc22/wg21/docs/papers/2008/n2761.pdf">N2761</a></td>
<td class="full" align="center">Clang 3.3 <a href="#n2761">(1)</a></td>
</tr>
More information about the cfe-commits
mailing list