Re: [patch][x86] Split-up march icelake on march=icelake-server and march=icelake-client

2018-03-14 Thread Uros Bizjak
On Wed, Mar 14, 2018 at 8:44 AM, Koval, Julia <julia.ko...@intel.com> wrote:
> Small fix.
>
> gcc/
> * config.gcc (icelake-client, icelake-server): New.
> (icelake): Remove.
> * config/i386/i386.c (initial_ix86_tune_features): Extend to 64 bit.
> (initial_ix86_arch_features): Ditto.
> (PTA_SKYLAKE): Add SGX.
> (PTA_ICELAKE): Remove.
> (PTA_ICELAKE_CLIENT): New.
> (PTA_ICELAKE_SERVER): New.
> (ix86_option_override_internal): Split up icelake on icelake client 
> and
> icelake server.
> (get_builtin_code_for_version): Ditto.
> (fold_builtin_cpu): Ditto.
> * config/i386/driver-i386.c (config/i386/driver-i386.c): Ditto.
> * config/i386/i386-c.c (ix86_target_macros_internal): Ditto
> * config/i386/i386.h (processor_type) Ditto.
> * doc/invoke.texi: Ditto.
>
> gcc/testsuite/
> * g++.dg/ext/mv16.C: Split up icelake on icelake client and
> icelake-server.
> * gcc.target/i386/funcspec-56.inc: Ditto.
>
> libgcc/
> * config/i386/cpuinfo.h (processor_subtypes): Split up icelake on 
> icelake
> client and icelake-server.

OK.

Thanks,
Uros.

> Thanks,
> Julia
>
>> -Original Message-
>> From: Koval, Julia
>> Sent: Tuesday, March 13, 2018 8:42 AM
>> To: Joseph Myers <jos...@codesourcery.com>
>> Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>; Uros Bizjak
>> <ubiz...@gmail.com>
>> Subject: RE: [patch][x86] Split-up march icelake on march=icelake-server and
>> march=icelake-client
>>
>> Fixed invoke.texi. Here is the new version.
>>
>> gcc/
>>   * config.gcc (icelake-client, icelake-server): New.
>>   (icelake): Remove.
>>   * config/i386/i386.c (initial_ix86_tune_features): Extend to 64 bit.
>>   (initial_ix86_arch_features): Ditto.
>>   (ix86_option_override_internal): Split up icelake on icelake client and
>>   icelake server.
>>   (get_builtin_code_for_version): Ditto.
>>   (fold_builtin_cpu): Ditto.
>>   * config/i386/driver-i386.c (config/i386/driver-i386.c): Ditto.
>>   * config/i386/i386-c.c (ix86_target_macros_internal): Ditto
>>   * config/i386/i386.h (processor_type) Ditto.
>>   * doc/invoke.texi: Ditto.
>>
>> gcc/testsuite/
>>   * g++.dg/ext/mv16.C: Split up icelake on icelake client and
>>   icelake-server.
>>   * gcc.target/i386/funcspec-56.inc: Ditto.
>>
>> libgcc/
>>   * config/i386/cpuinfo.h (processor_subtypes): Split up icelake on 
>> icelake
>>   client and icelake-server.
>>
>> Thanks,
>> Julia
>>
>> > -Original Message-
>> > From: Joseph Myers [mailto:jos...@codesourcery.com]
>> > Sent: Monday, March 12, 2018 10:21 PM
>> > To: Koval, Julia <julia.ko...@intel.com>
>> > Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>; Uros Bizjak
>> > <ubiz...@gmail.com>
>> > Subject: Re: [patch][x86] Split-up march icelake on march=icelake-server 
>> > and
>> > march=icelake-client
>> >
>> > On Mon, 12 Mar 2018, Koval, Julia wrote:
>> >
>> > > Hi,
>> > > This patch introduces separate client and server arch options instead of
>> > > -march=icelake. Ok for trunk?
>> >
>> > I don't see any invoke.texi updates here to document what these two
>> > options mean (including, presumably, different lists of features for
>> > them).
>> >
>> > --
>> > Joseph S. Myers
>> > jos...@codesourcery.com


