This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Ping: Re: [patch middle-end]: Fix PR/48814 - [4.4/4.5/4.6/4.7 Regression] Incorrect scalar increment result
- From: Kai Tietz <ktietz70 at googlemail dot com>
- To: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Cc: Richard Guenther <richard dot guenther at gmail dot com>
- Date: Tue, 10 Jan 2012 10:58:46 +0100
- Subject: Ping: Re: [patch middle-end]: Fix PR/48814 - [4.4/4.5/4.6/4.7 Regression] Incorrect scalar increment result
Ping
2012/1/8 Kai Tietz <ktietz70@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 ?<ktietz@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)
> ? ? {
--
|? (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination