This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR bootstrap/40788 / problem with location in builtins.c:gimplify_va_arg_expr
- From: Richard Guenther <richard dot guenther at gmail dot com>
- To: Laurent GUERBY <laurent at guerby dot net>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>, Eric Botcazou <ebotcazou at adacore dot com>, Aldy Hernandez <aldyh at redhat dot com>, Manuel López-Ibáñez <manu at gcc dot gnu dot org>
- Date: Tue, 11 Aug 2009 11:11:01 +0200
- Subject: Re: [PATCH] Fix PR bootstrap/40788 / problem with location in builtins.c:gimplify_va_arg_expr
- References: <1249943539.20287.8.camel@localhost>
On Tue, Aug 11, 2009 at 12:32 AM, Laurent GUERBY<laurent@guerby.net> wrote:
> Hi,
>
> As suggested by Richard on the PR:
>
> http://gcc.gnu.org/bugzilla/show_bug.cgi?id=40788
>
> I tested that the following patch and it restores c,ada bootstrap on
> sparc --with-cpu=v8
>
> Ok to commit?
Ok.
Thanks,
Richard.
> Laurent
>
> 2009-08-11 ?Richard Guenther ?<rguenther@suse.de>
>
> ? ? ? ?PR bootstrap/40788
> ? ? ? ?* builtins.c (gimplify_va_arg_expr): Do not call
> ? ? ? ?SET_EXPR_LOCATION.
>
> Index: gcc/builtins.c
> ===================================================================
> --- gcc/builtins.c ? ? ?(revision 150589)
> +++ gcc/builtins.c ? ? ?(working copy)
> @@ -5104,7 +5104,6 @@
> ? ? ? ?return GS_ALL_DONE;
>
> ? ? ? *expr_p = targetm.gimplify_va_arg_expr (valist, type, pre_p, post_p);
> - ? ? ?SET_EXPR_LOCATION (*expr_p, loc);
> ? ? ? return GS_OK;
> ? ? }
> ?}
>
>
>
>