Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
On 08/06/15 17:31, Jakub Jelinek wrote: On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: Please work out with Vlad whether

Re: [C++/58583] ICE instantiating NSDMIs

2015-06-08 Thread Jason Merrill
How about using a DECL_LANG_FLAG instead of creating a garbage DEFAULT_ARG? Jason

Re: [PATCH] Refactor -Wmisleading-indentation API and extend coverage

2015-06-08 Thread Patrick Palka
On Mon, Jun 8, 2015 at 2:45 PM, Richard Sandiford rdsandif...@googlemail.com wrote: Patrick Palka patr...@parcs.ath.cx writes: At the same time this patch extends the coverage of the -Wmisleading-indentation implementation to catch misleading indentation involving the semicolon. (These

Re: [patch] Adjust gcc-plugin.h

2015-06-08 Thread Andrew MacLeod
On 06/08/2015 09:32 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 2:07 PM, Andrew MacLeod amacl...@redhat.com wrote: During the original flattening process I decided to use gcc-plugin.h as the kitchen sink for all includes that plugins might need. I think this has worked well for plugins,

Re: [PATCH] Refactor -Wmisleading-indentation API and extend coverage

2015-06-08 Thread Richard Sandiford
Patrick Palka patr...@parcs.ath.cx writes: On Mon, Jun 8, 2015 at 2:11 PM, David Malcolm dmalc...@redhat.com wrote: void warn_for_misleading_indentation (const common_token_info guard_tinfo, const common_token_info body_tinfo,

Re: RFA: RL78: With -mes0 put read only data in the .frodata section

2015-06-08 Thread DJ Delorie
Ok. Thanks!

Re: [PATCH] Refactor -Wmisleading-indentation API and extend coverage

2015-06-08 Thread Patrick Palka
On Mon, Jun 8, 2015 at 2:11 PM, David Malcolm dmalc...@redhat.com wrote: On Sun, 2015-06-07 at 16:06 -0400, Patrick Palka wrote: This patch refactors the entry point of -Wmisleading-indentation Thanks for working on this. I was hoping to submit a patch to propose putting

Re: [C++/58583] ICE instantiating NSDMIs

2015-06-08 Thread Nathan Sidwell
On 06/08/15 13:47, Jason Merrill wrote: How about using a DECL_LANG_FLAG instead of creating a garbage DEFAULT_ARG? good idea, I'll go look for an available one. nathan

Re: [PATCH] Refactor -Wmisleading-indentation API and extend coverage

2015-06-08 Thread David Malcolm
On Sun, 2015-06-07 at 16:06 -0400, Patrick Palka wrote: This patch refactors the entry point of -Wmisleading-indentation Thanks for working on this. I was hoping to submit a patch to propose putting -Wmisleading-indentation into -Wall, and have been testing the warning on the linux kernel to

Re: [PATCH] Refactor -Wmisleading-indentation API and extend coverage

2015-06-08 Thread Richard Sandiford
Patrick Palka patr...@parcs.ath.cx writes: At the same time this patch extends the coverage of the -Wmisleading-indentation implementation to catch misleading indentation involving the semicolon. (These changes are all contained in c-indentation.c.) For example, now we warn on the following

Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
On 08/06/15 17:31, Jakub Jelinek wrote: On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: Please work out with Vlad whether

Re: debug-early branch merged into mainline

2015-06-08 Thread Richard Biener
On June 8, 2015 7:14:19 PM GMT+02:00, Aldy Hernandez al...@redhat.com wrote: On 06/08/2015 09:30 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 2:05 PM, Aldy Hernandez al...@redhat.com wrote: On 06/08/2015 04:26 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 3:23 AM, Aldy Hernandez

Re: [C++ Patch] PR 65815

2015-06-08 Thread Paolo Carlini
Hi, On 06/08/2015 06:16 PM, Jason Merrill wrote: On 05/22/2015 02:46 PM, Paolo Carlini wrote: take a type, not a decl, as first argument. Why? This complicates calls. Yes, but, on the other hand, it's more consistent with the arguments of the various digest_init_*. Also, we don't

Re: [C++ Patch] PR 65815

2015-06-08 Thread Paolo Carlini
Hi again, On 06/08/2015 10:33 PM, Paolo Carlini wrote: Could you also check that we do the right thing for mem-initializers? Sure I will. I think we have a similar issue in expand_default_init: exactly when reshape_init is in order we fail to call it before digest_init. The below also passes

Re: debug-early branch merged into mainline

2015-06-08 Thread Aldy Hernandez
On 06/08/2015 02:59 PM, Richard Biener wrote: On June 8, 2015 7:14:19 PM GMT+02:00, Aldy Hernandez al...@redhat.com wrote: On 06/08/2015 09:30 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 2:05 PM, Aldy Hernandez al...@redhat.com wrote: On 06/08/2015 04:26 AM, Richard Biener wrote: On

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Jan Hubicka
On Mon, 8 Jun 2015, Jan Hubicka wrote: Hi, this is a variant of patch that globs also the rest of integer types. Note that we will still get false warnings out of lto-symtab when the values are not wrapped up in structures. This is because lto-symtab uses types_compatible_p that in

Re: Fix LTO streaming of BUILTINS_LOCATION

2015-06-08 Thread Jan Hubicka
Yeah, I think streaming all locations up to RESERVED_LOCATION_COUNT literally is more robust. Thus do bp_pack_int_in_range (bp, 0, RESERVED_LOCATION_COUNT, loc RESERVED_LOCATION_COUNT ? loc : RESERVED_LOCATION_COUNT); if (loc RESERVED_LOCATION_COUNT)

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Jan Hubicka
On Mon, 8 Jun 2015, Jan Hubicka wrote: Thank you. It is interesting that the DR exists. We do have comments about possibly completing the types by equiality established by the symbol table but I tought it is strictly invalid. Not sure how much that buy us though. As for specific

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-08 Thread Tom de Vries
On 08/06/15 17:55, Thomas Schwinge wrote: Hi Tom! On Mon, 8 Jun 2015 12:43:01 +0200, Tom de Vries tom_devr...@mentor.com wrote: There are two problems in try_transform_to_exit_first_loop_alt: 1. In case the latch is not a singleton bb, the function should return false rather than true. 2.

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Joseph Myers
On Mon, 8 Jun 2015, Jan Hubicka wrote: Joseph, I may be wrong, but I believe that the cross-compilation-unit representation compatibility (in C standard sense) is however not an equivalence class, so it can't be fully represented by TYPE_CANOINICAL Indeed, type compatibility is not

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Jan Hubicka
On Mon, 8 Jun 2015, Jan Hubicka wrote: Joseph, I may be wrong, but I believe that the cross-compilation-unit representation compatibility (in C standard sense) is however not an equivalence class, so it can't be fully represented by TYPE_CANOINICAL Indeed, type compatibility is not

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Joseph Myers
On Mon, 8 Jun 2015, Jan Hubicka wrote: Thank you. It is interesting that the DR exists. We do have comments about possibly completing the types by equiality established by the symbol table but I tought it is strictly invalid. Not sure how much that buy us though. As for specific examples.

Re: [PATCH] Optimize (CST1 A) == CST2 (PR tree-optimization/66299)

2015-06-08 Thread Marc Glisse
On Mon, 8 Jun 2015, Marek Polacek wrote: PR tree-optimization/66299 * match.pd ((CST1 A) == CST2 - A == ctz (CST2) - ctz (CST1) ((CST1 A) != CST2 - A != ctz (CST2) - ctz (CST1)): New You are braver than I am, I would have abbreviated ctz (CST2) - ctz (CST1) to CST3 in the

Re: debug-early branch merged into mainline

2015-06-08 Thread Aldy Hernandez
On 06/08/2015 09:30 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 2:05 PM, Aldy Hernandez al...@redhat.com wrote: On 06/08/2015 04:26 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 3:23 AM, Aldy Hernandez al...@redhat.com wrote: On 06/07/2015 02:33 PM, Richard Biener wrote: On June

pr66345.c size_t assumption bug

2015-06-08 Thread DJ Delorie
The testcase for pr 66345 assumes size_t is unsigned long instead of using the real type, which causes failures on some 16-bit targets. Ok? Also, I note that some tests check for __SIZE_TYPE__ as I do below, and others use it unconditionally as a replacement for size_t. Is there a convention?

Re: [C++ Patch] PR 65815

2015-06-08 Thread Paolo Carlini
.. in case isn't obvious: this case is already Ok: struct array { int data [2]; }; struct X { X() : a{ 1, 2 } { } array a; }; because perform_member_init calls reshape_init. Paolo.

Re: [PATCH, AARCH64] make stdarg functions work with +nofp

2015-06-08 Thread Jim Wilson
On Tue, Jun 2, 2015 at 3:45 AM, James Greenhalgh james.greenha...@arm.com wrote: On Tue, Jun 02, 2015 at 11:38:29AM +0100, Kyrill Tkachov wrote: Hi James, Jim, On 02/06/15 10:42, James Greenhalgh wrote: On Sat, May 23, 2015 at 12:24:00AM +0100, Jim Wilson wrote: The compiler currently ICEs

Re: pr66345.c size_t assumption bug

2015-06-08 Thread Marc Glisse
On Mon, 8 Jun 2015, DJ Delorie wrote: Also, I note that some tests check for __SIZE_TYPE__ as I do below, and others use it unconditionally as a replacement for size_t. Is there a convention? As far as I can tell, __SIZE_TYPE__ is always defined. The tests that check for it probably date

Re: [Patch, fortran, PR44672, v9] [F08] ALLOCATE with SOURCE and no array-spec

2015-06-08 Thread Damian Rouson
All, I sincerely hope this patch will hit the trunk soon. There are 9 users on the cc list for this bug so it is clearly of considerable user interest.I was recently informed that the following three-line program does not compile: $ cat source-allocation.f90 integer, allocatable ::

Re: [PATCH] Add debug msg to dump_file in add_new_function

2015-06-08 Thread Richard Biener
On Thu, Jun 4, 2015 at 4:42 PM, Tom de Vries tom_devr...@mentor.com wrote: On 04/06/15 15:20, Tom de Vries wrote: Hi, [ posted earlier as part of Don't dump low gimple functions in gimple dump, https://gcc.gnu.org/ml/gcc-patches/2014-05/msg01586.html, currently discussed at:

Re: genmatch: guess the type of a?b:c as b instead of a

2015-06-08 Thread Richard Biener
On Sat, Jun 6, 2015 at 1:34 PM, Marc Glisse marc.gli...@inria.fr wrote: Hello, as discussed around https://gcc.gnu.org/ml/gcc-patches/2015-06/msg00041.html we are currently guessing the type of a?b:c incorrectly. This does not affect current simplifications, because the only 'cond' in output

Re: [patch] fix _OBJC_Module defined but not used warning

2015-06-08 Thread Iain Sandoe
Hi Aldy, On 7 Jun 2015, at 12:37, Aldy Hernandez wrote: On 06/07/2015 06:19 AM, Andreas Schwab wrote: Another fallout: FAIL: obj-c++.dg/try-catch-5.mm -fgnu-runtime (test for excess errors) Excess errors: built-in: warning: '_OBJC_Module' defined but not used [-Wunused-variable]

Re: [PR64164] drop copyrename, integrate into expand

2015-06-08 Thread Richard Biener
On Sat, Jun 6, 2015 at 3:14 AM, Alexandre Oliva aol...@redhat.com wrote: On Apr 27, 2015, Richard Biener richard.guent...@gmail.com wrote: This should also mention that is_gimple_reg vars do not have their address taken. check +static tree +leader_merge (tree cur, tree next) Ick -

Re: Expand oacc kernels after pass_fre

2015-06-08 Thread Richard Biener
On Thu, 4 Jun 2015, Tom de Vries wrote: { gsi_next (gsi); continue; diff --git gcc/tree-ssa-sccvn.c gcc/tree-ssa-sccvn.c index e417a15..449a615 100644 --- gcc/tree-ssa-sccvn.c +++ gcc/tree-ssa-sccvn.c @@ -85,6 +85,7 @@ along

Re: Fix LTO streaming of BUILTINS_LOCATION

2015-06-08 Thread Jan Hubicka
+ bp_pack_value (bp, loc == BUILTINS_LOCATION, 1); + if (loc == BUILTINS_LOCATION) +return; Hmm, with this and #define DECL_IS_BUILTIN(DECL) \ (LOCATION_LOCUS (DECL_SOURCE_LOCATION (DECL)) = BUILTINS_LOCATION) shouldn't we rather stream all locations = BUILTINS_LOCATION

Re: Don't dump low gimple functions in gimple dump

2015-06-08 Thread Richard Biener
On Thu, Jun 4, 2015 at 5:02 PM, Tom de Vries tom_devr...@mentor.com wrote: On 22/05/15 11:27, Richard Biener wrote: On Thu, May 21, 2015 at 5:36 PM, Thomas Schwinge tho...@codesourcery.com wrote: Hi! It's just been a year. ;-P In early March, I (hopefully correctly) adapted Tom's patch

Re: [patch] Implement Ada support for DragonFly, improve it for FreeBSD

2015-06-08 Thread Eric Botcazou
Okay, I've attached them. I hope it helps! Thanks. The patch has been installed on the mainline. -- Eric Botcazou

Re: Fix LTO streaming of BUILTINS_LOCATION

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Jan Hubicka wrote: Hi, currently we stream BUILTINS_LOCATION by expanding it and streaming resulting filename/line/col tripplet. That is a nonsense and breaks some logic that special case it. This patch fixes it by special casing it same way as we do UNKNOWN_LOCATION

Re: [PR58315] reset inlined debug vars at return-to point

2015-06-08 Thread Richard Biener
On Wed, Jun 3, 2015 at 11:55 PM, Alexandre Oliva aol...@redhat.com wrote: On Feb 25, 2015, Alexandre Oliva aol...@redhat.com wrote: This patch fixes a problem that has been with us for several years. Variable tracking has no idea about the end of the lifetime of inlined variables, so it keeps

Re: [patch#2] PR other/65366: Fix gdbhooks.py for GDB with Python3

2015-06-08 Thread Richard Biener
On Wed, Jun 3, 2015 at 3:26 PM, Jan Kratochvil jan.kratoch...@redhat.com wrote: On Wed, 03 Jun 2015 10:25:20 +0200, Richard Biener wrote: gdb 7.9, python 2.7.6 attaching a fix; I do not know much Python so check it, please. OK for check-in? Python Exception type 'exceptions.NameError'

Re: debug-early branch merged into mainline

2015-06-08 Thread Richard Biener
On Mon, Jun 8, 2015 at 3:23 AM, Aldy Hernandez al...@redhat.com wrote: On 06/07/2015 02:33 PM, Richard Biener wrote: On June 7, 2015 6:00:05 PM GMT+02:00, Aldy Hernandez al...@redhat.com wrote: On 06/07/2015 11:25 AM, Richard Biener wrote: On June 7, 2015 5:03:30 PM GMT+02:00, Aldy

Re: Fix LTO streaming of BUILTINS_LOCATION

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Jan Hubicka wrote: + bp_pack_value (bp, loc == BUILTINS_LOCATION, 1); + if (loc == BUILTINS_LOCATION) +return; Hmm, with this and #define DECL_IS_BUILTIN(DECL) \ (LOCATION_LOCUS (DECL_SOURCE_LOCATION (DECL)) = BUILTINS_LOCATION) shouldn't we

Re: [PATCH] [AArch64] PR63870 Improve error messages for NEON single lane memory access intrinsics

2015-06-08 Thread Alan Lawrence
Thanks for working on this! I'd been fiddling around with a patch with some similar elements to this, but many trials with union types, subregs, etc., all worsened the register allocation and led to more unnecessary shuffling / moves. The only real thing I tried which you don't do here, was

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Tom de Vries wrote: On 04/06/15 10:28, Tom de Vries wrote: I'm ok with the patch and count on you to fix eventual fallout ;) Great, will do. And here is the fallout: * PR66442 - [6 regression] FAIL: gcc.dg/autopar/pr46885.c (test for excess errors) There

Re: [PATCH] [AArch64] PR63870 Improve error messages for NEON single lane memory access intrinsics

2015-06-08 Thread Alan Lawrence
Oh, have you tested bigendian? --Alan Charles Baylis wrote: This is another attempt at fixing this PR63870 for AArch64 (ARM is still to come). As before, the Q register variants are handled by moving the check for the lane bounds into builtin expansion. The handling of lane numbers is made

Re: [patch] fix _OBJC_Module defined but not used warning

2015-06-08 Thread Aldy Hernandez
On 06/08/2015 04:03 AM, Iain Sandoe wrote: Hi Aldy, On 7 Jun 2015, at 12:37, Aldy Hernandez wrote: On 06/07/2015 06:19 AM, Andreas Schwab wrote: Another fallout: FAIL: obj-c++.dg/try-catch-5.mm -fgnu-runtime (test for excess errors) Excess errors: built-in: warning: '_OBJC_Module' defined

[PATCH] Disable -Wunused warning for artificial decls (PR sanitizer/66452)

2015-06-08 Thread Marek Polacek
This PR is about a bogus -Wunused-variable warning about an internal structure (*.Lubsan_data*), exposed by the debug-early merge. I think the defined but not used warning should not be allowed for the compiler-generated entities. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2015-06-08

Re: [PATCH] Disable -Wunused warning for artificial decls (PR sanitizer/66452)

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Marek Polacek wrote: This PR is about a bogus -Wunused-variable warning about an internal structure (*.Lubsan_data*), exposed by the debug-early merge. I think the defined but not used warning should not be allowed for the compiler-generated entities.

[patch] libstdc++/66417 fix codecvt_utf16 bigendian

2015-06-08 Thread Jonathan Wakely
I forgot to swap the byte-order for codepoints that fit in a single UTF-16 character, fixed like so. Tested powerpc64-linux and powerpc64le-linux. Committing to trunk and gcc-5-branch. commit cc2dca496553bba0d09db49b98ccef0d728d9a36 Author: Jonathan Wakely jwak...@redhat.com Date: Mon Jun 8

Re: [PATCH, ARM] attribute target (thumb,arm) [4/6] respin (5th)

2015-06-08 Thread Ramana Radhakrishnan
On 08/06/15 09:45, Christian Bruel wrote: Hi Ramana, Ok, I see. The patch looks ok to me modulo the typo nits I pointed out, but I think Ramana should have the final say here as he's already started reviewing it and it adds quite a lot of functionality. Thanks, Kyrill do you have other

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-08 Thread Tom de Vries
On 04/06/15 10:28, Tom de Vries wrote: I'm ok with the patch and count on you to fix eventual fallout ;) Great, will do. And here is the fallout: * PR66442 - [6 regression] FAIL: gcc.dg/autopar/pr46885.c (test for excess errors) There are two problems in

[PATCH] Fix PR66422

2015-06-08 Thread Richard Biener
The following patch should fix the bogus array-bound warning caused by loop peeling which fails to split blocks after inserted unreachable calls (which is now fatal to optimization after removing the quadraticness in CFG cleanup to scan for noreturn calls). Bootstrap and regtest in progress on

[PATCH] Yet another simple fix to enhance outer-loop vectorization.

2015-06-08 Thread Yuri Rumyantsev
Hi All, Here is a simple fix which allows duplication of outer loops to perform peeling for number of iterations if outer loop is marked with pragma omp simd. Bootstrap and regression testing did not show any new failures. Is it OK for trunk? ChangeLog: 2015-06-08 Yuri Rumyantsev

RE: [Patch] [X86_64]: fix operand constraints in sse3_mwait

2015-06-08 Thread Kumar, Venkataramanan
Hi Uros, Checked the patch in the following branches and trunk after bootstrapping and regression testing them individually. GCC 5 branch https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=224215 GCC 4.9 branch https://gcc.gnu.org/viewcvs/gcc?view=revisionrevision=224214 GCC 4.8 branch

Commit: RX: Do not promote vector types

2015-06-08 Thread Nick Clifton
Hi Guys, I am applying the patch below to the RX's handling of vector functions. The RX ABI specifies that small integer return values should always be promoted to 32-bit values, but the code that performs this promotion was also affecting vector types. This results in internal

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Joseph Myers wrote: On Mon, 8 Jun 2015, Richard Biener wrote: I'm not sure the C standard mandates compatibility between struct { int i; } and struct { unsigned i; } for purposes of TBAA. Joseph? I don't think they are necessarily compatible for TBAA. Ok, but

Re: [commit#2] [patch#2] PR other/65366: Fix gdbhooks.py for GDB with Python3

2015-06-08 Thread Richard Biener
On Mon, Jun 8, 2015 at 3:37 PM, Jan Kratochvil jan.kratoch...@redhat.com wrote: On Mon, 08 Jun 2015 09:46:59 +0200, Richard Biener wrote: adding a import sys makes it work fine though. I do not see the sys error with either FSF GDB HEAD or Fedora 22 GDB. I agree it probably should be

[PATCH] Fix PR66413

2015-06-08 Thread Richard Biener
We fail to unshare exprs put into debug stmts during inlinign which creates bogus tree sharing. Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2015-06-08 Richard Biener rguent...@suse.de PR middle-end/66413 * tree-inline.c (insert_init_debug_bind):

Re: [patch] Adjust hash-table.h and it's pre-requisite includes.

2015-06-08 Thread Richard Biener
On Fri, Jun 5, 2015 at 5:24 PM, Andrew MacLeod amacl...@redhat.com wrote: There is a horrible morass of include dependencies between hash-map.h, mem-stats.h and hash-table.h. There are even includes in both directions (mem-stats.h and hash-map.h include each other, as do hash-map.h and

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Joseph Myers
On Mon, 8 Jun 2015, Richard Biener wrote: I'm not sure the C standard mandates compatibility between struct { int i; } and struct { unsigned i; } for purposes of TBAA. Joseph? I don't think they are necessarily compatible for TBAA. -- Joseph S. Myers jos...@codesourcery.com

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Joseph Myers
On Mon, 8 Jun 2015, Richard Biener wrote: On Mon, 8 Jun 2015, Joseph Myers wrote: On Mon, 8 Jun 2015, Richard Biener wrote: I'm not sure the C standard mandates compatibility between struct { int i; } and struct { unsigned i; } for purposes of TBAA. Joseph? I don't think

[gomp4] (NVPTX) thread barriers after OpenACC worker loops

2015-06-08 Thread Julian Brown
Hi, This patch adds a thread barrier after worker loops for OpenACC, in accordance with OpenACC 2.0a section 2.7.3 (worker loops): All workers will complete execution of their assigned iterations before any worker proceeds beyond the end of the loop.. (This is quite target-specific: work to

[PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Tom de Vries
Hi, this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: ... (insn 13 12 14 2 (parallel [ (set (reg/v/f:DI 37 r8 [orig:92 xD.1858 ] [92]) (plus:DI (reg:DI 37 r8 [96])

[PING^2][PATCH][3/3][PR65460] Mark offloaded functions as parallelized

2015-06-08 Thread Tom de Vries
On 17/04/15 12:08, Tom de Vries wrote: On 20-03-15 12:38, Tom de Vries wrote: On 19-03-15 12:05, Tom de Vries wrote: On 18-03-15 18:22, Tom de Vries wrote: Hi, this patch fixes PR65460. The patch marks offloaded functions as parallelized, which means the parloops pass no longer attempts to

Re: [patch] Adjust gcc-plugin.h

2015-06-08 Thread Richard Biener
On Mon, Jun 8, 2015 at 2:07 PM, Andrew MacLeod amacl...@redhat.com wrote: During the original flattening process I decided to use gcc-plugin.h as the kitchen sink for all includes that plugins might need. I think this has worked well for plugins, drastically reducing their dependency on gcc

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Jan Hubicka wrote: Hi, this is a variant of patch that globs also the rest of integer types. Note that we will still get false warnings out of lto-symtab when the values are not wrapped up in structures. This is because lto-symtab uses types_compatible_p that in turn

[patch] Adjust gcc-plugin.h

2015-06-08 Thread Andrew MacLeod
During the original flattening process I decided to use gcc-plugin.h as the kitchen sink for all includes that plugins might need. I think this has worked well for plugins, drastically reducing their dependency on gcc internal header file structure. What I didn't realize was that gcc's

Re: debug-early branch merged into mainline

2015-06-08 Thread Richard Biener
On Mon, Jun 8, 2015 at 2:05 PM, Aldy Hernandez al...@redhat.com wrote: On 06/08/2015 04:26 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 3:23 AM, Aldy Hernandez al...@redhat.com wrote: On 06/07/2015 02:33 PM, Richard Biener wrote: On June 7, 2015 6:00:05 PM GMT+02:00, Aldy Hernandez

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Jan Hubicka wrote: Hi, this patchs makes fortran's C_SIGNED_CHAR and C_SIZE_T interoperable with signed char and size_t as standard require. There are two issues here. First Fortran integers are always signed, but the standard insist on the signed integer to

Re: debug-early branch merged into mainline

2015-06-08 Thread Aldy Hernandez
On 06/08/2015 04:26 AM, Richard Biener wrote: On Mon, Jun 8, 2015 at 3:23 AM, Aldy Hernandez al...@redhat.com wrote: On 06/07/2015 02:33 PM, Richard Biener wrote: On June 7, 2015 6:00:05 PM GMT+02:00, Aldy Hernandez al...@redhat.com wrote: On 06/07/2015 11:25 AM, Richard Biener wrote: On

RFA: RL78: With -mes0 put read only data in the .frodata section

2015-06-08 Thread Nick Clifton
Hi DJ, The -mes0 option for the RL78 backend should put read only data into a special .frodata section, but unfortunately my recent update to the rl78_select_section function broke this behaviour. Below is a patch to fix this. Tested with no regressions on an rl78-elf toolchain. OK

[patch] libstdc++/66441 Fix wstring_convert when generating BOM

2015-06-08 Thread Jonathan Wakely
If the codecvt facet generates a BOM on every call to out() then we need to ensure there is enough capacity in the output string for the entire result, otherwise we loop and insert more than one BOM into the result. Tested powerpc64-linux and powerpc64-linux. Committing to trunk and

Re: [PATCH 01/13] recog: Increased max number of alternatives - v2

2015-06-08 Thread Andreas Krebbel
On 06/01/2015 10:22 AM, Jakub Jelinek wrote: On Fri, May 22, 2015 at 09:54:00AM +0200, Andreas Krebbel wrote: On Tue, May 19, 2015 at 10:40:26AM +0200, Andreas Krebbel wrote: On 05/18/2015 04:19 PM, Richard Biener wrote: Please use uint64_t instead. Done. Ok with that change? I've applied

Re: [BUILDROBOT]

2015-06-08 Thread Andreas Krebbel
On 06/05/2015 01:04 AM, Jan-Benedict Glaw wrote: Hi Andreas, On Mon, 2015-05-11 15:23:33 +0200, Andreas Krebbel kreb...@linux.vnet.ibm.com wrote: gcc/ * config/s390/constraints.md (j00, jm1, jxx, jyy, v): New constraints. * config/s390/predicates.md (const0_operand,

Re: [PATCH 01/13] recog: Increased max number of alternatives - v2

2015-06-08 Thread Jakub Jelinek
On Mon, Jun 08, 2015 at 03:32:50PM +0200, Andreas Krebbel wrote: On 06/01/2015 10:22 AM, Jakub Jelinek wrote: On Fri, May 22, 2015 at 09:54:00AM +0200, Andreas Krebbel wrote: On Tue, May 19, 2015 at 10:40:26AM +0200, Andreas Krebbel wrote: On 05/18/2015 04:19 PM, Richard Biener wrote:

[commit#2] [patch#2] PR other/65366: Fix gdbhooks.py for GDB with Python3

2015-06-08 Thread Jan Kratochvil
On Mon, 08 Jun 2015 09:46:59 +0200, Richard Biener wrote: adding a import sys makes it work fine though. I do not see the sys error with either FSF GDB HEAD or Fedora 22 GDB. I agree it probably should be there. Thus, ok with also adding a imoprt sys. Done and checked in: r224223 Jan

Re: [PATCH] Lift restrictions on SLP permutation for loop vect

2015-06-08 Thread Richard Biener
On Wed, 3 Jun 2015, Richard Biener wrote: This allows all permutations we can generate (according to the target). Bootstrap and regtest pending on x86_64-unknown-linux-gnu. So this turned up other issues thus the following is what I have committed after bootstrapping and testing on

[PING^2][PR65637] Fix ssa-handling code in expand_omp_for_static_chunk

2015-06-08 Thread Tom de Vries
On 18/05/15 14:53, Tom de Vries wrote: On 15-04-15 15:10, Tom de Vries wrote: Hi, This patch series fixes PR65637. Currently, ssa-handling code in expand_omp_for_static_chunk is dead and not exercised by testing. Ssa-handling code in omp-low.c is only triggered by pass_parallelize_loops, and

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Jan Hubicka
On Mon, 8 Jun 2015, Richard Biener wrote: On Mon, 8 Jun 2015, Joseph Myers wrote: On Mon, 8 Jun 2015, Richard Biener wrote: I'm not sure the C standard mandates compatibility between struct { int i; } and struct { unsigned i; } for purposes of TBAA. Joseph? I

[gomp4] declare directive [0/5]

2015-06-08 Thread James Norris
Hi! This patch series completes the implementation of the OpenACC declare directive. Patches applied to gomp-4_0-branch Thanks! Jim = gcc/ChangeLog.gomp * builtin-types.def (BT_FN_VOID_PTR_INT_UINT): New type. * gimple-pretty-print.c (dump_gimple_omp_target):

[gomp4] declare directive [4/5]

2015-06-08 Thread James Norris
diff --git a/gcc/builtin-types.def b/gcc/builtin-types.def index 7c3273f..0774da5 100644 --- a/gcc/builtin-types.def +++ b/gcc/builtin-types.def @@ -451,6 +451,7 @@ DEF_FUNCTION_TYPE_3 (BT_FN_BOOL_ULONG_ULONG_ULONGPTR, BT_BOOL, BT_ULONG, DEF_FUNCTION_TYPE_3

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Jan Hubicka
On Mon, 8 Jun 2015, Jan Hubicka wrote: Hi, this is a variant of patch that globs also the rest of integer types. Note that we will still get false warnings out of lto-symtab when the values are not wrapped up in structures. This is because lto-symtab uses types_compatible_p that in

Re: [PATCH] Optimize (CST1 A) == CST2 (PR tree-optimization/66299)

2015-06-08 Thread Marek Polacek
On Thu, May 28, 2015 at 09:48:10PM +0200, Marc Glisse wrote: Side note: if we are looking for extra patterns to simplify, llvm has an almost unlimited supply. Here are a few we don't seem to have (there are more where those came from), of course several need constraining / generalizing, it is

[patch] libstdc++/66030 fix codecvt exports for mingw32

2015-06-08 Thread Jonathan Wakely
The linker script assumes that std::mbstate_t has the name __mbstate_t for linkage purposes, but that's not necessarily true. For mingw32 it's just a typedef for int, so the patterns don't match. This adds a new mingw32-specific pattern for codecvt_byname's constructors and destructors, and

Fortran's C_CHAR type

2015-06-08 Thread Jan Hubicka
Hi, to furhter add to the topics to discuss, I noticed that Fortran FE seems to be quite ambivalent about C_CHAR type: [jh@gcc2-power8 gcc]$ cat ../b.f90 ! This testcase will abort if C_CHAR types are not interoperable module lto_type_merge_test use, intrinsic :: iso_c_binding implicit none

[Ping] [C++ Patch] PR 65815

2015-06-08 Thread Paolo Carlini
Hi, gently pinging this... On 05/22/2015 08:46 PM, Paolo Carlini wrote: Hi, surprisingly, for NSDMIs we don't use reshape_init and we end-up rejecting simple testcases like the below. It seems clear to me that we should - consistently with the comment preceding digest_init too - but I'm

Re: [gomp4] declare directive [2/5]

2015-06-08 Thread James Norris
diff --git a/gcc/cp/decl.c b/gcc/cp/decl.c index 261a12d..15da51e 100644 --- a/gcc/cp/decl.c +++ b/gcc/cp/decl.c @@ -78,6 +78,7 @@ along with GCC; see the file COPYING3. If not see #include cilk.h #include wide-int.h #include builtins.h +#include gomp-constants.h /* Possible cases of bad

[gomp4] declare directive [5/5]

2015-06-08 Thread James Norris
diff --git a/libgomp/libgomp.map b/libgomp/libgomp.map index fe38dc6..663c27c 100644 --- a/libgomp/libgomp.map +++ b/libgomp/libgomp.map @@ -318,6 +318,7 @@ GOACC_2.0 { global: GOACC_data_end; GOACC_data_start; + GOACC_declare; GOACC_enter_exit_data; GOACC_parallel; GOACC_update; @@

Re: [v3 PATCH] Implement N4387 and LWG 2367

2015-06-08 Thread Martin Sebor
* include/std/tuple In the hunk below, should the sizes_match template parameter be privatized (since it isn't part of the public interface)? Or perhaps even removed if it's not used? Martin @ -457,6 +457,73 @@ _GLIBCXX_BEGIN_NAMESPACE_VERSION } }; + + // Concept utility

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Richard Biener
On Mon, 8 Jun 2015, Jan Hubicka wrote: On Mon, 8 Jun 2015, Richard Biener wrote: On Mon, 8 Jun 2015, Joseph Myers wrote: On Mon, 8 Jun 2015, Richard Biener wrote: I'm not sure the C standard mandates compatibility between struct { int i; } and struct {

Re: [gomp4, fortran] Patch to fix continuation checks of OpenACC and OpenMP directives

2015-06-08 Thread Cesar Philippidis
On 06/07/2015 02:05 PM, Ilmir Usmanov wrote: Fixed fortran mail-list address. Sorry for inconvenience. 08.06.2015, 00:01, Ilmir Usmanov m...@ilmir.us: Hi Cesar! This patch fixes checks of OpenMP and OpenACC continuations in case if someone mixes them (i.e. continues OpenMP directive with

[gomp4] declare directive [1/5]

2015-06-08 Thread James Norris
diff --git a/gcc/c/c-parser.c b/gcc/c/c-parser.c index f508b91..83c1432 100644 --- a/gcc/c/c-parser.c +++ b/gcc/c/c-parser.c @@ -82,6 +82,7 @@ along with GCC; see the file COPYING3. If not see #include omp-low.h #include builtins.h #include gomp-constants.h +#include tree-iterator.h /*

[gomp4] declare directive [3/5]

2015-06-08 Thread James Norris
diff --git a/gcc/fortran/f95-lang.c b/gcc/fortran/f95-lang.c index 5003581..a889342 100644 --- a/gcc/fortran/f95-lang.c +++ b/gcc/fortran/f95-lang.c @@ -119,6 +119,8 @@ static const struct attribute_spec gfc_attribute_table[] = affects_type_identity } */ { omp declare target, 0, 0,

Re: [PATCH 1/2] Memory statistics enhancement.

2015-06-08 Thread Martin Liška
On 06/01/2015 06:16 PM, mliska wrote: Hi. Following 2 patches improve memory statistics infrastructure. First one ports pool allocator to the new infrastructure. And the second one makes column alignment properly. Both can bootstrap on x86_64-linux-pc and survive regression tests.

[PATCH] Fix PR66419

2015-06-08 Thread Richard Biener
This fixes PR66419. Bootstrap / regtest running on x86_64-unknown-linux-gnu. Richard. 2015-06-08 Richard Biener rguent...@suse.de PR tree-optimization/66419 * tree-vect-slp.c (vect_supported_load_permutation_p): Properly consider GROUP_GAP when detecting a perfect

Re: Fix more of C/fortran canonical type issues

2015-06-08 Thread Jan Hubicka
I think we should instead work towards eliminating the get_alias_set langhook first. The LTO langhook variant contains the same handling, btw, so just inline that into get_alias_set and see what remains? I see, i completely missed existence of gimple_get_alias_set. It makes more sense now.

Re: [PATCH, PR66444] Handle -fipa-ra in reload_combine

2015-06-08 Thread Jakub Jelinek
On Mon, Jun 08, 2015 at 02:04:12PM +0200, Tom de Vries wrote: this patch fixes PR66444, a problem with -fipa-ra in reload_combine. The problem is that for the test-case, reload_combine combines these two insns: Please work out with Vlad whether reload_cse_move2add doesn't need similar fix

Re: [patch] libstdc++/66030 fix codecvt exports for mingw32

2015-06-08 Thread Martin Sebor
On 06/08/2015 09:12 AM, Jonathan Wakely wrote: The linker script assumes that std::mbstate_t has the name __mbstate_t for linkage purposes, but that's not necessarily true. For mingw32 it's just a typedef for int, so the patterns don't match. This adds a new mingw32-specific pattern for

Re: [C++ Patch] PR 65815

2015-06-08 Thread Jason Merrill
On 05/22/2015 02:46 PM, Paolo Carlini wrote: take a type, not a decl, as first argument. Why? This complicates calls. Could you also check that we do the right thing for mem-initializers? Jason

Re: [PING][PATCH][PR65443] Add transform_to_exit_first_loop_alt

2015-06-08 Thread Thomas Schwinge
Hi Tom! On Mon, 8 Jun 2015 12:43:01 +0200, Tom de Vries tom_devr...@mentor.com wrote: There are two problems in try_transform_to_exit_first_loop_alt: 1. In case the latch is not a singleton bb, the function should return false rather than true. 2. The check for singleton bb should ignore

  1   2   >