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 GCC/ARM]Fix rtx cost for Thumb1

From: Richard Earnshaw <rearnsha at arm dot com>
To: Bin Cheng <bin dot cheng at arm dot com>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Tue, 26 Mar 2013 10:57:17 +0000
Subject: Re: [PATCH GCC/ARM]Fix rtx cost for Thumb1
References: <002601ce29fc$afd803b0$0f880b10$ at cheng@arm.com>

On 26/03/13 08:34, Bin Cheng wrote:

Hi,
As reported in PR56102, arm back end returns wrong rtx cost for pattern
SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT with multi-word mode. This
causes GCC skipping the split process in lower-subreg.c, and generating
bigger constant pool.

This patch fixes the issue. Tested on arm-none-eabi/thumb1/O2/Os, ok for
trunk?

Thanks.

2013-03-26  Bin Cheng  <bin.cheng@arm.com>

	PR target/56102
	* config/arm/arm.c (thumb1_rtx_costs, thumb1_size_rtx_costs): Fix
	rtx costs for SET/ASHIFT/ASHIFTRT/LSHIFTRT/ROTATERT patterns with
	mult-word mode.


OK.

R.

Follow-Ups:
- RE: [PATCH GCC/ARM]Fix rtx cost for Thumb1
  - From: Bin Cheng

References:
- [PATCH GCC/ARM]Fix rtx cost for Thumb1
  - From: Bin Cheng

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