[cfe-commits] r63991 - /cfe/trunk/lib/Analysis/GRExprEngine.cpp

Sebastian Redl sebastian.redl at getdesigned.at
Fri Feb 6 16:58:52 PST 2009


Ted Kremenek wrote:
> Author: kremenek
> Date: Fri Feb  6 18:52:24 2009
> New Revision: 63991
>
> URL: http://llvm.org/viewvc/llvm-project?rev=63991&view=rev
> Log:
> GRExprEngine: When processing compound assignments, do a switch table lookup to get the non-compound opcode from the compound opcode instead of relying on the order of BinaryOperator::opcode values.  This unbreaks the misc-ps.c test.
>   
Actually, you could rely on the order (they're in precedence order),
just not on the starting point. The assumption was that Mul is the first
value.
But the switch is definitely more robust.

Sebastian



More information about the cfe-commits mailing list