On Tue, Jan 10, 2012 at 10:58 AM, Kai Tietz <ktiet...@googlemail.com> wrote:
> Ping
>
> 2012/1/8 Kai Tietz <ktiet...@googlemail.com>:
>> Hi,
>>
>> this patch makes sure that for increment of
>> postfix-increment/decrement we use also orignal lvalue instead of tmp
>> lhs value for increment. This fixes reported issue about sequence
>> point in PR/48814
>>
>> ChangeLog
>>
>> 2012-01-08 Kai Tietz <kti...@redhat.com>
>>
>> PR middle-end/48814
>> * gimplify.c (gimplify_self_mod_expr): Use for
>> postfix-inc/dec lvalue instead of temporary
>> lhs.
>>
>> Regression tested for x86_64-unknown-linux-gnu for all languages
>> (including Ada and Obj-C++). Ok for apply?
>>
>> Regards,
>> Kai
>>
>> Index: gimplify.c
>> ===================================================================
>> --- gimplify.c (revision 182720)
>> +++ gimplify.c (working copy)
>> @@ -2258,7 +2258,7 @@
>> arith_code = POINTER_PLUS_EXPR;
>> }
>>
>> - t1 = build2 (arith_code, TREE_TYPE (*expr_p), lhs, rhs);
>> + t1 = build2 (arith_code, TREE_TYPE (*expr_p), lvalue, rhs);
>>
>> if (postfix)
>> {
Please add testcases. Why does your patch make a difference?
lhs is just the gimplified lvalue.
Richard.
>
>
> --
> | (\_/) This is Bunny. Copy and paste
> | (='.'=) Bunny into your signature to help
> | (")_(") him gain world domination
