llvmbot wrote:

<!--LLVM PR SUMMARY COMMENT-->

@llvm/pr-subscribers-clang

Author: Shafik Yaghmour (shafik)

<details>
<summary>Changes</summary>

As noted in this bug report: https://github.com/llvm/llvm-project/issues/37647

Due to this clang bug we added the template keyword in Redeclarable.h. The bug 
has been fixed since then, so removing it.

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


1 Files Affected:

- (modified) clang/include/clang/AST/Redeclarable.h (+1-3) 


``````````diff
diff --git a/clang/include/clang/AST/Redeclarable.h 
b/clang/include/clang/AST/Redeclarable.h
index 091bb886f2d4962..be022bf5322ad61 100644
--- a/clang/include/clang/AST/Redeclarable.h
+++ b/clang/include/clang/AST/Redeclarable.h
@@ -114,9 +114,7 @@ class Redeclarable {
 
     bool isFirst() const {
       return Link.is<KnownLatest>() ||
-             // FIXME: 'template' is required on the next line due to an
-             // apparent clang bug.
-             Link.get<NotKnownLatest>().template is<UninitializedLatest>();
+             Link.get<NotKnownLatest>().is<UninitializedLatest>();
     }
 
     decl_type *getPrevious(const decl_type *D) const {

``````````

</details>


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

Reply via email to