================
@@ -45,7 +45,7 @@ RecordTy *addTopLevelRecord(DenseMap<StringRef, APIRecord *> 
&USRLookupTable,
 
 NamespaceRecord *
 APISet::addNamespace(APIRecord *Parent, StringRef Name, StringRef USR,
-                     PresumedLoc Loc, AvailabilitySet Availability,
+                     PresumedLoc Loc, const AvailabilityInfo &Availability,
----------------
daniel-grumberg wrote:

Can we keep these as plain values (i.e. just `AvailabilityInfo Availability`), 
this prevents the move operation below and leads to a copy. Additionally, this 
prevents copy ellision with how we pass the parameters in.

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

Reply via email to