================
@@ -407,10 +431,21 @@ class DefineOutline : public Tweak {
return !SameFile;
}
- // Bail out in templated classes, as it is hard to spell the class name,
- // i.e if the template parameter is unnamed.
- if (MD->getParent()->isTemplated())
- return false;
+ for (const CXXRecordDecl *Parent = MD->getParent(); Parent;
+ Parent =
+ llvm::dyn_cast_or_null<const CXXRecordDecl>(Parent->getParent()))
{
+ if (auto Params = Parent->getDescribedTemplateParams()) {
+
+ // Class template member functions must be defined in the
+ // same file.
+ SameFile = true;
+
+ for (NamedDecl *P : *Params) {
+ if (!P->getIdentifier())
+ return false;
----------------
5chmidti wrote:
A comment explaining why we bail out here would be good. FWICT, it is because
we can't spell out unnamed params in the outlined definition?
https://github.com/llvm/llvm-project/pull/95235
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
