================
@@ -179,9 +227,8 @@ bool BugSuppression::isSuppressed(const BugReport &R) {
// attribute.
//
// The function handles two cases:
-// 1. Instantiation from a class template - searches redeclarations to find
-// the definition (not just a forward declaration).
-// 2. Instantiation from a partial specialization - returns it directly.
+// 1. Class template specializations.
+// 2. Class template partial specializations.
//
// For non-template-specialization decls, returns the input unchanged.
----------------
steakhal wrote:
`For non-template-specialization decls, returns the input unchanged.`
Remains still true. I decided to simplify this because I wanted uniformity and
walk the redecls in both cases.
So I decided to simplify this part. I wrote this part.
If you refer to the introductory part (beginning of this doc comment), that was
written by the AI, and I figured I'd keep it because it gives some context
about the forward declarations - but I'm not opposing dropping that part.
What do you think is AI bloat in particular?
https://github.com/llvm/llvm-project/pull/183727
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
