================ @@ -62,6 +62,7 @@ class SemaSYCL : public SemaBase { ParsedType ParsedTy); void handleKernelAttr(Decl *D, const ParsedAttr &AL); + void handleSYCLExternalAttr(Decl *D, const ParsedAttr &AL); ---------------- tahonermann wrote:
Since this function is a member of the `SemaSYCL` class, I think we can drop "SYCL" from its name. This is consistent with what was done for the other attribute handlers. ```suggestion void handleExternalAttr(Decl *D, const ParsedAttr &AL); ``` https://github.com/llvm/llvm-project/pull/140282 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits