Re: [PATCH] x86: Add -march=cascadelake

2018-12-16 Thread Uros Bizjak
On Sun, Dec 16, 2018 at 4:15 PM Wei Xiao wrote: > > Thanks for the comments! > Fixed as attached. > Ok for trunk? OK. Thanks, Uros. > Jakub Jelinek 于2018年12月14日周五 下午6:47写道: > > > > On Fri, Dec 14, 2018 at 06:33:37PM +0800, Wei Xiao wrote: > > --- a/gcc/config/i386/driver-i386.c > > +++ b/gcc/c

Re: [PATCH] x86: Add -march=cascadelake

2018-12-16 Thread Wei Xiao
Thanks for the comments! Fixed as attached. Ok for trunk? Jakub Jelinek 于2018年12月14日周五 下午6:47写道: > > On Fri, Dec 14, 2018 at 06:33:37PM +0800, Wei Xiao wrote: > --- a/gcc/config/i386/driver-i386.c > +++ b/gcc/config/i386/driver-i386.c > @@ -832,8 +832,16 @@ const char *host_detect_local_cpu (int a

Re: [PATCH] x86: Add -march=cascadelake

2018-12-14 Thread Jakub Jelinek
On Fri, Dec 14, 2018 at 06:33:37PM +0800, Wei Xiao wrote: --- a/gcc/config/i386/driver-i386.c +++ b/gcc/config/i386/driver-i386.c @@ -832,8 +832,16 @@ const char *host_detect_local_cpu (int argc, const char **argv) cpu = "skylake"; break; case 0x55: - /* Skylake

Re: [PATCH] x86: Add -march=cascadelake

2018-12-14 Thread Wei Xiao
The part 2 is implemented by attached patch. Ok for trunk? Wei gcc/ * config/i386/driver-i386.c (host_detect_local_cpu): Detect cascadelake. * config/i386/i386.c (fold_builtin_cpu): Handle cascadelake. * doc/extend.texi: Add cascadelake. gcc/testsuite/ * g++.target/i3

Re: [PATCH] x86: Add -march=cascadelake

2018-12-12 Thread Uros Bizjak
On Wed, Dec 12, 2018 at 10:48 AM Wei Xiao wrote: > > Hi Uros and other reviewers, > > I'd like to split the work into 2 parts: > 1) Basic processor enabling. > 2) Processor type dynamic check. > > Let's use a separate patch to implement the part 2. > The part 1 is implemented by attached patch. >

Re: [PATCH] x86: Add -march=cascadelake

2018-12-12 Thread Wei Xiao
Hi Uros and other reviewers, I'd like to split the work into 2 parts: 1) Basic processor enabling. 2) Processor type dynamic check. Let's use a separate patch to implement the part 2. The part 1 is implemented by attached patch. Is it ok for trunk? Wei gcc/ * common/config/i386/i386-commo

Re: [PATCH] x86: Add -march=cascadelake

2018-11-29 Thread Wei Xiao
Hi Distinguish based on stepping number is not recommended for some reasons: 1) Intel doesn't officially disclose stepping information in SDM. 2) Stepping can be changing in the future. We still prefer the conventional distinguish approach based on feature bits. I have refined the patch as attach

Re: [PATCH] x86: Add -march=cascadelake

2018-11-27 Thread Wei Xiao
Thanks for the helpful information! But I'm still checking with hardware team about the family/model/stepping numbers for Cascadelake which are not officially disclosed by Intel (to my best knowledge). Wei Martin Liška 于2018年11月26日周一 下午10:00写道: > > On 11/26/18 12:18 PM, Jakub Jelinek wrote: > > O

Re: [PATCH] x86: Add -march=cascadelake

2018-11-26 Thread Martin Liška
On 11/26/18 12:18 PM, Jakub Jelinek wrote: > On Mon, Nov 26, 2018 at 12:03:53PM +0100, Martin Liška wrote: >>> For Cascade Lake the model number is the same as Skylake Server, >>> it can only be distinguished based on the stepping (5 vs 4) >> >> Very interesting, probably the first time a distingui

Re: [PATCH] x86: Add -march=cascadelake

2018-11-26 Thread Jakub Jelinek
On Mon, Nov 26, 2018 at 12:03:53PM +0100, Martin Liška wrote: > > For Cascade Lake the model number is the same as Skylake Server, > > it can only be distinguished based on the stepping (5 vs 4) > > Very interesting, probably the first time a distinguish is based on stepping > number? Wouldn't

Re: [PATCH] x86: Add -march=cascadelake

2018-11-26 Thread Martin Liška
On 11/24/18 12:34 AM, Andi Kleen wrote: > Uros Bizjak writes: > >> On Thu, Nov 22, 2018 at 12:48 PM Martin Liška wrote: >>> >>> On 11/22/18 8:07 AM, Wei Xiao wrote: Jakub, Thanks for the comments! I have addressed them as attached. >>> >>> Hi. >>> >>> Can you please add the n

Re: [PATCH] x86: Add -march=cascadelake

2018-11-23 Thread Andi Kleen
Uros Bizjak writes: > On Thu, Nov 22, 2018 at 12:48 PM Martin Liška wrote: >> >> On 11/22/18 8:07 AM, Wei Xiao wrote: >> > Jakub, >> > >> > Thanks for the comments! >> > I have addressed them as attached. >> >> Hi. >> >> Can you please add the new march into: >> >> - gcc/doc/extend.texi:20530 >>

Re: [PATCH] x86: Add -march=cascadelake

2018-11-22 Thread Uros Bizjak
On Thu, Nov 22, 2018 at 1:03 PM Uros Bizjak wrote: > > > Thanks for the comments! > > > I have addressed them as attached. > > > > Hi. > > > > Can you please add the new march into: > > > > - gcc/doc/extend.texi:20530 > > - gcc/testsuite/gcc.target/i386/builtin_target.c - test it here > > IIRC, f

Re: [PATCH] x86: Add -march=cascadelake

2018-11-22 Thread Uros Bizjak
On Thu, Nov 22, 2018 at 12:48 PM Martin Liška wrote: > > On 11/22/18 8:07 AM, Wei Xiao wrote: > > Jakub, > > > > Thanks for the comments! > > I have addressed them as attached. > > Hi. > > Can you please add the new march into: > > - gcc/doc/extend.texi:20530 > - gcc/testsuite/gcc.target/i386/buil

Re: [PATCH] x86: Add -march=cascadelake

2018-11-22 Thread Martin Liška
On 11/22/18 8:07 AM, Wei Xiao wrote: > Jakub, > > Thanks for the comments! > I have addressed them as attached. Hi. Can you please add the new march into: - gcc/doc/extend.texi:20530 - gcc/testsuite/gcc.target/i386/builtin_target.c - test it here Thanks, Martin

Re: [PATCH] x86: Add -march=cascadelake

2018-11-22 Thread Uros Bizjak
On Thu, Nov 22, 2018 at 8:08 AM Wei Xiao wrote: > > Jakub, > > Thanks for the comments! > I have addressed them as attached. > > Wei > > gcc/ > * common/config/i386/i386-common.c (processor_names): Add cascadelake. > (processor_alias_table): Add cascadelake. > * config.

Re: [PATCH] x86: Add -march=cascadelake

2018-11-21 Thread Wei Xiao
Jakub, Thanks for the comments! I have addressed them as attached. Wei gcc/ * common/config/i386/i386-common.c (processor_names): Add cascadelake. (processor_alias_table): Add cascadelake. * config.gcc: Add -march=cascadelake. * config/i386/driver-i386.c

Re: [PATCH] x86: Add -march=cascadelake

2018-11-21 Thread Jakub Jelinek
On Wed, Nov 21, 2018 at 06:23:41PM +0800, Wei Xiao wrote: > The attached patch added -march=cascadelake for x86. > Tested with bootstrap and regression tests on x86_64. No regressions. > Is it ok for trunk? Not a real review, just nits: index bff4dfb..f7c1c98 100644 --- a/gcc/ChangeLog +++ b/gcc/

[PATCH] x86: Add -march=cascadelake

2018-11-21 Thread Wei Xiao
Hi, The attached patch added -march=cascadelake for x86. Tested with bootstrap and regression tests on x86_64. No regressions. Is it ok for trunk? Wei gcc/ * common/config/i386/i386-common.c (processor_names): Add cascadelake. (processor_alias_table): Add cascadelake.