On Tue, Nov 14, 2017 at 3:26 PM, Peryt, Sebastian <sebastian.pe...@intel.com> wrote: > Attached is fixed patch. > > Sebastian > > >> -----Original Message----- >> From: H.J. Lu [mailto:hjl.to...@gmail.com] >> Sent: Tuesday, November 14, 2017 1:18 PM >> To: Peryt, Sebastian <sebastian.pe...@intel.com> >> Cc: Jakub Jelinek <ja...@redhat.com>; gcc-patches@gcc.gnu.org; Uros Bizjak >> <ubiz...@gmail.com>; Kirill Yukhin <kirill.yuk...@gmail.com>; Lu, Hongjiu >> <hongjiu...@intel.com> >> Subject: Re: [PATCH][i386] PR82941/PR82942 - Adding vzeroupper generation >> for SKX >> >> On Tue, Nov 14, 2017 at 3:18 AM, Peryt, Sebastian <sebastian.pe...@intel.com> >> wrote: >> > I have updated tests and changelog according to Jakub's suggestions. >> > Please find attached v2 of my patch. >> > >> > >> > 14.11.2017 Sebastian Peryt <sebastian.pe...@intel.com> >> > >> > gcc/ >> > >> > PR target/82941 >> > PR target/82942 >> > * config/i386/i386.c (pass_insert_vzeroupper): Modify gate >> > condition >> > to return true on Xeon and not on Xeon Phi. >> > (ix86_check_avx256_register): Changed to ... >> > (ix86_check_avx_upper_register): ... this. Add extra check for >> > VALID_AVX512F_REG_OR_XI_MODE. >> > (ix86_avx_u128_mode_needed): Changed >> > ix86_check_avx256_register to ix86_check_avx_upper_register. >> > (ix86_check_avx256_stores): Changed to ... >> > (ix86_check_avx_upper_stores): ... this. Changed >> > ix86_check_avx256_register to ix86_check_avx_upper_register. >> > (ix86_avx_u128_mode_after): Changed >> > avx_reg256_found to avx_upper_reg_found. Changed >> > ix86_check_avx256_stores to ix86_check_avx_upper_stores. >> > (ix86_avx_u128_mode_entry): Changed >> > ix86_check_avx256_register to ix86_check_avx_upper_register. >> > (ix86_avx_u128_mode_exit): Ditto. >> > * config/i386/i386.h: (host_detect_local_cpu): New define.
OK. Thanks, Uros.
