On 13-04-06 11:16 PM, Bin Cheng wrote:
-----Original Message-----
From: [email protected] [mailto:[email protected]]
On
Behalf Of Bin Cheng
Sent: Monday, March 25, 2013 3:15 PM
To: [email protected]
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:[email protected]]
Sent: Monday, March 25, 2013 3:00 PM
To: [email protected]
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 <[email protected]>
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.
