Re: [PATCH] Remove x86 pcommit instruction

2017-02-27 Thread Gerald Pfeifer
Hi Andrew, On Mon, 27 Feb 2017, Andrew Senkevich wrote: >> +Support for the > + >> href="https://software.intel.com/en-us/blogs/2016/09/12/deprecate-pcommit-instruction";>deprecated >> +pcommit instruction has been removed. > Could you please add the same statement for gcc-7 changes?

Re: [PATCH] Remove x86 pcommit instruction

2017-02-27 Thread Andrew Senkevich
2017-02-25 18:10 GMT+01:00 Gerald Pfeifer : > On Wed, 7 Dec 2016, Andrew Senkevich wrote: But how to update changes for upcoming GCC 6.3 (and future GCC 5.5)? >>> There are corresponding documents at [1] and [2], please add a "GCC >>> 6.3" or "GCC 5.5" entry with "Target Specific Changes" at

Re: [PATCH] Remove x86 pcommit instruction

2017-02-25 Thread Gerald Pfeifer
On Sat, 25 Feb 2017, Gerald Pfeifer wrote: > And here is what I applied for GCC 5. Except that I fixed some things, but failed to remove the duplicate id. Applied as a follow-up. Gerald Index: gcc-5/changes.html === RCS file: /cvs/

Re: [PATCH] Remove x86 pcommit instruction

2017-02-25 Thread Gerald Pfeifer
On Sat, 25 Feb 2017, Gerald Pfeifer wrote: > Below is what I just applied. And here is what I applied for GCC 5. Gerald Index: changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-5/changes.html,v retrieving revision 1.143 diff -u

Re: [PATCH] Remove x86 pcommit instruction

2017-02-25 Thread Gerald Pfeifer
On Wed, 7 Dec 2016, Andrew Senkevich wrote: >>> But how to update changes for upcoming GCC 6.3 (and future GCC 5.5)? >> There are corresponding documents at [1] and [2], please add a "GCC >> 6.3" or "GCC 5.5" entry with "Target Specific Changes" at the end of >> the documents. >> >> [1] https://gc

Re: [PATCH] Remove x86 pcommit instruction

2016-12-07 Thread Andrew Senkevich
2016-12-07 15:34 GMT+03:00 Uros Bizjak : > On Wed, Dec 7, 2016 at 1:25 PM, Andrew Senkevich > wrote: > >> But how to update changes for upcoming GCC 6.3 (and future GCC 5.5)? > > There are corresponding documents at [1] and [2], please add a "GCC > 6.3" or "GCC 5.5" entry with "Target Specific Ch

Re: [PATCH] Remove x86 pcommit instruction

2016-12-07 Thread Uros Bizjak
On Wed, Dec 7, 2016 at 1:25 PM, Andrew Senkevich wrote: > But how to update changes for upcoming GCC 6.3 (and future GCC 5.5)? There are corresponding documents at [1] and [2], please add a "GCC 6.3" or "GCC 5.5" entry with "Target Specific Changes" at the end of the documents. [1] https://gcc

Re: [PATCH] Remove x86 pcommit instruction

2016-12-07 Thread Andrew Senkevich
2016-10-14 12:23 GMT+03:00 Richard Biener : > On Thu, Oct 13, 2016 at 11:39 PM, H.J. Lu wrote: >> On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich >> wrote: >>> 2016-10-11 20:09 GMT+03:00 H.J. Lu : On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich wrote: > 2016-10-06 1:07 GMT+03:

Re: [PATCH] Remove x86 pcommit instruction

2016-10-14 Thread Andrew Senkevich
> That said - why do we need to remove pcommit support on the branches at all? I think it is better to have old branches updated because pcommit will not be implemented in hardware. -- WBR, Andrew

Re: [PATCH] Remove x86 pcommit instruction

2016-10-14 Thread Richard Biener
On Fri, Oct 14, 2016 at 11:23 AM, Richard Biener wrote: > On Thu, Oct 13, 2016 at 11:39 PM, H.J. Lu wrote: >> On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich >> wrote: >>> 2016-10-11 20:09 GMT+03:00 H.J. Lu : On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich wrote: > 2016-10-06

Re: [PATCH] Remove x86 pcommit instruction

2016-10-14 Thread Richard Biener
On Thu, Oct 13, 2016 at 11:39 PM, H.J. Lu wrote: > On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich > wrote: >> 2016-10-11 20:09 GMT+03:00 H.J. Lu : >>> On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich >>> wrote: 2016-10-06 1:07 GMT+03:00 H.J. Lu : > On Wed, Oct 5, 2016 at 1:42 PM, A