RE: [patch][x86] Split-up march icelake on march=icelake-server and march=icelake-client

2018-03-14 Thread Koval, Julia
Small fix.

gcc/
* config.gcc (icelake-client, icelake-server): New.
(icelake): Remove.
* config/i386/i386.c (initial_ix86_tune_features): Extend to 64 bit.
(initial_ix86_arch_features): Ditto.
(PTA_SKYLAKE): Add SGX.
(PTA_ICELAKE): Remove.
(PTA_ICELAKE_CLIENT): New.
(PTA_ICELAKE_SERVER): New.
(ix86_option_override_internal): Split up icelake on icelake client and
icelake server.
(get_builtin_code_for_version): Ditto.
(fold_builtin_cpu): Ditto.
* config/i386/driver-i386.c (config/i386/driver-i386.c): Ditto.
* config/i386/i386-c.c (ix86_target_macros_internal): Ditto
* config/i386/i386.h (processor_type) Ditto.
* doc/invoke.texi: Ditto.

gcc/testsuite/
* g++.dg/ext/mv16.C: Split up icelake on icelake client and
icelake-server.
* gcc.target/i386/funcspec-56.inc: Ditto.

libgcc/
* config/i386/cpuinfo.h (processor_subtypes): Split up icelake on 
icelake 
client and icelake-server.

Thanks,
Julia

> -Original Message-
> From: Koval, Julia
> Sent: Tuesday, March 13, 2018 8:42 AM
> To: Joseph Myers <jos...@codesourcery.com>
> Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>; Uros Bizjak
> <ubiz...@gmail.com>
> Subject: RE: [patch][x86] Split-up march icelake on march=icelake-server and
> march=icelake-client
> 
> Fixed invoke.texi. Here is the new version.
> 
> gcc/
>   * config.gcc (icelake-client, icelake-server): New.
>   (icelake): Remove.
>   * config/i386/i386.c (initial_ix86_tune_features): Extend to 64 bit.
>   (initial_ix86_arch_features): Ditto.
>   (ix86_option_override_internal): Split up icelake on icelake client and
>   icelake server.
>   (get_builtin_code_for_version): Ditto.
>   (fold_builtin_cpu): Ditto.
>   * config/i386/driver-i386.c (config/i386/driver-i386.c): Ditto.
>   * config/i386/i386-c.c (ix86_target_macros_internal): Ditto
>   * config/i386/i386.h (processor_type) Ditto.
>   * doc/invoke.texi: Ditto.
> 
> gcc/testsuite/
>   * g++.dg/ext/mv16.C: Split up icelake on icelake client and
>   icelake-server.
>   * gcc.target/i386/funcspec-56.inc: Ditto.
> 
> libgcc/
>   * config/i386/cpuinfo.h (processor_subtypes): Split up icelake on 
> icelake
>   client and icelake-server.
> 
> Thanks,
> Julia
> 
> > -Original Message-
> > From: Joseph Myers [mailto:jos...@codesourcery.com]
> > Sent: Monday, March 12, 2018 10:21 PM
> > To: Koval, Julia <julia.ko...@intel.com>
> > Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>; Uros Bizjak
> > <ubiz...@gmail.com>
> > Subject: Re: [patch][x86] Split-up march icelake on march=icelake-server and
> > march=icelake-client
> >
> > On Mon, 12 Mar 2018, Koval, Julia wrote:
> >
> > > Hi,
> > > This patch introduces separate client and server arch options instead of
> > > -march=icelake. Ok for trunk?
> >
> > I don't see any invoke.texi updates here to document what these two
> > options mean (including, presumably, different lists of features for
> > them).
> >
> > --
> > Joseph S. Myers
> > jos...@codesourcery.com


