Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread Uros Bizjak
; Cc: Jakub Jelinek ; gcc-patches@gcc.gnu.org; Uros Bizjak >> ; Kirill Yukhin ; Lu, Hongjiu >> >> Subject: Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation >> for SKX >> >> On Tue, Nov 14, 2017 at 3:18 AM, Peryt, Sebastian >> wrote: >

Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread H.J. Lu
On Tue, Nov 14, 2017 at 6:20 AM, Jakub Jelinek wrote: > On Tue, Nov 14, 2017 at 06:14:10AM -0800, H.J. Lu wrote: >> Just a thought, should we have a separate patch to add -mprefer-vzeroupper >> to cover all bases in the future, like >> >> /* opt_pass methods: */ >> virtual bool gate (function

RE: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread Peryt, Sebastian
; PR target/82941 > > PR target/82942 > > * gcc.target/i386/pr82941-1.c: New test. > > * gcc.target/i386/pr82941-2.c: New test. > > * gcc.target/i386/pr82942-1.c: New test. > > * gcc.target/i386/pr82942-2.c: New test. > > &

Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread Jakub Jelinek
On Tue, Nov 14, 2017 at 06:14:10AM -0800, H.J. Lu wrote: > Just a thought, should we have a separate patch to add -mprefer-vzeroupper > to cover all bases in the future, like > > /* opt_pass methods: */ > virtual bool gate (function *) > { > return TARGET_AVX && (!TARGET_AVX512ER ||

Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread H.J. Lu
t; PR target/82941 >> PR target/82942 >> * gcc.target/i386/pr82941-1.c: New test. >> * gcc.target/i386/pr82941-2.c: New test. >> * gcc.target/i386/pr82942-1.c: New test. >> * gcc.target/i386/pr82942-2.c: New test. >> >&

Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread H.J. Lu
gcc.target/i386/pr82941-2.c: New test. > * gcc.target/i386/pr82942-1.c: New test. > * gcc.target/i386/pr82942-2.c: New test. > > > Thanks, > Sebastian > >> -Original Message- >> From: Jakub Jelinek [mailto:ja...@redhat.com] >> Sent: Tuesday, N

RE: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread Peryt, Sebastian
stian > -Original Message- > From: Jakub Jelinek [mailto:ja...@redhat.com] > Sent: Tuesday, November 14, 2017 10:51 AM > To: Peryt, Sebastian > Cc: gcc-patches@gcc.gnu.org; Uros Bizjak ; Kirill Yukhin > ; Lu, Hongjiu > Subject: Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper ge

Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread Jakub Jelinek
On Tue, Nov 14, 2017 at 09:45:12AM +, Peryt, Sebastian wrote: > Hi, > > This patch fixes PR82941 and PR82942 by adding vzeroupper generation on SKX. > Bootstrapped and tested. > > 14.11.2017 Sebastian Peryt > > gcc/ In that case the ChangeLog entry should list the PRs, i.e. PR ta

[PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation for SKX

2017-11-14 Thread Peryt, Sebastian
Hi, This patch fixes PR82941 and PR82942 by adding vzeroupper generation on SKX. Bootstrapped and tested. 14.11.2017 Sebastian Peryt gcc/ * config/i386/i386.c (pass_insert_vzeroupper): Modify gate condition to return true on Xeon and not on Xeon Phi. (ix86_check_avx256