diff options
Diffstat (limited to 'clang')
| -rw-r--r-- | clang/include/clang/Basic/Attr.td | 11 | ||||
| -rw-r--r-- | clang/test/Sema/annotate.c | 6 | ||||
| -rw-r--r-- | clang/utils/TableGen/ClangAttrEmitter.cpp | 2 |
3 files changed, 14 insertions, 5 deletions
diff --git a/clang/include/clang/Basic/Attr.td b/clang/include/clang/Basic/Attr.td index 8b84c4b8b50..7bb6c30807b 100644 --- a/clang/include/clang/Basic/Attr.td +++ b/clang/include/clang/Basic/Attr.td @@ -231,9 +231,12 @@ class GCC<string name> : Spelling<name, "GCC"> { let KnownToGCC = 1; } -// The Clang spelling implies GNU<name> and CXX11<"clang", name>. This spelling -// should be used for any Clang-specific attributes. -class Clang<string name> : Spelling<name, "Clang">; +// The Clang spelling implies GNU<name>, CXX11<"clang", name>, and optionally, +// C2x<"clang", name>. This spelling should be used for any Clang-specific +// attributes. +class Clang<string name, bit allowInC = 0> : Spelling<name, "Clang"> { + bit AllowInC = allowInC; +} class Accessor<string name, list<Spelling> spellings> { string Name = name; @@ -618,7 +621,7 @@ def AnalyzerNoReturn : InheritableAttr { } def Annotate : InheritableParamAttr { - let Spellings = [Clang<"annotate">]; + let Spellings = [Clang<"annotate", 1>]; let Args = [StringArgument<"Annotation">]; // Ensure that the annotate attribute can be used with // '#pragma clang attribute' even though it has no subject list. diff --git a/clang/test/Sema/annotate.c b/clang/test/Sema/annotate.c index 4a786d0a016..0d1a1c29a31 100644 --- a/clang/test/Sema/annotate.c +++ b/clang/test/Sema/annotate.c @@ -1,9 +1,13 @@ -// RUN: %clang_cc1 %s -fsyntax-only -verify +// RUN: %clang_cc1 %s -fsyntax-only -fdouble-square-bracket-attributes -verify void __attribute__((annotate("foo"))) foo(float *a) { __attribute__((annotate("bar"))) int x; + [[clang::annotate("bar")]] int x2; __attribute__((annotate(1))) int y; // expected-error {{'annotate' attribute requires a string}} + [[clang::annotate(1)]] int y2; // expected-error {{'annotate' attribute requires a string}} __attribute__((annotate("bar", 1))) int z; // expected-error {{'annotate' attribute takes one argument}} + [[clang::annotate("bar", 1)]] int z2; // expected-error {{'annotate' attribute takes one argument}} + int u = __builtin_annotation(z, (char*) 0); // expected-error {{second argument to __builtin_annotation must be a non-wide string constant}} int v = __builtin_annotation(z, (char*) L"bar"); // expected-error {{second argument to __builtin_annotation must be a non-wide string constant}} int w = __builtin_annotation(z, "foo"); diff --git a/clang/utils/TableGen/ClangAttrEmitter.cpp b/clang/utils/TableGen/ClangAttrEmitter.cpp index b0e2ddd9136..2a8b891e64b 100644 --- a/clang/utils/TableGen/ClangAttrEmitter.cpp +++ b/clang/utils/TableGen/ClangAttrEmitter.cpp @@ -87,6 +87,8 @@ GetFlattenedSpellings(const Record &Attr) { } else if (Variety == "Clang") { Ret.emplace_back("GNU", Name, "", false); Ret.emplace_back("CXX11", Name, "clang", false); + if (Spelling->getValueAsBit("AllowInC")) + Ret.emplace_back("C2x", Name, "clang", false); } else Ret.push_back(FlattenedSpelling(*Spelling)); } |