0001-icelake-client.patch
Description: 0001-icelake-client.patch


RE: [patch][x86] Split-up march icelake on march=icelake-server and march=icelake-client

2018-03-13 Thread Koval, Julia
Fixed invoke.texi. Here is the new version.

gcc/
* config.gcc (icelake-client, icelake-server): New.
(icelake): Remove.
* config/i386/i386.c (initial_ix86_tune_features): Extend to 64 bit.
(initial_ix86_arch_features): Ditto.
(ix86_option_override_internal): Split up icelake on icelake client and
icelake server.
(get_builtin_code_for_version): Ditto.
(fold_builtin_cpu): Ditto.
* config/i386/driver-i386.c (config/i386/driver-i386.c): Ditto.
* config/i386/i386-c.c (ix86_target_macros_internal): Ditto
* config/i386/i386.h (processor_type) Ditto.
* doc/invoke.texi: Ditto.

gcc/testsuite/
* g++.dg/ext/mv16.C: Split up icelake on icelake client and
icelake-server.
* gcc.target/i386/funcspec-56.inc: Ditto.

libgcc/
* config/i386/cpuinfo.h (processor_subtypes): Split up icelake on 
icelake 
client and icelake-server.

Thanks,
Julia

> -Original Message-
> From: Joseph Myers [mailto:jos...@codesourcery.com]
> Sent: Monday, March 12, 2018 10:21 PM
> To: Koval, Julia <julia.ko...@intel.com>
> Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>; Uros Bizjak
> <ubiz...@gmail.com>
> Subject: Re: [patch][x86] Split-up march icelake on march=icelake-server and
> march=icelake-client
> 
> On Mon, 12 Mar 2018, Koval, Julia wrote:
> 
> > Hi,
> > This patch introduces separate client and server arch options instead of
> > -march=icelake. Ok for trunk?
> 
> I don't see any invoke.texi updates here to document what these two
> options mean (including, presumably, different lists of features for
> them).
> 
> --
> Joseph S. Myers
> jos...@codesourcery.com


0001-icelake-client.patch
Description: 0001-icelake-client.patch


Re: [patch][x86] Split-up march icelake on march=icelake-server and march=icelake-client

2018-03-12 Thread Joseph Myers
On Mon, 12 Mar 2018, Koval, Julia wrote:

> Hi,
> This patch introduces separate client and server arch options instead of 
> -march=icelake. Ok for trunk?

I don't see any invoke.texi updates here to document what these two 
options mean (including, presumably, different lists of features for 
them).

-- 
Joseph S. Myers
jos...@codesourcery.com


[patch][x86] Split-up march icelake on march=icelake-server and march=icelake-client

2018-03-12 Thread Koval, Julia
Hi,
This patch introduces separate client and server arch options instead of 
-march=icelake. Ok for trunk?

Thanks,
Julia


gcc/
* config.gcc (icelake-client, icelake-server): New.
(icelake): Remove.
* config/i386/i386.c (initial_ix86_tune_features): Extend to 64 bit.
(initial_ix86_arch_features): Ditto.
(ix86_option_override_internal): Split up icelake on icelake client and
icelake server.
(get_builtin_code_for_version): Ditto.
(fold_builtin_cpu): Ditto.
* config/i386/driver-i386.c (config/i386/driver-i386.c): Ditto.
* config/i386/i386-c.c (ix86_target_macros_internal): Ditto
* config/i386/i386.h (processor_type) Ditto.

gcc/testsuite/
* g++.dg/ext/mv16.C: Split up icelake on icelake client and
icelake-server.
* gcc.target/i386/funcspec-56.inc: Ditto.

libgcc/
* config/i386/cpuinfo.h (processor_subtypes): Split up icelake on 
icelake 
client and icelake-server.


0001-icelake-client.patch
Description: 0001-icelake-client.patch