================
@@ -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

Reply via email to