> -----Original Message----- > From: Vladimir Makarov [mailto:[email protected]] > Sent: Thursday, April 11, 2013 7:20 AM > To: Bin Cheng > Cc: [email protected] > Subject: Re: [PATCH GCC/pr56124] Don't prefer memory if the source of load > operation has side effect > > 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. > Committed as r197691.
Thanks.
