2014-06-12 20:52 GMT+02:00 Segher Boessenkool <seg...@kernel.crashing.org>: > On Thu, Jun 12, 2014 at 06:21:32PM +0200, Kai Tietz wrote: >> with addition of adding a second peephole2 pass after sched2 pass, I >> was able to get some improvement for PR target/39284. I think by this >> addition we can close bug as fixed. >> Additionally additional peephole2 pass shows better results for PR >> target/51840 testcase with disabled ASM_GOTO, too.
Well, this is the only point I am a bit concerned too. In general I wouldn't expect here any issues to run peephole after scheduling, as peephole doesn't do anything a new run of ira/lra would require. Anyway it would be good if a global maintainer could comment on that. > Will that work on other targets? Also, it needs a doc fix (md.texi > says peephole2 runs before scheduling). Thanks for pointing on that. When I send patch for this additional peephole pass with testcase, I will adjust md.texi. > > Segher
