On 13-04-06 11:16 PM, Bin Cheng wrote:
-----Original Message-----
From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org]
On
Behalf Of Bin Cheng
Sent: Monday, March 25, 2013 3:15 PM
To: gcc-patches@gcc.gnu.org
Subject: FW: [PATCH GCC/pr56124] Don't prefer memory if the source of load
operation has side effect
Sorry for the wrong list.
-----Original Message-----
From: Bin Cheng [mailto:bin.ch...@arm.com]
Sent: Monday, March 25, 2013 3:00 PM
To: g...@gcc.gnu.org
Subject: [PATCH GCC/pr56124] Don't prefer memory if the source of load
operation has side effect
Hi,
As reported in PR56124, IRA causes redundant reload by preferring to put
pseudo which is target of loading in memory. Generally this is good but
the
case in which the src of loading has side effect.
This patch fixes this issue by checking whether source of loading has side
effect.
I tested the patch on x86/thumb2. Is it OK? Thanks.
2013-03-25 Bin Cheng <bin.ch...@arm.com>
PR target/56124
* ira-costs.c (scan_one_insn): Check whether the source rtx of
loading has side effect.
Ping.
This patch is ok for trunk. Thanks. And sorry, for the delay with the
answer.
