This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Don't run var-tracking for targetm.no_register_allocation targets
- From: Richard Biener <rguenther at suse dot de>
- To: Jakub Jelinek <jakub at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Thu, 26 Mar 2015 12:20:19 +0100 (CET)
- Subject: Re: [PATCH] Don't run var-tracking for targetm.no_register_allocation targets
- Authentication-results: sourceware.org; auth=none
- References: <20150326110015 dot GW1746 at tucnak dot redhat dot com>
On Thu, 26 Mar 2015, Jakub Jelinek wrote:
> Hi!
>
> As discussed earlier, var-tracking assumes no pseudos are in the IL
> at the point it is run, which isn't the case for nvptx.
> While it would be also reasonable to kill debug stmts earlier or
> disable var-tracking, making it clear that var-tracking can't handle this
> case is IMHO desirable too.
>
> Ok for trunk?
Ok.
Thanks,
Richard.
> 2015-03-26 Jakub Jelinek <jakub@redhat.com>
>
> * var-tracking.c (variable_tracking_main_1): Don't track
> variables for targetm.no_register_allocation targets.
>
> --- gcc/var-tracking.c.jj 2015-03-10 07:35:30.000000000 +0100
> +++ gcc/var-tracking.c 2015-03-24 16:55:07.611328164 +0100
> @@ -10311,7 +10311,10 @@ variable_tracking_main_1 (void)
> {
> bool success;
>
> - if (flag_var_tracking_assignments < 0)
> + if (flag_var_tracking_assignments < 0
> + /* Var-tracking right now assumes the IR doesn't contain
> + any pseudos at this point. */
> + || targetm.no_register_allocation)
> {
> delete_debug_insns ();
> return 0;
>
> Jakub
>
>
--
Richard Biener <rguenther@suse.de>
SUSE LINUX GmbH, GF: Felix Imendoerffer, Jane Smithard, Jennifer Guild,
Dilip Upmanyu, Graham Norton HRB 21284 (AG Nuernberg)