[cfe-commits] r72460 - /cfe/trunk/lib/Sema/SemaExpr.cpp

[Chris Lattner]

On May 26, 2009, at 10:00 PM, Douglas Gregor wrote:

> Author: dgregor
> Date: Wed May 27 00:00:47 2009
> New Revision: 72460
>
> URL: http://llvm.org/viewvc/llvm-project?rev=72460&view=rev
> Log:
> Don't drop the computed implicit conversions when building a call to
> overloaded operator[] or overloaded postfix unary operator (++,
> --). Thanks to Eli for finding this bug!

Testcase? :)

-Chris

>
>
> Modified:
>    cfe/trunk/lib/Sema/SemaExpr.cpp
>
> Modified: cfe/trunk/lib/Sema/SemaExpr.cpp
> URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Sema/SemaExpr.cpp?rev=72460&r1=72459&r2=72460&view=diff
>
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- cfe/trunk/lib/Sema/SemaExpr.cpp (original)
> +++ cfe/trunk/lib/Sema/SemaExpr.cpp Wed May 27 00:00:47 2009
> @@ -1609,6 +1609,7 @@
>         UsualUnaryConversions(FnExpr);
>
>         Input.release();
> +        Args[0] = Arg;
>         return Owned(new (Context) CXXOperatorCallExpr(Context,  
> OverOp, FnExpr,
>                                                        Args, 2,  
> ResultTy,
>                                                        OpLoc));
> @@ -1725,6 +1726,8 @@
>
>         Base.release();
>         Idx.release();
> +        Args[0] = LHSExp;
> +        Args[1] = RHSExp;
>         return Owned(new (Context) CXXOperatorCallExpr(Context,  
> OO_Subscript,
>                                                        FnExpr, Args,  
> 2,
>                                                        ResultTy,  
> LLoc));
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits