[cfe-commits] r72344 - in /cfe/trunk/lib/CodeGen: CGExpr.cpp CGExprAgg.cpp

[Daniel Dunbar]

Hi Mike,

On Sat, May 23, 2009 at 2:40 PM, Mike Stump <mrs at apple.com> wrote:
>  void AggExprEmitter::VisitBinComma(const BinaryOperator *E) {
> -  CGF.EmitAnyExpr(E->getLHS());
> -  CGF.EmitAggExpr(E->getRHS(), DestPtr, false);
> +  CGF.EmitAnyExprToTemp(E->getLHS(), 0, VolatileDest);

I don't think this is correct. The volatility of the destiniation
doesn't effect how the LHS of "a, b" is evaluated?

 - Daniel