From: John M. <ato...@gm...> - 2009-12-09 19:43:55
|
LOL, I must have had a bad build after I added that because original I didn't see any changes. On Wed, Dec 9, 2009 at 7:08 AM, Josef Eisl <za...@za...> wrote: > John Myers wrote: > > What's up with the redundant mov? Or will future optimization take > > care of removing this? > > > > > ret > > > .size func, .-func > > > > > > > > > > The redudant mov is gone now. So it looks like whatever they changed in > > the LLVM trunk fixed it. > > Nope. You fixed it in r53 ;)! The isMoveInstr() allows llvm to detect > redundant MOVs. > BTW: great work! > > Josef > |