[PATCH, i386]: Some further improvements to patterns with embedded zero-extract RTXes

2016-12-27 Thread Uros Bizjak
Hello! While we still don't allow memory operands for x86_64, we can improve generated code by allowing memory operands until reload. This way, combine is free to create complex patterns, illustrated by following test: --cut here-- struct S1 { unsigned char pad1; unsigned char val; unsigned

Re: [PATCH] PR 78534 Change character length from int to size_t

2016-12-27 Thread Andre Vehreschild
Hi Janne, sorry for being late in voicing my opinion, but I personally would prefer to have this patch in a separately. That way bisecting for performance regressions points only to the offending code and not to the change of the character length (the latter might add a tiny bit of cost, too). Re

Re: [PATCH] PR 78534 Change character length from int to size_t

2016-12-27 Thread Janne Blomqvist
On Mon, Dec 26, 2016 at 12:32 PM, FX wrote: > Hi Janne, > > Thanks for the patch, it is hard and tedious work. Here is the formal review. > I don’t want to be a pain, but I have several questions about the patch, and > given its size and the importance I think we should be double-sure :) Thanks

Re: [PATCH] PR 78534 Change character length from int to size_t

2016-12-27 Thread Janne Blomqvist
On Tue, Dec 27, 2016 at 12:47 PM, Andre Vehreschild wrote: > Hi Janne, > > sorry for being late in voicing my opinion, but I personally would prefer to > have this patch in a separately. That way bisecting for performance > regressions points only to the offending code and not to the change of the

New Spanish PO file for 'gcc' (version 6.2.0)