Re: [PATCH] Remove x86 pcommit instruction

2016-10-13 Thread H.J. Lu
On Thu, Oct 13, 2016 at 5:09 AM, Andrew Senkevich wrote: > 2016-10-11 20:09 GMT+03:00 H.J. Lu : >> On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich >> wrote: >>> 2016-10-06 1:07 GMT+03:00 H.J. Lu : On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senkevich wrote: > 2016-10-05 18:06 GMT+03:

Re: [PATCH] Remove x86 pcommit instruction

2016-10-13 Thread Andrew Senkevich
2016-10-11 20:09 GMT+03:00 H.J. Lu : > On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich > wrote: >> 2016-10-06 1:07 GMT+03:00 H.J. Lu : >>> On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senkevich >>> wrote: 2016-10-05 18:06 GMT+03:00 Uros Bizjak : > On Wed, Oct 5, 2016 at 3:47 PM, Andrew Senk

Re: [PATCH] Remove x86 pcommit instruction

2016-10-11 Thread H.J. Lu
On Tue, Oct 11, 2016 at 10:04 AM, Andrew Senkevich wrote: > 2016-10-06 1:07 GMT+03:00 H.J. Lu : >> On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senkevich >> wrote: >>> 2016-10-05 18:06 GMT+03:00 Uros Bizjak : On Wed, Oct 5, 2016 at 3:47 PM, Andrew Senkevich wrote: >> -mpcommit >> -Ta

Re: [PATCH] Remove x86 pcommit instruction

2016-10-11 Thread Andrew Senkevich
2016-10-06 1:07 GMT+03:00 H.J. Lu : > On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senkevich > wrote: >> 2016-10-05 18:06 GMT+03:00 Uros Bizjak : >>> On Wed, Oct 5, 2016 at 3:47 PM, Andrew Senkevich >>> wrote: > -mpcommit > -Target Report Mask(ISA_PCOMMIT) Var(ix86_isa_flags) Save > -Suppor

Re: [PATCH] Remove x86 pcommit instruction

2016-10-05 Thread H.J. Lu
On Wed, Oct 5, 2016 at 1:42 PM, Andrew Senkevich wrote: > 2016-10-05 18:06 GMT+03:00 Uros Bizjak : >> On Wed, Oct 5, 2016 at 3:47 PM, Andrew Senkevich >> wrote: -mpcommit -Target Report Mask(ISA_PCOMMIT) Var(ix86_isa_flags) Save -Support PCOMMIT instruction. - You sh

Re: [PATCH] Remove x86 pcommit instruction

2016-10-05 Thread Andrew Senkevich
2016-10-05 18:06 GMT+03:00 Uros Bizjak : > On Wed, Oct 5, 2016 at 3:47 PM, Andrew Senkevich > wrote: >>> -mpcommit >>> -Target Report Mask(ISA_PCOMMIT) Var(ix86_isa_flags) Save >>> -Support PCOMMIT instruction. >>> - >>> >>> You should not simply delete a option that was in the released >>> compil

Re: [PATCH] Remove x86 pcommit instruction

2016-10-05 Thread Uros Bizjak
On Wed, Oct 5, 2016 at 9:03 PM, Andrew Senkevich wrote: >> (Please also provide an entry for Release Changes, since this is >> user-facing change. Also for release branches.) > > What is BKM for doing that for 7.1 release? File a PR and mention it > in ChangeLog? No, just mention the change at [1

Re: [PATCH] Remove x86 pcommit instruction

2016-10-05 Thread Andrew Senkevich
> (Please also provide an entry for Release Changes, since this is > user-facing change. Also for release branches.) What is BKM for doing that for 7.1 release? File a PR and mention it in ChangeLog? -- WBR, Andrew

Re: [PATCH] Remove x86 pcommit instruction

2016-10-05 Thread Andrew Senkevich
> -mpcommit > -Target Report Mask(ISA_PCOMMIT) Var(ix86_isa_flags) Save > -Support PCOMMIT instruction. > - > > You should not simply delete a option that was in the released > compiler, but a warning should be emitted instead. Please see how > msse5 is handled in i386.opt. Thank you, it is fixed

Re: [PATCH] Remove x86 pcommit instruction

2016-10-04 Thread Uros Bizjak
Hello! > 2016-10-03 Andrew Senkevich > >gcc/ > >* common/config/i386/i386-common.c (OPTION_MASK_ISA_PCOMMIT_UNSET, >OPTION_MASK_ISA_PCOMMIT_SET): Deleted definitions. >(ix86_handle_option): Deleted handle of OPT_mpcommit. >* config.gcc: Deleted pcommitintrin.