Re: [PATCH 01/10] C++ FE: expression ranges v4

2015-12-04 Thread David Malcolm
On Fri, 2015-12-04 at 12:10 -0500, Jason Merrill wrote: > On 12/03/2015 09:55 AM, David Malcolm wrote: > > @@ -362,10 +362,11 @@ convert_to_real_1 (tree type, tree expr, bool fold_p) > > case REAL_TYPE: > > /* Ignore the conversion if we don't need to store intermediate > > resul

Re: [PATCH] Fix -Werror= handling for Joined warnings, add a few missing Warning keywords (PRs c/48088, c/68657)

2015-12-04 Thread Manuel López-Ibáñez
On 4 December 2015 at 17:53, Jakub Jelinek wrote: > + > + if (e->unknown_error) > + error_at (loc, e->unknown_error, option->opt_text); > + else > + error_at (loc, "unrecognized argument in option %qs", > +

Re: [PATCH][1/2] Fix PR68553

2015-12-04 Thread Alan Lawrence
On 04/12/15 17:46, Ramana Radhakrishnan wrote: On 04/12/15 16:04, Richard Biener wrote: On December 4, 2015 4:32:33 PM GMT+01:00, Alan Lawrence wrote: On 27/11/15 08:30, Richard Biener wrote: This is part 1 of a fix for PR68533 which shows that some targets cannot can_vec_perm_p on an ide

Re: [ARM] Fix PR middle-end/65958

2015-12-04 Thread Eric Botcazou
> Minor nit. Since this is used in an unspec_volatile, the name should be > UNSPECV_ and defined in the unspecv enum. Ouch. It turns out that the ARM implementation has it too so I have installed the attached patchlet on top of the others after minimal retesting. PR middle-end/65958

Re: Add fuzzing coverage support

2015-12-04 Thread Dmitry Vyukov
On Fri, Dec 4, 2015 at 6:39 PM, Jakub Jelinek wrote: > On Fri, Dec 04, 2015 at 06:32:38PM +0100, Dmitry Vyukov wrote: >> +2015-12-04 Dmitry Vyukov >> + >> + * sancov.c: New file. >> + * Makefile.in (OBJS): Add sancov.o. >> + * invoke.texi (-fsanitize-coverage=trace-pc): Describe. >>

[PATCH] [graphite] fix PR68693: Check for loop structure when extending the SCoP

2015-12-04 Thread Aditya Kumar
The check for dominance while extending the scop assumed that multiple successors meant a loop which is not true in case of conditionals around the loop. Improved pretty printers for better debugging. gcc/ChangeLog: 2015-12-02 Aditya Kumar Sebastian Pop * graphite-scop-

[PATCH 1/2] [graphite] check that all the scev applied ops have are dominated by their defs

2015-12-04 Thread Aditya Kumar
2015-12-02 Aditya Kumar Sebastian Pop * gcc.dg/graphite/id-29.c: New test. gcc/ChangeLog: 2015-12-02 Aditya Kumar Sebastian Pop * graphite-isl-ast-to-gimple.c (translate_isl_ast_node_user): Improve debug. (get_rename_from_scev): Check

[PTX] var emission

2015-12-04 Thread Nathan Sidwell
I've committed this patch to unify the variable emission machinery. All four hooks were doing small variations of a common theme. This introduces nvptx_assemble_decl_begin to do the common work. While there, I expanded the init initialization code to work with multi-dimensional arrays. nath

Re: [PATCH] Fix declaration of pthread-structs in s-osinte-rtems.ads (ada/68169)

2015-12-04 Thread Jeff Law
On 12/02/2015 03:23 PM, Jan Sommer wrote: Am Wednesday 02 December 2015, 08:13:20 schrieb Joel Sherrill: On December 2, 2015 2:14:22 AM EST, Jeff Law wrote: On 12/01/2015 12:56 PM, Jan Sommer wrote: Am Monday 30 November 2015, 16:19:30 schrieb Jeff Law: On 11/30/2015 03:06 PM, Jan Sommer wr

Re: [PATCH] Fix declaration of pthread-structs in s-osinte-rtems.ads (ada/68169)

2015-12-04 Thread Joel Sherrill
On December 4, 2015 12:44:57 PM CST, Jeff Law wrote: >On 12/02/2015 03:23 PM, Jan Sommer wrote: >> Am Wednesday 02 December 2015, 08:13:20 schrieb Joel Sherrill: >>> >>> On December 2, 2015 2:14:22 AM EST, Jeff Law >>> wrote: On 12/01/2015 12:56 PM, Jan Sommer wrote: > Am Monday 30 Nov

Re: [PTX] var emission

2015-12-04 Thread Nathan Sidwell
On 12/04/15 13:44, Nathan Sidwell wrote: I've committed this patch to unify the variable emission machinery. All four hooks were doing small variations of a common theme. This introduces nvptx_assemble_decl_begin to do the common work. While there, I expanded the init initialization code to wo

Re: PR c/68657 - Add missing 'Warning' flags to c-family/c.opt + java/lang.opt

2015-12-04 Thread Jeff Law
On 12/04/2015 03:00 AM, Tobias Burnus wrote: And now with attached patch ... On Fri, Dec 04, 2015 at 10:59:29AM +0100, Tobias Burnus wrote: A few warning options lack the 'Warning' flag, which since r228094 (2015-09-24) has the effect that -W(no-)error= doesn't work for them. Additionally, --he

Re: [PATCH] Do not sanitize left shifts for -fwrapv (PR68418)

2015-12-04 Thread Jeff Law
On 12/04/2015 10:51 AM, Paolo Bonzini wrote: On 25/11/2015 14:55, Paolo Bonzini wrote: Left shifts into the sign bit is a kind of overflow, and the standard chooses to treat left shifts of negative values the same way. However, the -fwrapv option modifies the language to one where integers ar

patch to fix PR68349

2015-12-04 Thread Vladimir Makarov
The following patch fixes https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68349 The patch was tested and bootstrapped on x86/x86-64. Committed as rev. 231300. Index: ChangeLog === --- ChangeLog (revision 231299) +++ ChangeLog (wo

Re: [GOOGLE] Remove overly-aggressive LIPO assert

2015-12-04 Thread Teresa Johnson
Ping. Thanks, Teresa On Wed, Dec 2, 2015 at 12:46 PM, Teresa Johnson wrote: > Remove an assert that was overly-strict and already partially redundant > with an immediately prior assert. In this case we had a hidden visibility > function clone that was created after the LIPO link due to indirect c

Re: [PATCH] Fix -Werror= handling for Joined warnings, add a few missing Warning keywords (PRs c/48088, c/68657)

2015-12-04 Thread Jakub Jelinek
On Fri, Dec 04, 2015 at 06:19:19PM +, Manuel López-Ibáñez wrote: > On 4 December 2015 at 17:53, Jakub Jelinek wrote: > > + > > + if (e->unknown_error) > > + error_at (loc, e->unknown_error, option->opt_text); > > + else > > +

[AArch64] Add register constraints to add3_pluslong

2015-12-04 Thread James Greenhalgh
constraints. gcc/testsuite/ 2015-12-04 James Greenhalgh * gcc.c-torture/compile/20151204.c: New. diff --git a/gcc/config/aarch64/aarch64.md b/gcc/config/aarch64/aarch64.md index 765df6a..79d1414 100644 --- a/gcc/config/aarch64/aarch64.md +++ b/gcc/config/aarch64/aarch64.md @@ -1613,9

Re: [AArch64] Add register constraints to add3_pluslong

2015-12-04 Thread Marcus Shawcroft
tested on aarch64-none-elf > with no issues. > > OK? > > Thanks, > James > > --- > gcc/ > > 2015-12-04 James Greenhalgh > > * config/aarch64/aarch64.md (add3_pluslong): Add register > constraints. > > gcc/testsuite/ > > 2015-12-04 James Greenhalgh > > * gcc.c-torture/compile/20151204.c: New. > +main(int argc, char** argv) Space before (. OK /M

[PATCH] Adjust vect-widen-mult-const-[su]16.c for r226675

2015-12-04 Thread Bill Schmidt
Since r226675, we have been seeing these failures: FAIL: gcc.dg/vect/vect-widen-mult-const-s16.c -flto -ffat-lto-objects scan-tree-dump-times vect "pattern recognized" 2 FAIL: gcc.dg/vect/vect-widen-mult-const-s16.c scan-tree-dump-times vect "pattern recognized" 2 FAIL: gcc.dg/vect/vect-widen-mult

RE: [PATCH] enable loop fusion on isl-15

2015-12-04 Thread Sebastian Paul Pop
I would highly recommend updating the required version of ISL to isl-0.15: that would simplify the existing code, removing a lot of code under "#ifdef old ISL", and allow us to fully transition to schedule_trees instead of dealing with the old antiquated union_maps in the scheudler. The result is

[testsuite] VLA test uses alloca

2015-12-04 Thread Nathan Sidwell
gcc.dg/vla-24.c uses alloca. Marked accordingly and committed as obvious. nathan 2015-12-04 Nathan Sidwell * gcc.dg/vla-24.c: Requires alloca. Index: testsuite/gcc.dg/vla-24.c === --- testsuite/gcc.dg/vla-24.c (revision 23130

Re: [PATCH] Fix missing range information for "%q+D" format code

2015-12-04 Thread David Malcolm
On Fri, 2015-12-04 at 12:09 +0100, Bernd Schmidt wrote: > On 12/03/2015 09:33 PM, David Malcolm wrote: > > The attached patch updates the handling of %q+D, simplifying > > the implementation, and ensuring that it retains the range > > information of the decl, giving: > > > > diagnostic-ranges-1.c:6

Re: [PATCH] Do not sanitize left shifts for -fwrapv (PR68418)

2015-12-04 Thread Paolo Bonzini
> >> gcc: > >>PR sanitizer/68418 > >>* c-family/c-ubsan.c (ubsan_instrument_shift): Disable > >>sanitization of left shifts for wrapping signed types as well. > >> > >> gcc/testsuite: > >>PR sanitizer/68418 > >>* gcc.dg/ubsan/c99-wrapv-shift-1.c, > >>gcc.dg/ubsan/c99-wrapv-

Re: [PATCH] fix PR65726

2015-12-04 Thread Andreas Tobler
On 01.12.15 22:55, Jeff Law wrote: On 12/01/2015 02:50 PM, Andreas Tobler wrote: On 30.11.15 23:30, Jeff Law wrote: On 11/26/2015 11:49 AM, Andreas Tobler wrote: Hi all, the attached patch fixes the build issue from this ticket if bootstrap is disabled. Tested on x86_64-*-linux* and on x86_6

Re: [patch] add ELFv2 check to FreeBSD PowerPC64

2015-12-04 Thread Andreas Tobler
On 01.12.15 22:47, Andreas Tobler wrote: On 29.11.15 22:15, Andreas Tobler wrote: Hi all, I'd like to commit this patch to trunk. It is FreeBSD only. If nobody objects I'll commit it within two days. Thanks, Andreas 2015-11-29 Andreas Tobler * config/rs6000/freebsd64.h (ELFv2_A

Re: [PATCH] Fix missing range information for "%q+D" format code

2015-12-04 Thread David Malcolm
On Fri, 2015-12-04 at 15:45 -0500, David Malcolm wrote: > On Fri, 2015-12-04 at 12:09 +0100, Bernd Schmidt wrote: > > On 12/03/2015 09:33 PM, David Malcolm wrote: > > > The attached patch updates the handling of %q+D, simplifying > > > the implementation, and ensuring that it retains the range > >

Re: [Patch, Contrib] Download ISL 0.15 by download_prerequisites

2015-12-04 Thread Sebastian Pop
LGTM. Thanks! Tobias Burnus wrote: > If there are no objections, I intent to commit the following patch > tomorrow as obvious. > > Tobias > > On Tue, Oct 27, 2015 at 11:27:41AM +0100, Tobias Burnus wrote: > > recently, support for ISL 0.15 was added to GCC and also > > ftp://gcc.gnu.org/pub/gcc

Re: [PATCH] Convert manual unsigned +/- overflow checking into {ADD,SUB}_OVERFLOW (PR target/67089)

2015-12-04 Thread Marc Glisse
On Wed, 25 Nov 2015, Marc Glisse wrote: So, I'd really prefer doing x-y>x to y>x only for single use. Ok Let me post this patch (needs testing on x86, I only tested on ppc which does not implement the new optabs) so I can more easily find it again at next stage 1. It produces imag()==0 o

[gomp4] more runtime test coverage

2015-12-04 Thread Cesar Philippidis
This patch adds some more runtime test coverage for all of the clauses supported by the kernels, parallel and loop directives. Specifically, acc loops inside parallel regions were missing coverage for the tile clause. Kernels loop were missing coverage for gang, worker, vector, seq, auto, collapse,

Re: [PATCH] Do not sanitize left shifts for -fwrapv (PR68418)

2015-12-04 Thread Jeff Law
On 12/04/2015 01:48 PM, Paolo Bonzini wrote: gcc: PR sanitizer/68418 * c-family/c-ubsan.c (ubsan_instrument_shift): Disable sanitization of left shifts for wrapping signed types as well. gcc/testsuite: PR sanitizer/68418 * gcc.dg/ubsan/c99-wrapv-shift-1.

Re: [PATCH] Do not sanitize left shifts for -fwrapv (PR68418)

2015-12-04 Thread Jeff Law
On 12/04/2015 01:48 PM, Paolo Bonzini wrote: gcc: PR sanitizer/68418 * c-family/c-ubsan.c (ubsan_instrument_shift): Disable sanitization of left shifts for wrapping signed types as well. gcc/testsuite: PR sanitizer/68418 * gcc.dg/ubsan/c99-wrapv-shift-1.

Re: -fstrict-aliasing fixes 6/6: permit inlining of comdats

2015-12-04 Thread H.J. Lu
On Fri, Dec 4, 2015 at 10:04 AM, Jan Hubicka wrote: > Hi, > this is the patch implementing renaming and fixing logic in ipa-icf/lto-symtab > WRT compuattion of this flag. I don't seem to be able to construct testcase > for > this: the merged flag is currently used in inliner only to decide wheth

Re: patch to fix PR68349

2015-12-04 Thread H.J. Lu
On Fri, Dec 4, 2015 at 11:26 AM, Vladimir Makarov wrote: > The following patch fixes > > https://gcc.gnu.org/bugzilla/show_bug.cgi?id=68349 > > The patch was tested and bootstrapped on x86/x86-64. > > Committed as rev. 231300. unsigned long strlen(); ^^^ I got .

Re: Add fuzzing coverage support

2015-12-04 Thread Nathan Sidwell
On 12/04/15 13:28, Dmitry Vyukov wrote: On Fri, Dec 4, 2015 at 6:39 PM, Jakub Jelinek wrote: On Fri, Dec 04, 2015 at 06:32:38PM +0100, Dmitry Vyukov wrote: +2015-12-04 Dmitry Vyukov + + * sancov.c: New file. + * Makefile.in (OBJS): Add sancov.o. + * invoke.texi (-fsanitize-cover

Re: Gimple loop splitting v2

2015-12-04 Thread Jeff Law
On 12/02/2015 06:23 AM, Michael Matz wrote: Hi, On Tue, 1 Dec 2015, Jeff Law wrote: So, okay for trunk? -ENOPATCH Sigh :) Here it is. Ciao, Michael. * common.opt (-fsplit-loops): New flag. * passes.def (pass_loop_split): Add. * opts.c (default_options_table): Add O

<    1   2