2016-12-27 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Spanish team of translators. The file is available at: http://translationproject.org/latest/gcc/es.po (This file, 'gcc-6.2.0.es.po', has just

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2016-12-27 Thread Uros Bizjak
Hello! > this patch enables AVX512 VPOPCNTD/VPOPCNTQ instructions recently > added in Instruction Set Extensions > (https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf). @@ -265,6 +268,9 @@ (define_mode_iterator VF_512

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2016-12-27 Thread Andrew Senkevich
2016-12-27 16:35 GMT+03:00 Uros Bizjak : > Hello! > >> this patch enables AVX512 VPOPCNTD/VPOPCNTQ instructions recently >> added in Instruction Set Extensions >> (https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf). > > @@

Re: Pointer Bounds Checker and trailing arrays (PR68270)

2016-12-27 Thread Alexander Ivchenko
Committed as r243936. Thank you, Alexander 2016-12-22 2:47 GMT+03:00 Ilya Enkovich : > 2016-12-21 22:18 GMT+03:00 Alexander Ivchenko : >> Right.. here is this updated chunk (otherwise no difference in the patch) >> >> diff --git a/gcc/tree-chkp.c b/gcc/tree-chkp.c >> index 2769682..6c7862c 100644

[X86 PATCH] Fix up i386 option translations (PR translation/78922)

2016-12-27 Thread Jakub Jelinek
Hi! r201645 apparently added a useless file that nothing sources in in a format incompatible with all other *.opt files (comments in *.opt files start with ;, and no preprocessing is performed), the only thing that it affects is that all *.opt files from the tree are parsed to extract translation

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2016-12-27 Thread Uros Bizjak
On Tue, Dec 27, 2016 at 2:40 PM, Andrew Senkevich wrote: > 2016-12-27 16:35 GMT+03:00 Uros Bizjak : >> Hello! >> >>> this patch enables AVX512 VPOPCNTD/VPOPCNTQ instructions recently >>> added in Instruction Set Extensions >>> (https://software.intel.com/sites/default/files/managed/c5/15/architect

Re: [PATCH, Fortran] Allow warnings given through gfc_error to associate with warning flags

2016-12-27 Thread Jakub Jelinek
On Tue, Nov 01, 2016 at 11:10:39AM -0400, Fritz Reese wrote: > Currently warnings given by the GNU Fortran front-end typically > indicate which flag controls the warning, if any, as given by the > first argument to gfc_warning. However, there is no support for > controlling warnings which are emitt

Re: Use aligned SSE movs for re-aligned MS ABI pro/epilogues

2016-12-27 Thread Uros Bizjak
Hello! > According to the Microsoft 64-bit ABI specification, registers RDI, RSI and > XMM6-15 are > non-volatile and the stack alignment is 16 bytes. In practice, the Windows > implementation > appears to not be so picky about the 16-byte alignment requirement, probably > because it never > to

Re: [PATCH][x86_64] Enable AVX512 VPOPCNTD/VPOPCNTQ instructions

2016-12-27 Thread Uros Bizjak
On Tue, Dec 27, 2016 at 2:47 PM, Uros Bizjak wrote: > On Tue, Dec 27, 2016 at 2:40 PM, Andrew Senkevich > wrote: >> 2016-12-27 16:35 GMT+03:00 Uros Bizjak : >>> Hello! >>> this patch enables AVX512 VPOPCNTD/VPOPCNTQ instructions recently added in Instruction Set Extensions (https:/

[PING**3] [PATCH, ARM] Further improve stack usage on sha512 (PR 77308)

2016-12-27 Thread Bernd Edlinger
Ping... On 12/19/16 06:49, Bernd Edlinger wrote: > Ping... > > On 12/12/16 06:59, Bernd Edlinger wrote: >> Ping... >> >> On 12/05/16 14:41, Bernd Edlinger wrote: >>> Hi, >>> >>> this was the latest version of my patch: >>> https://gcc.gnu.org/ml/gcc-patches/2016-11/msg02796.html >>> >>> >>> Thanks

[PATCH, i386]: Allow constant memory for x86_64 operations involving high registers

2016-12-27 Thread Uros Bizjak
Hello! While we can't allow only non-REX memory operands in patterns involving high registers, we can at least allow constant memory operands here. 2016-12-27 Uros Bizjak PR target/78904 * config/i386/constraints.md (Bc): New special memory constraint. * config/i386/i386.md (*cmpq

Re: [X86 PATCH] Fix up i386 option translations (PR translation/78922)

2016-12-27 Thread Uros Bizjak
On Tue, Dec 27, 2016 at 2:46 PM, Jakub Jelinek wrote: > Hi! > > r201645 apparently added a useless file that nothing sources in > in a format incompatible with all other *.opt files (comments in *.opt > files start with ;, and no preprocessing is performed), the only thing that > it affects is th

Re: [PATCH, Fortran] Allow warnings given through gfc_error to associate with warning flags

2016-12-27 Thread FX
> 2016-12-27 Jakub Jelinek > > * gfortran.h (gfc_error): Rename overload with OPT argument to... > (gfc_error_opt): ... this. > * error.c (gfc_error): Rename overloads with OPT argument to... > (gfc_error_opt): ... this. Adjust callers. > (gfc_notify_std, gfc_erro

Re: PR78631 fix

2016-12-27 Thread Alexander Ivchenko
Committed as r243942 with the ChangeLog entries thanks, Alexander 2016-12-23 21:48 GMT+03:00 Ilya Enkovich : > Hi, > > ChangeLog? Otherwise OK. > > Ilya > > 2016-12-23 14:02 GMT+03:00 Alexander Ivchenko : >> Hi Ilya, >> >> Would that patch be OK to submit? (it is HJ's one with added testcase) >>

[PATCH] Add RejectNegative for a c option.

2016-12-27 Thread Martin Liška
Without RejectNegative one can cause an ICE in the compiler. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin >From e005f69008daf6ad7705a2f9c3d0242e426c168f Mon Sep 17 00:00:00 2001 From: marxin Date: Tue, 27 Dec 2016 15:35:03 +0100 Subject:

Re: Use aligned SSE movs for re-aligned MS ABI pro/epilogues

2016-12-27 Thread Daniel Santos
On 12/27/2016 07:56 AM, Uros Bizjak wrote: Hello! According to the Microsoft 64-bit ABI specification, registers RDI, RSI and XMM6-15 are non-volatile and the stack alignment is 16 bytes. In practice, the Windows implementation appears to not be so picky about the 16-byte alignment requiremen

[PATCH, i386]: Remove VI_512 mode iterator ...

2016-12-27 Thread Uros Bizjak
... and use equivalent VI_AVX512BW mode iterator instead. No functional changes. 2016-12-27 Uros Bizjak * config/i386/i386.md (VI_512): Remove. (vcond): Use VI_AVX512BW mode iterator instead of VI_512. (vcondu): Ditto. Bootstrapped and regression tested on x86_64-linux-gnu {,

Re: [PATCH] PR 78534 Change character length from int to size_t

2016-12-27 Thread FX
> The _size member is the size of the object; for polymorphic objects > the size isn't known at compile time, so it is to be stored in the > vtable (in principle, since the vtable tells the exact class of a > polymorphic object, it should be possible to figure out the size > without an explicit _si

Re: [PATCH] Add RejectNegative for a c option.

2016-12-27 Thread Sandra Loosemore
On 12/27/2016 09:26 AM, Martin Liška wrote: Without RejectNegative one can cause an ICE in the compiler. Patch can bootstrap on ppc64le-redhat-linux and survives regression tests. Ready to be installed? Martin Any chance you can also fix the manual to fully document the -fstrong-eval-order= fo

Re: [PATCH] Add support for Fuchsia (OS)

2016-12-27 Thread Josh Conner via gcc-patches
Ping^2? On 12/19/16 10:05 AM, Josh Conner wrote: Ping? On 12/12/16 1:31 PM, Josh Conner wrote: On 12/10/16 3:26 AM, Richard Earnshaw wrote: On 08/12/16 22:55, Josh Conner wrote: +arm*-*-fuchsia*) + tm_file="${tm_file} fuchsia.h arm/fuchsia-elf.h glibc-stdint.h" + tmake_file="

[doc, committed] preprocessor documentation cleanup, part 3/n

2016-12-27 Thread Sandra Loosemore
This patch takes care of a leftover from the last installment of this series -- I'd missed that there was a duplicate entry for -x in cppopts.texi. I've also moved -fno-show-column out of the preprocessor options and into the "Diagnostic Message Formatting Options" section, where people will b

[PATCH, i386]: Require QImode intermediate for QImode mask register spill only for !TARGET_AVX512DQ.

2016-12-27 Thread Uros Bizjak
The patch also cleans relevant part of ix86_secondary_reload a bit. true_regno does everything needed to determine if the operand is in memory. 2016-12-27 Uros Bizjak * config/i386/i386.c (ix86_secondary_reload): Require QImode intermediate for QImode mask register spill only for !TARG

Re: [PATCH] Fix late dwarf generated early from optimized out globals

2016-12-27 Thread Andreas Tobler
On 16.09.16 13:30, Richard Biener wrote: On Thu, 15 Sep 2016, Richard Biener wrote: This addresses sth I needed to address with the early LTO debug patches (you might now figure I'm piecemail merging stuff from that patch). When the cgraph code optimizes out a global we call the late_global_d

[PATCH] gcc-dg.exp: support more than one digit in relative line number notation

2016-12-27 Thread David Malcolm
In r240391 Jakub added relative line number notation (.+4 or .-1 etc) for DejaGnu messages. This patch tweaks the regex to allow relative line numbers containing more than one digit (I ran into this with a test for a diagnostic that issues multiple notes, for which I wanted to verify the diagnosti

Re: [PATCH] gcc-dg.exp: support more than one digit in relative line number notation

2016-12-27 Thread Mike Stump
On Dec 27, 2016, at 2:51 PM, David Malcolm wrote: > > In r240391 Jakub added relative line number notation (.+4 or .-1 etc) > for DejaGnu messages. > > This patch tweaks the regex to allow relative line numbers containing > more than one digit (I ran into this with a test for a diagnostic > that

Re: [PATCH], Fix PowerPC ISA 3.0 word extract/insert thinkos

2016-12-27 Thread Michael Meissner
On Fri, Dec 23, 2016 at 04:34:10PM -0600, Segher Boessenkool wrote: > On Fri, Dec 23, 2016 at 04:47:22PM -0500, Michael Meissner wrote: > > I had two thinkos in my previous patches for ISA 3.0 (power9) support that > > both > > relate to word extraction and insertion. > > > > The first thinko was

Re: [wwwdocs] Remove Java

2016-12-27 Thread Gerald Pfeifer
On Mon, 5 Dec 2016, Gerald Pfeifer wrote: > Applied now, and more changes to follow later. Applied. Gerald Remove further GCJ related pages, including how to use GCJ, how to contribute, how to debug it, references to docs and projects done with it, and open projects. Index: java/compile.html ==

[wwwdocs] Remove GCJ / java/ reference from frontends.html

2016-12-27 Thread Gerald Pfeifer
Applied. Gerald Index: frontends.html === RCS file: /cvs/gcc/wwwdocs/htdocs/frontends.html,v retrieving revision 1.39 diff -u -r1.39 frontends.html --- frontends.html 3 Dec 2013 01:04:41 - 1.39 +++ frontends.html

[wwwdocs] Remove GCJ/libgcj link from news/javaannounce.html

2016-12-27 Thread Gerald Pfeifer
Applied. Gerald Index: news/javaannounce.html === RCS file: /cvs/gcc/wwwdocs/htdocs/news/javaannounce.html,v retrieving revision 1.6 diff -u -r1.6 javaannounce.html --- news/javaannounce.html 3 Jul 2014 19:02:10 - 1.6

[wwwdocs] en.wikibooks.org now uses https

2016-12-27 Thread Gerald Pfeifer
Applied. Gerald Index: readings.html === RCS file: /cvs/gcc/wwwdocs/htdocs/readings.html,v retrieving revision 1.253 diff -u -r1.253 readings.html --- readings.html 23 Aug 2016 07:21:36 - 1.253 +++ readings.html

[doc, committed] split up cppopts.texi

2016-12-27 Thread Sandra Loosemore
This patch splits up cppopts.texi into 3 pieces. Moving the include path options and warning options into separate files allows those parts to be included in more appropriate places in the GCC manual. Previously, for instance, some of the include path options were listed as preprocessor option