This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 01/12] add default definition of EH_RETURN_DATA_REGNO
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jeff Law <law at redhat dot com>
- Cc: tbsaunde+gcc at tbsaunde dot org, gcc-patches at gcc dot gnu dot org
- Date: Tue, 21 Apr 2015 16:00:22 +0200
- Subject: Re: [PATCH 01/12] add default definition of EH_RETURN_DATA_REGNO
- Authentication-results: sourceware.org; auth=none
- References: <1429622658-9034-1-git-send-email-tbsaunde+gcc at tbsaunde dot org> <1429622658-9034-2-git-send-email-tbsaunde+gcc at tbsaunde dot org> <55365355 dot 1030002 at redhat dot com>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Apr 21, 2015 at 07:40:37AM -0600, Jeff Law wrote:
> On 04/21/2015 07:24 AM, tbsaunde+gcc@tbsaunde.org wrote:
> >From: Trevor Saunders <tbsaunde+gcc@tbsaunde.org>
> >
> >gcc/ChangeLog:
> >
> >2015-04-21 Trevor Saunders <tbsaunde+gcc@tbsaunde.org>
> >
> > * defaults.h: New definition of EH_RETURN_DATA_REGNO.
> > * except.c: Remove definition of EH_RETURN_DATA_REGNO.
> > * builtins.c (expand_builtin): Remove check if
> > EH_RETURN_DATA_REGNO is defined.
> > * df-scan.c (df_bb_refs_collect): Likewise.
> > (df_get_exit_block_use_set): Likewise.
> > * haifa-sched.c (initiate_bb_reg_pressure_info): Likewise.
> > * ira-lives.c (process_bb_node_lives): Likewise.
> > * lra-lives.c (process_bb_lives): Likewise.
> This one wasn't as obvious as the others, but is clearly OK once the full
> loops being guarded by EH_RETURN_DATA_REGNO are examined.
Except that the bb_has_eh_pred predicate might burn CPU time for basic
blocks with many predecessors. Though, the question is if there are any
important targets that don't define EH_RETURN_DATA_REGNO already.
Jakub