================
@@ -732,6 +732,16 @@ 
CodeGenTypes::arrangeBuiltinFunctionDeclaration(CanQualType resultType,
                                  RequiredArgs::All);
 }
 
+const CGFunctionInfo &
+CodeGenTypes::arrangeSYCLKernelCallerDeclaration(QualType resultType,
+                                                 const FunctionArgList &args) {
+  auto argTypes = getArgTypesForDeclaration(Context, args);
----------------
tahonermann wrote:

I don't think these uses violate the LLVM coding standard. The relevant 
guidelines are linked below.
- 
https://llvm.org/docs/CodingStandards.html#use-auto-type-deduction-to-make-code-more-readable
- https://llvm.org/docs/CodingStandards.html#beware-unnecessary-copies-with-auto

It looks to me like `getArgTypesForDeclaration()` follows NRVO, so copies 
should be avoided.

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

Reply via email to