This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH GCC][2/6]Compute available register for each register classes
- From: Jeff Law <law at redhat dot com>
- To: Bin Cheng <Bin dot Cheng at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: nd <nd at arm dot com>
- Date: Fri, 23 Jun 2017 10:54:16 -0600
- Subject: Re: [PATCH GCC][2/6]Compute available register for each register classes
- Authentication-results: sourceware.org; auth=none
- Authentication-results: ext-mx07.extmail.prod.ext.phx2.redhat.com; dmarc=none (p=none dis=none) header.from=redhat.com
- Authentication-results: ext-mx07.extmail.prod.ext.phx2.redhat.com; spf=pass smtp.mailfrom=law at redhat dot com
- Dkim-filter: OpenDKIM Filter v2.11.0 mx1.redhat.com B2948C049D59
- Dmarc-filter: OpenDMARC Filter v1.3.2 mx1.redhat.com B2948C049D59
- References: <VI1PR0802MB2176CE2CE8B245583E36A195E7E20@VI1PR0802MB2176.eurprd08.prod.outlook.com>
On 05/12/2017 05:27 AM, Bin Cheng wrote:
> Hi,
> Currently available/clobber registers are computed only for GENERAL_REGS, this
> patch extends it for all reg pressure classes. It also updates existing uses
> in various places.
>
> Bootstrap and test on x86_64 and AArch64. Is it OK?
>
> Thanks,
> bin
> 2017-05-10 Bin Cheng <bin.cheng@arm.com>
>
> * cfgloop.h (struct target_cfgloop): Change x_target_avail_regs and
> x_target_clobbered_regs into array fields.
> (init_avail_clobber_regs): New declaration.
> * cfgloopanal.c (memmodel.h, ira.h): Include header files.
> (init_set_costs): Remove computation for old x_target_avail_regs and
> x_target_clobbered_regs fields.
> (init_avail_clobber_regs): New function.
> (estimate_reg_pressure_cost): Update the uses.
> * toplev.c (cfgloop.h): Update comment why the header file is needed.
> (backend_init_target): Call init_avail_clobber_regs.
> * tree-predcom.c (memmodel.h, ira.h): Include header files.
> (MAX_DISTANCE): Update the use.
> * tree-ssa-loop-ivopts.c (determine_set_costs): Update the uses.
> (determine_set_costs): Ditto.
>
OK.
jeff