This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: Patch 3/9: create some more small helper functions

From: Jeff Law <law at redhat dot com>
To: Bernd Schmidt <bernds at codesourcery dot com>
Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
Date: Fri, 18 Jun 2010 13:07:36 -0600
Subject: Re: Patch 3/9: create some more small helper functions
References: <4C1B7BC0.7010803@codesourcery.com> <4C1B7D5B.6060803@codesourcery.com>

On 06/18/10 08:06, Bernd Schmidt wrote:

    * ira-int.h (ira_parent_allocno, ira_parent_or_cap_allocno): Declare.
    * ira-build.c (ira_parent_allocno, ira_parent_or_cap_allocno): New
    functions.
    (ira_flattening): Use ira_parent_allocno.
    * ira-conflicts.c (process_regs_for_copy, propagate_copies)
    build_allocno_conflicts): Use ira_parent_or_cap_allocno.

I agree the original code was rather confusing. It wasn't trivial to verify that the code would work the same before/after your changes, primarily due to the convoluted way some of the original code was written.

Thanks. Please install,

Jeff

References:
- Patch 0/9: IRA cleanups and preparations for tracking subwords of DImode
  - From: Bernd Schmidt
- Patch 3/9: create some more small helper functions
  - From: Bernd Schmidt

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]