[clang] fix: complete comment (PR #97322)

via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 1 10:17:40 PDT 2024


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


More information about the cfe-commits mailing list