[PATCH]ArrayRef'ize Sema::AddMemberOperatorCandidates, Sema::AddBuiltinCandidate and Sema::AddBuiltinOperatorCandidates

Richard Smith richard at metafoo.co.uk
Sun May 5 08:52:13 PDT 2013


Thanks, committed as r181158 with some minor cleanups:

   Expr *Args[2] = { LHS.get(), RHS.get() };
   OverloadCandidateSet CandidateSet(QuestionLoc);
-  Self.AddBuiltinOperatorCandidates(OO_Conditional, QuestionLoc, Args, 2,
-                                    CandidateSet);
+  Self.AddBuiltinOperatorCandidates(OO_Conditional, QuestionLoc,
+                                llvm::makeArrayRef<Expr *>(Args, 2),
CandidateSet);

Line over 80 columns. Also, you don't need makeArrayRef to convert an array
to an ArrayRef, you can just pass Args directly to the function here.


On Sun, May 5, 2013 at 8:22 AM, Robert Wilhelm <robert.wilhelm at gmx.net>wrote:

> This patch converts three methods to ArrayRef in SemaOverload.cpp.
> No functionality change.
> Passes make test on x86_64-unknown-linux-gnu
>
> Please commit.
>
> Thanks,
> Robert
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20130505/9495f2aa/attachment.html>


More information about the cfe-commits mailing list