llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Alfiya Siddique (AlfiyaSiddique)

<details>
<summary>Changes</summary>

## Description.

Completing a comment related to dependent template names that abruptly ended 
and might cause confusion for beginner contributors.

Fixes #<!-- -->96718

---
Full diff: https://github.com/llvm/llvm-project/pull/97322.diff


1 Files Affected:

- (modified) clang/include/clang/AST/TemplateName.h (+2-1) 


``````````diff
diff --git a/clang/include/clang/AST/TemplateName.h 
b/clang/include/clang/AST/TemplateName.h
index 24a7fde76195d..e3b7dd261535d 100644
--- a/clang/include/clang/AST/TemplateName.h
+++ b/clang/include/clang/AST/TemplateName.h
@@ -198,7 +198,8 @@ class SubstTemplateTemplateParmPackStorage : public 
UncommonTemplateNameStorage,
 ///
 /// Here, "apply" is treated as a template name within the typename
 /// specifier in the typedef. "apply" is a nested template, and can
-/// only be understood in the context of
+/// only be understood in the context of a template instantiation,
+/// hence is represented as a dependent template name.
 class TemplateName {
   // NameDecl is either a TemplateDecl or a UsingShadowDecl depending on the
   // NameKind.

``````````

</details>


https://github.com/llvm/llvm-project/pull/97322
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to