llvmorg-github-actions[bot] wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Neha  (nehaGautam07)

<details>
<summary>Changes</summary>

Fixes #<!-- -->143719

`[[clang::no_specializations]]` diagnoses both explicit specializations
and partial specializations. The old wording, "explicitly specialized",
is ambiguous because that term means only `template &lt;&gt;` specializations
in the C++ standard.

This uses the wording agreed in #<!-- -->143839.

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


1 Files Affected:

- (modified) clang/include/clang/Basic/AttrDocs.td (+2-2) 


``````````diff
diff --git a/clang/include/clang/Basic/AttrDocs.td 
b/clang/include/clang/Basic/AttrDocs.td
index 5ec2a68bc038b..95e27ad9a47c4 100644
--- a/clang/include/clang/Basic/AttrDocs.td
+++ b/clang/include/clang/Basic/AttrDocs.td
@@ -1466,8 +1466,8 @@ Query for this feature with 
`__has_attribute(diagnose_if)`.
 def NoSpecializationsDocs : Documentation {
   let Category = DocCatDecl;
   let Content = [{
-`[[clang::no_specializations]]` can be applied to function, class, or variable
-templates which should not be explicitly specialized by users. This is 
primarily
+``[[clang::no_specializations]]`` can be applied to function, class, or 
variable
+templates for which neither an explicit specialization nor a partial 
specialization should be declared by users. This is primarily
 used to diagnose user specializations of standard library type traits.
   }];
 }

``````````

</details>


https://github.com/llvm/llvm-project/pull/218758
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to