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 <[email protected]>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
> [email protected]
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
>
>
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to