[clang] [Clang] Add template argument support for {con,de}structor attributes. (PR #151400)

[via cfe-commits]

================
@@ -2152,29 +2152,44 @@ static void handleUnusedAttr(Sema &S, Decl *D, const ParsedAttr &AL) {
   D->addAttr(::new (S.Context) UnusedAttr(S.Context, AL));
 }
 
+static std::optional<Expr *> sharedGetConstructorDestructorAttrExpr(Sema &S, const ParsedAttr &AL) {
+  Expr *E = nullptr;
+  if (AL.getNumArgs() == 1) {
+    E = AL.getArgAsExpr(0);
+    if (E->isValueDependent()) {
----------------
Sirraide wrote:

Since you’re only checking `isValueDependent()` here, can you add a test to see what happens if an expression is only type-dependent, e.g.
```c++
template <typename T>
[[gnu::constructor(static_cast<T>(42))]] void foo() {}

template void foo<int>();
```

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