This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [patch i386]: Combine memory and indirect jump
- From: Kai Tietz <ktietz70 at googlemail dot com>
- To: Segher Boessenkool <segher at kernel dot crashing dot org>
- Cc: Steven Bosscher <stevenb dot gcc at gmail dot com>, GCC Patches <gcc-patches at gcc dot gnu dot org>, Richard Henderson <rth at redhat dot com>
- Date: Thu, 12 Jun 2014 22:40:58 +0200
- Subject: Re: [patch i386]: Combine memory and indirect jump
- Authentication-results: sourceware.org; auth=none
- References: <CAEwic4brJeBvoe+J5ss=Qo+=qoo-=2nV0FnjdUxBhm-fV4aqeQ at mail dot gmail dot com> <CABu31nNwUoLaAo0QcD-3O1QYhBWpLsYuH0cMS-XOgz2W+8KMAA at mail dot gmail dot com> <CAEwic4Zwd4HECD+kxtkouyA3Urbyzh2NFar7kZ5XLdNnUK9w6A at mail dot gmail dot com> <CAEwic4anzQysfHqfQGgKF_Hu-c_hLY+mkWr2CzERVe=gQ5AWRw at mail dot gmail dot com> <20140612185258 dot GA9914 at gate dot crashing dot org>
2014-06-12 20:52 GMT+02:00 Segher Boessenkool <segher@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