On 14/6/16 5:55 PM, James Greenhalgh wrote: > On Fri, Jun 13, 2014 at 05:46:45PM +0100, Vladimir Makarov wrote: >> On 2014-06-11, 1:17 PM, Chung-Lin Tang wrote: >>> Looking at this too, as an LRA exercise. I don't really think the >>> pattern is wrong, rather LRA should just avoid creating the copy in this >>> case; it's a result of operand constraining, after all. >>> >>> Attached is the small LRA patch, pending testing. Vladimir should weight >>> in on this. >>> >> >> The patch is safe and ok. Thanks for working on it, Chung-Lin. > > As this patch fixes a build failure on ARM I'd like to have it applied > today. If I don't hear anything which would stop me, I'll commit this on > Chung-Lin's behalf in a few hours. > > Cheers, > James
I just committed it, after a testsuite run on x86_64 (to be sure). And thanks to James for doing the ARM tests. Thanks, Chung-Lin >> >>> * ira-lives.c (process_bb_lives): Skip creating copy during >>> insn sca when src/dest has constrained to same regno. >>> >> >>
