[clang] [clang] All {con, de}structor attributes to use template args (PR #67376)

Erich Keane via cfe-commits cfe-commits at lists.llvm.org
Mon Sep 25 17:24:54 PDT 2023


================
@@ -1422,9 +1424,11 @@ def Deprecated : InheritableAttr {
 
 def Destructor : InheritableAttr {
   let Spellings = [GCC<"destructor">];
-  let Args = [DefaultIntArgument<"Priority", 65535>];
+  let Args = [ExprArgument<"Priority", 1>];
   let Subjects = SubjectList<[Function]>;
   let Documentation = [CtorDtorDocs];
+  let TemplateDependent = 1;
+  let AdditionalMembers = [{ static const int DefaultPriority = 65535; }];
----------------
erichkeane wrote:

```suggestion
  let AdditionalMembers = [{ static constexpr int DefaultPriority = 65535; }];
```

Also wonder if these should be `unsigned`?

https://github.com/llvm/llvm-project/pull/67376


More information about the cfe-commits mailing list