Re: Splitting up gcc/omp-low.c?

2015-12-10 Thread Jakub Jelinek
On Wed, Dec 09, 2015 at 06:23:22PM +0100, Bernd Schmidt wrote: > On 12/09/2015 05:24 PM, Thomas Schwinge wrote: > > > >In addition to that, how about we split up gcc/omp-low.c into several > >files? Would it make sense (I have not yet looked in detail) to do so > >along the borders of the several

Re: [Fortran, Patch] Memory sync after coarray image control statements and assignment

2015-12-10 Thread Alessandro Fanfarillo
2015-12-09 23:16 GMT+01:00 Tobias Burnus : > Thanks. Committed as r231476. Thanks. > > Do we need to do anything about GCC 5 or is this only a GCC 6 issue? > Yes, the patch should be applied to GCC 5 too. > That can be changed: Simply fill out the form and list me (burnus (at] > gcc.gnu.org) as

[committed, obvious] Remove invalid assert in find_func_aliases_for_builtin_call

2015-12-10 Thread Tom de Vries
[ was : Re: [PATCH, PR68716] Fix GOMP/GOACC_parallel handling in find_func_clobbers ] On 09/12/15 11:01, Tom de Vries wrote: [ was: Re: [PATCH, PR46032] Handle BUILT_IN_GOMP_PARALLEL in ipa-pta ] On 30/11/15 13:11, Tom de Vries wrote: On 30/11/15 10:16, Richard Biener wrote: On Mon, 30 Nov 2

[PATCH] Fix PR68721

2015-12-10 Thread Richard Biener
This fixes a wrong-code case after my IPA split fix (with PR68672 still unfixed). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2015-12-10 Richard Biener PR ipa/68721 * ipa-split.c (split_function): Record return value properly when

[PATCH] Make tree if-conversion honor PARAM_ALLOW_STORE_DATA_RACES

2015-12-10 Thread Richard Biener
The following makes us honor PARAM_ALLOW_STORE_DATA_RACES instead of making -ftree-loop-if-convert-stores an if-conversion specific "alias" of that. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. I've also verified it passes SPEC CPU 2006 on x86_64. Richard. 2015-12-10

[PATCH] Fix wrong-code with IPA PTA

2015-12-10 Thread Richard Biener
This fixes a missing constraint for by-reference DECL_RESULT in nonlocal_p mode. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-12-10 Richard Biener * tree-ssa-structalias.c (create_function_info_for): Add missing constraint from nonlocal for DEC

[PATCH] Fix IPA PTA vs. PTA regressions

2015-12-10 Thread Richard Biener
The following patch takes a stab at fixing the most obvious regression of IPA PTA vs. PTA - dumbing down of pt_solution_includes_global with - /* ??? This predicate is not correct for the IPA-PTA solution - as we do not properly distinguish between unit escape points - and global var

Re: [PATCH] Fix up fold_ctor_reference and fully_constant_vn_reference_p (PR tree-optimization/68785)

2015-12-10 Thread Richard Biener
On Wed, 9 Dec 2015, Jakub Jelinek wrote: > Hi! > > On a testcase like below which would trigger UB at runtime we trigger > UB in the compiler, by reading uninitialized bytes. > > The VCE folding for which native_{encode,interpret}_expr has been originally > written passes the length from the fir

Re: [PATCH, ARM] PR68674 Fix LTO support for neon builtins and error catching

2015-12-10 Thread Kyrill Tkachov
On 10/12/15 09:26, Christian Bruel wrote: Hi Kyrill, On 12/09/2015 06:32 PM, Kyrill Tkachov wrote: Hi Christian, On 08/12/15 12:53, Christian Bruel wrote: Hi, The order of the NEON builtins construction has led to complications since the attribute target support. This was not a problem whe

Re: [PATCH, ARM] PR68674 Fix LTO support for neon builtins and error catching

2015-12-10 Thread Kyrill Tkachov
On 10/12/15 10:11, Christian Bruel wrote: On 12/10/2015 10:59 AM, Kyrill Tkachov wrote: On 10/12/15 09:26, Christian Bruel wrote: Hi Kyrill, On 12/09/2015 06:32 PM, Kyrill Tkachov wrote: Hi Christian, On 08/12/15 12:53, Christian Bruel wrote: Hi, The order of the NEON builtins construc

Re: [AArch64] Emit square root using the Newton series

2015-12-10 Thread Kyrill Tkachov
On 09/12/15 18:50, Evandro Menezes wrote: On 12/09/2015 11:16 AM, Kyrill Tkachov wrote: On 09/12/15 17:02, Kyrill Tkachov wrote: On 09/12/15 16:59, Evandro Menezes wrote: On 12/09/2015 10:52 AM, Kyrill Tkachov wrote: Hi Evandro, On 08/12/15 21:35, Evandro Menezes wrote: Emit square root

[PATCH][Testsuite]Cleanup logs from gdb tests by adding newlines

2015-12-10 Thread Alan Lawrence
Runs of the guality testsuite can sometimes end up with gcc.log containing malformed lines like: A debugging session is active.PASS: gcc.dg/guality/pr36728-1.c -O2 line 18 arg4 == 4 A debugging session is active.PASS: gcc.dg/guality/restrict.c -O2 line 30 type:ip == int * Inferior

Re: [PATCH 2/7][ARM] Multilib support for ARMv8.1.

2015-12-10 Thread Ramana Radhakrishnan
On Mon, Dec 7, 2015 at 4:05 PM, Matthew Wahab wrote: > Ping. Updated patch attached. > Matthew > > > On 26/11/15 15:58, Matthew Wahab wrote: >> >> This patch sets up multilib support for ARMv8.1, treating it as a >> synonym for ARMv8. Since ARMv8.1 integer, FP or SIMD >> instructions are only gene

Re: [PATCH 1/7][ARM] Add support for ARMv8.1.

2015-12-10 Thread Ramana Radhakrishnan
On Mon, Dec 7, 2015 at 4:04 PM, Matthew Wahab wrote: > Ping. Updated patch attached. > Matthew > > > On 26/11/15 15:55, Matthew Wahab wrote: >> >> Hello, >> >> >> ARMv8.1 includes an extension to ARM which adds two Adv.SIMD >> instructions, vqrdmlah and vqrdmlsh. This patch set adds support for >>

Re: [PATCH 3/7][ARM] Add patterns for new instructions

2015-12-10 Thread Ramana Radhakrishnan
On Mon, Dec 7, 2015 at 4:06 PM, Matthew Wahab wrote: > Ping. Updated patch attached. > Matthew > > > On 26/11/15 16:00, Matthew Wahab wrote: >> >> Hello, >> >> This patch adds patterns for the instructions, vqrdmlah and vqrdmlsh, >> introduced in the ARMv8.1 architecture. The instructions are made

Re: [PATCH 4/7][ARM] Add ACLE feature macro for ARMv8.1 instructions.

2015-12-10 Thread Ramana Radhakrishnan
On Tue, Dec 8, 2015 at 7:45 AM, Christian Bruel wrote: > Hi Matthew, > > > On 12/07/2015 05:07 PM, Matthew Wahab wrote: >> >> Ping. Updated patch attached. >> Matthew >> >> >> On 26/11/15 16:01, Matthew Wahab wrote: >>> >>> Hello, >>> >>> This patch adds the feature macro __ARM_FEATURE_QRDMX to in

Re: ipa-cp heuristics fixes

2015-12-10 Thread Martin Jambor
Hi, thanks for looking into this, I only have one question: On Thu, Dec 10, 2015 at 08:30:37AM +0100, Jan Hubicka wrote: > Martin, > while looking into the ipa-cp dumps for bzip and Firefox I noticed few issues. > First of all, ipcp_cloning_candidate_p calls > optimize_function_for_speed_p (DECL

Re: [PATCH 5/7][Testsuite] Support ARMv8.1 ARM tests.

2015-12-10 Thread Ramana Radhakrishnan
On Mon, Dec 7, 2015 at 4:10 PM, Matthew Wahab wrote: > On 27/11/15 17:11, Matthew Wahab wrote: >> >> On 27/11/15 13:44, Christophe Lyon wrote: On 26/11/15 16:02, Matthew Wahab wrote: >> >> > This patch adds ARMv8.1 support to GCC Dejagnu, to allow ARM > tests to specify targest a

Re: [PATCH 6/7][ARM] Add ACLE intrinsics vqrdmlah and vqrdmlsh

2015-12-10 Thread Ramana Radhakrishnan
On Mon, Dec 7, 2015 at 4:12 PM, Matthew Wahab wrote: > Ping. Updated patch attached. > Matthew > > > On 26/11/15 16:04, Matthew Wahab wrote: >> >> Hello, >> >> This patch adds the ACLE intrinsics for the instructions introduced in >> ARMv8.1. It adds the vqrmdlah and vqrdmlsh forms of the instrins

Re: [PATCH 7/7][ARM] Add ACLE intrinsics vqrdmlah_lane and vqrdmlsh_lane

2015-12-10 Thread Ramana Radhakrishnan
On Thu, Nov 26, 2015 at 4:04 PM, Matthew Wahab wrote: > Hello, > > This patch adds the ACLE intrinsics for the instructions introduced in > ARMv8.1. It adds the vqrmdlah_lane and vqrdmlsh_lane forms of the > instrinsics to the arm_neon.h header, together with the ARM builtins > used to implement t

Re: [PATCH 1/7][ARM] Add support for ARMv8.1.

2015-12-10 Thread Ramana Radhakrishnan
On Thu, Dec 10, 2015 at 10:43 AM, Ramana Radhakrishnan wrote: > On Mon, Dec 7, 2015 at 4:04 PM, Matthew Wahab > wrote: >> Ping. Updated patch attached. >> Matthew >> >> >> On 26/11/15 15:55, Matthew Wahab wrote: >>> >>> Hello, >>> >>> >>> ARMv8.1 includes an extension to ARM which adds two Adv.SI

Re: [PATCH] Avoid false vector mask conversion

2015-12-10 Thread Richard Biener
On Fri, Dec 4, 2015 at 4:00 PM, Ilya Enkovich wrote: > On 02 Dec 16:27, Richard Biener wrote: >> On Wed, Dec 2, 2015 at 4:24 PM, Ilya Enkovich wrote: >> > >> > The problem is that conversion is supposed to be handled by >> > vectorizable_conversion, >> > but it fails to because it is not actually

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

2015-12-10 Thread Richard Biener
On Wed, Dec 9, 2015 at 10:31 PM, Markus Trippelsdorf wrote: > On 2015.12.09 at 10:53 -0800, H.J. Lu wrote: >> >> Empty C++ class is a corner case which isn't covered in psABI nor C++ ABI. >> There is no mention of "empty record" in GCC documentation. But there are >> plenty of "empty class" in gc

Re: [PATCH, i386] Use scalar mask for 16-byte and 32-byte vectors when possible

2015-12-10 Thread Ilya Enkovich
Ping 2015-11-26 13:33 GMT+03:00 Ilya Enkovich : > Hi, > > This patch allows usage of scalar masks for ymm and xmm registers when target > supports it. Bootstrapped and regtested on x86_64-unknown-linux-gnu. OK for > trunk? > > Thanks, > Ilya > -- > gcc/ > > 2015-11-26 Ilya Enkovich > >

Re: Splitting up gcc/omp-low.c?

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 09:08 AM, Jakub Jelinek wrote: On Wed, Dec 09, 2015 at 06:23:22PM +0100, Bernd Schmidt wrote: On 12/09/2015 05:24 PM, Thomas Schwinge wrote: In addition to that, how about we split up gcc/omp-low.c into several files? Would it make sense (I have not yet looked in detail) to do s

Re: Splitting up gcc/omp-low.c?

2015-12-10 Thread Jakub Jelinek
On Thu, Dec 10, 2015 at 12:26:10PM +0100, Bernd Schmidt wrote: > On 12/10/2015 09:08 AM, Jakub Jelinek wrote: > >On Wed, Dec 09, 2015 at 06:23:22PM +0100, Bernd Schmidt wrote: > >>On 12/09/2015 05:24 PM, Thomas Schwinge wrote: > >>> > >>>In addition to that, how about we split up gcc/omp-low.c into

Re: [PATCH v4] Fix shrink-wrapping bug (PR67778, PR68634)

2015-12-10 Thread Bernd Schmidt
On 12/09/2015 09:47 PM, Segher Boessenkool wrote: After shrink-wrapping has found the "tightest fit" for where to place the prologue, it tries move it earlier (so that frame saves are run earlier) -- but without copying any more basic blocks. Unfortunately a candidate block we select can be insi

Re: [PATCH] update -Wall and -Wextra documentation

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 12:07 AM, Martin Sebor wrote: * invoke.texi (Warning Options): Update -Wall options. Clarify when some -Wextra options are enabled. Add -Wplacement-new example. I tried to check this list against c.opt - I figure this should contain essentially the ones that ar

Re: [hsa 3/10] HSA libgomp plugin [part 1/2]

2015-12-10 Thread Martin Liška
On 12/09/2015 01:15 PM, Jakub Jelinek wrote: > On Mon, Dec 07, 2015 at 12:20:49PM +0100, Martin Jambor wrote: >> +#include >> +#include >> +#include >> +#include >> +#include "libgomp-plugin.h" >> +#include "gomp-constants.h" >> +#include "hsa.h" >> +#include "hsa_ext_finalize.h" > > If these

Re: [hsa 3/10] HSA libgomp plugin [part 2/2]

2015-12-10 Thread Martin Liška
On 12/09/2015 01:15 PM, Jakub Jelinek wrote: > On Mon, Dec 07, 2015 at 12:20:49PM +0100, Martin Jambor wrote: >> +#include >> +#include >> +#include >> +#include >> +#include "libgomp-plugin.h" >> +#include "gomp-constants.h" >> +#include "hsa.h" >> +#include "hsa_ext_finalize.h" > > If these

Re: [PATCH][ARM] PR 49526: Add support for smmul,smmla,smmls instructions

2015-12-10 Thread Kyrill Tkachov
On 25/11/15 10:36, Kyrill Tkachov wrote: Hi Yvan, On 24/11/15 15:05, Yvan Roux wrote: Hi Kyrill, On 24 November 2015 at 14:31, Kyrill Tkachov wrote: Ping. Thanks, Kyrill On 13/11/15 11:50, Kyrill Tkachov wrote: Ping. https://gcc.gnu.org/ml/gcc-patches/2015-11/msg00686.html Thanks, Kyr

Re: [RFA] Transparent alias suport part 10: Fix base+offset alias analysis oracle WRT aliases

2015-12-10 Thread Eric Botcazou
> PR ipa/61886 > PR middle-end/25140 > * tree-ssa-alias.c (ptr_deref_may_alias_decl_p): Usecompare_base_decls > (nonoverlapping_component_refs_of_decl_p): Update sanity check. > (decl_refs_may_alias_p): Use compare_base_decls. > * alias.c: Include cgraph.h >

Re: [PATCH] rtlanal: Fix bits/bytes confusion in set_noop_p (PR68814)

2015-12-10 Thread Eric Botcazou
> My case comes from the gcc.dg/pr65492-2.c, the "test1int2" case. > combine has made an insn > modifying insn i321: zero_extract(%3:DI,0x20,0)=%3:DI > which is splatting the SImode parameter to both the high and low halves > of the dest reg. Then, it tries to combine it with the USE of

Re: [testsuite][ARM target attributes] Fix effective_target tests

2015-12-10 Thread Kyrill Tkachov
Hi Christophe, On 08/12/15 11:18, Christophe Lyon wrote: On 8 December 2015 at 11:50, Kyrill Tkachov wrote: Hi Christophe, On 27/11/15 13:00, Christophe Lyon wrote: Hi, After the recent commits from Christian adding target attributes support for ARM FPU settings, I've noticed that some of

[v3] avoid alignment of static variables affecting stack's

2015-12-10 Thread Jan Beulich
Function (or more narrow) scope static variables (as well as others not placed on the stack) should also not have any effect on the stack alignment. I noticed the issue first with Linux'es dynamic_pr_debug() construct using an 8-byte aligned sub-file-scope local variable. According to my checking

Re: [RFA] [PATCH] [PR tree-optimization/68619] Avoid direct cfg cleanups in tree-ssa-dom.c [0/3] v2

2015-12-10 Thread Richard Biener
On Thu, Dec 10, 2015 at 12:23 AM, Jeff Law wrote: > Richi and I have been discussing revamping slightly how DOM handles > conditionals which it detects are always true or always false. > > During gcc6 stage1 I added code to allow DOM to clean them up > immediately, primarily to avoid the waste of

Re: [testsuite][ARM target attributes] Fix effective_target tests

2015-12-10 Thread Christophe Lyon
On 10 December 2015 at 13:30, Kyrill Tkachov wrote: > Hi Christophe, > > > On 08/12/15 11:18, Christophe Lyon wrote: >> >> On 8 December 2015 at 11:50, Kyrill Tkachov >> wrote: >>> >>> Hi Christophe, >>> >>> >>> On 27/11/15 13:00, Christophe Lyon wrote: Hi, After the recent co

Re: [testsuite][ARM target attributes] Fix effective_target tests

2015-12-10 Thread Kyrill Tkachov
On 10/12/15 13:04, Christophe Lyon wrote: On 10 December 2015 at 13:30, Kyrill Tkachov wrote: Hi Christophe, On 08/12/15 11:18, Christophe Lyon wrote: On 8 December 2015 at 11:50, Kyrill Tkachov wrote: Hi Christophe, On 27/11/15 13:00, Christophe Lyon wrote: Hi, After the recent commi

Re: [PATCH] Handle BUILT_IN_GOACC_PARALLEL in ipa-pta

2015-12-10 Thread Tom de Vries
[ copy-pasting-with-quote from https://gcc.gnu.org/ml/gcc-patches/2015-12/msg00420.html , for some reason I didn't get this email ] On Thu, 3 Dec 2015, Tom de Vries wrote: The flag is set here in expand_omp_target: ... 12682 /* Prevent IPA from removing child_fn as unreachable,

Re: [PATCH] Fix up noce_try_abs again (PR rtl-optimization/68670)

2015-12-10 Thread Bernd Schmidt
On 12/09/2015 11:37 PM, Jakub Jelinek wrote: Not sure what I've been thinking when writing the previous noce_try_abs fix. I thought that the optimization can be applied for all the conditions, and whether it can be applied depends on if it is cond ? ~x : x or cond ? x : ~x. But that is not the c

[PATCH] Fix PR68817

2015-12-10 Thread Richard Biener
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-12-10 Richard Biener PR tree-optimization/68817 * tree-vect-slp.c (vect_attempt_slp_rearrange_stmts): Detect gaps early. * gfortran.dg/pr68817.f90: New testcase. Index: gcc/tree-vec

Re: [PATCH] Fix up noce_try_abs again (PR rtl-optimization/68670)

2015-12-10 Thread Jakub Jelinek
On Thu, Dec 10, 2015 at 02:20:04PM +0100, Bernd Schmidt wrote: > This turns into a much simpler if statement, doesn't it? Ok with that > change. Ok, will do, thanks. > >+/* PR rtl-optimization/68670 */ > >+/* { dg-do run } */ > >+/* { dg-options "-O2 -ftracer" } */ > > Curious what effect -ftrac

RE: [PATCH] [ARC] Add support for atomic memory built-in.

2015-12-10 Thread Claudiu Zissulescu
Patch applied: Committed r231509 Thanks, Claudiu > -Original Message- > From: Joern Wolfgang Rennecke [mailto:g...@amylaar.uk] > Sent: Wednesday, December 09, 2015 6:11 AM > To: Claudiu Zissulescu; gcc-patches@gcc.gnu.org > Cc: francois.bed...@synopsys.com; jeremy.benn...@embecosm.com >

Re: [PATCH PR68542]

2015-12-10 Thread Richard Biener
On Fri, Dec 4, 2015 at 4:07 PM, Yuri Rumyantsev wrote: > Hi Richard. > > Thanks a lot for your review. > Below are my answers. > > You asked why I inserted additional check to > ++ b/gcc/tree-ssa-forwprop.c > @@ -373,6 +373,11 @@ combine_cond_expr_cond (gimple *stmt, enum > tree_code code, tree ty

[Ada] Comment fixes

2015-12-10 Thread Eric Botcazou
Tested on x86-64/Linux, applied on the mainline. 2015-12-10 Eric Botcazou * gcc-interface/gigi.h (create_var_decl): Adjust comment. (create_subprog_decl): Likewise. * gcc-interface/utils.c (create_var_decl): Likewise. (create_subprog_decl): Likewise. -- Eric

Re: [PATCH] Fix detection of setrlimit in libstdc++ testsuite

2015-12-10 Thread Maxim Kuvyrkov
> On Nov 11, 2015, at 7:56 PM, Maxim Kuvyrkov wrote: > > Hi, > > This patch fixes an obscure cross-testing problem that crashed (OOMed) our > boards at Linaro. Several tests in libstdc++ (e.g., [1]) limit themselves to > some reasonable amount of RAM and then try to allocate 32 gigs. > Unfo

Re: [v3] avoid alignment of static variables affecting stack's

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 01:38 PM, Jan Beulich wrote: * cfgexpand.c (expand_one_var): Exclude static and external variables when adjusting stack alignment related state. gcc/testsuite/ 2015-12-10 Jan Beulich * gcc.c-torture/execute/stkalign.c: New. --- 2015-12-09/gcc/cfgexpand

[patch] Improve generated libstdc++ API docs

2015-12-10 Thread Jonathan Wakely
This adjusts some Doxygen comments and updates the Doxygen config file to ensure all headers are processed (previously doxygen was ignoring filenames without an extension, which is a lot of the library!) There's a workaround in include/std/bitset for the bug https://bugzilla.gnome.org/show_bug.c

[Ada] Fix internal error on call taking empty array Out parameter

2015-12-10 Thread Eric Botcazou
This is a regression present on the mainline only, for a pathological case where an empty array is passed as Out parameter to a procedure, but the code is actually never executed because it's inside an empty loop... Tested on x86-64/Linux, applied on the mainline. 2015-12-10 Eric Botcazou

[PATCH][RX] v2 instructions support.

2015-12-10 Thread Yoshinori Sato
Add RX v2 enhancement instructions support. gcc/ChangeLog 2015/12/10 Yoshinori Sato * config/rx/constraints.md: Add constraint "q". * doc/md.texi: Likewise. * config/rx/rx-opts.h(rx_cpu_types): Add type RXV2. * config/rx/rx.c(rx_print_operand): Add a0, a1 and ex

[PATCH] Fix IPA PTA wrong-code bug with LTO

2015-12-10 Thread Richard Biener
The following patch fixes a latent issue in IPA PTA with LTO where it doesn't properly honor partition boundaries. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-12-10 Richard Biener * tree-ssa-structalias.c (struct variable_info): Add is_ipa_esc

Fix PR ada/48013

2015-12-10 Thread Eric Botcazou
This reports an interesting case where the compiler emits an empty elaboration routine and doesn't report the violation of the associated No_Elaboration_Code restriction. Probably harmless in practice, but this small hole in gigi can easily be plugged. Tested on x86-64/Linux, applied on the ma

[patch] libstdc++/68825 Fix dejagnu directives in shared_ptr test

2015-12-10 Thread Jonathan Wakely
This XFAIL test should have been used { dg-do compile } so it doesn't get marked UNRESOLVED when it fails to compile. The test was also broken, the assignment operator compiled OK, but because the test used dg-excess-errors instead of dg-error the later failure for the copy construction matched it

[PATCH][combine] Don't create LSHIFTRT of zero bits in change_zero_ext

2015-12-10 Thread Kyrill Tkachov
Hi all, At the end of combine if it fails to recognise the patterns it produces it tries again after transforming all zero_extends and zero_extracts into and-immediate plus an LSHIFTRT if needed. However, it will construct an LSHIFTRT inside the AND even if the shift distance is 0, which can hu

[PATCH] testsuite/lib/multline.exp: show test name and line numbers

2015-12-10 Thread David Malcolm
The output from multiline.exp doesn't scale well when reviewing and comparing .sum files. Lines in e.g. gcc.sum from a dg-{begin|end}-multiline-output pair are currently of the form: PASS: expected multiline pattern 0 was found: "\s*myvar = myvar\.x;.*\n ~\^~\n" as compared to e.g

[patch] Update -std=gnu++1y in experimental::shared_ptr tests

2015-12-10 Thread Jonathan Wakely
Mechanical change to update the spelling of the -std option. Committed to trunk. commit ec7bb2c77a41d6e34e4e29e24195845380ef071f Author: Jonathan Wakely Date: Thu Dec 10 14:34:52 2015 + Change tests to use -std=gnu++14 not -std=gnu++1y * testsuite/experimental/memory/shared_

RE: [PATCH][ARC] Refurbish emitting DWARF2 for epilogue.

2015-12-10 Thread Claudiu Zissulescu
> > OTOH, the example you give also shows a much more nuanced approach to > throttling optimization: > the patch doesn't dead all epilogue scheduling, but specifically tests for the > presence of a frame related insn at the point where it could cause trouble. Actually, in my revised patch I do m

Free up bits in DECLs and TYPEs

2015-12-10 Thread Michael Matz
Hello, the other day Richi wondered why we specify alignment in bits, instead of in log2, as if e.g. a 12 byte alignment would make much sense (sure, an alignment of 12 byte means the address is evenly dividable by 12; great!). This patch changes the two places where we specify alignment (types

Re: [ARM] Use vector wide add for mixed-mode adds

2015-12-10 Thread Kyrill Tkachov
Hi Michael, A few comments while I look deeper into this patch... On 30/11/15 01:18, Michael Collison wrote: This is a modified version of my previous patch that supports vector wide add. I added support for vaddw on big endian when generating the parallel operand for the vector select. The

Re: [PATCH] Add testcase for c++/68348

2015-12-10 Thread Jason Merrill
OK. Jason

[committed 4/5] Fix -Wmisleading-indentation warning in ifcvt.c

2015-12-10 Thread David Malcolm
../../../src/gcc/ifcvt.c: In function ‘bool noce_try_inverse_constants(noce_if_info*)’: ../../../src/gcc/ifcvt.c:1233:2: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation] seq = end_ifcvt_sequence (if_info); ^~~ ../../../src/gcc/ifcvt.c:1230:7: note: ..

[committed 5/5] Fix -Wmisleading-indentation warning in graphite-optimize-isl.c

2015-12-10 Thread David Malcolm
../../../src/gcc/graphite-optimize-isl.c: In function ‘isl_union_set* scop_get_domains(scop_p)’: ../../../src/gcc/graphite-optimize-isl.c:362:5: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation] return res; ^~ In file included from ../../../s

[committed 2/5] Fix misleading indentation in gcc/fortran/io.c

2015-12-10 Thread David Malcolm
Fix this warning: ../../../src/gcc/fortran/io.c: In function ‘match gfc_match_open()’: ../../../src/gcc/fortran/io.c:2003:4: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation] if (!compare_to_allowed_values ("DELIM", delim, NULL, NULL, ^ ../../../src/

[committed 3/5] Fix -Wmisleading-indentation warning in gcc/regrename.c

2015-12-10 Thread David Malcolm
../../../src/gcc/regrename.c: In function ‘void scan_rtx_address(rtx_insn*, rtx_def**, reg_class, scan_actions, machine_mode, addr_space_t)’: ../../../src/gcc/regrename.c:1314:7: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation] break; ^ ../..

Re: [committed 1/5] Fix -Wmisleading-indentation warning in function.c

2015-12-10 Thread David Malcolm
FWIW, I believe trunk is clean for -Wmisleading-indentation after these 5 patches (for x86_64-pc-linux-gnu, at least).

Gather hash-tab statistics only with GATHER_STATISTICS

2015-12-10 Thread Michael Matz
Hi, while profiling cc1plus I noticed high hash-table activity for gathering statistics, even though I haven't configured with --enable-gather-detailed-mem-stats. Turns out the hash table rewrite hard-coded the relevant settings to true. This patch makes it initialized by GATHER_STATISTICS.

[committed 1/5] Fix -Wmisleading-indentation warning in function.c

2015-12-10 Thread David Malcolm
Fix this warning: ../../../src/gcc/function.c: In function ‘void locate_and_pad_parm(machine_mode, tree, int, int, int, tree, args_size*, locate_and_pad_arg_data*)’: ../../../src/gcc/function.c:4123:2: error: statement is indented as if it were guarded by... [-Werror=misleading-indentation] {

[Fortran, Patch} Fix ICE for coarray Critical inside module procedure

2015-12-10 Thread Alessandro Fanfarillo
Dear all, the compiler returns an ICE when a coarray critical section is used inside a module procedure. The symbols related with the lock variables were left uncommitted inside resolve_critical(). A gfc_commit_symbol after each symbol or a gfc_commit_symbols at the end of resolve_critical() fixed

Re: [PATCH][combine] Don't create LSHIFTRT of zero bits in change_zero_ext

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 03:36 PM, Kyrill Tkachov wrote: I'm okay with delaying this for next stage 1 if people prefer, though I think it's pretty low risk. I think this is something we should fix now. + x = XEXP (x, 0); + if (start > 0) + x = gen_rtx_LSHIFTRT (mode, x, GEN_INT

Re: [v3] avoid alignment of static variables affecting stack's

2015-12-10 Thread Jan Beulich
>>> On 10.12.15 at 14:53, wrote: > On 12/10/2015 01:38 PM, Jan Beulich wrote: >> --- 2015-12-09/gcc/cfgexpand.c >> +++ 2015-12-09/gcc/cfgexpand.c >> @@ -1544,12 +1544,15 @@ static HOST_WIDE_INT >> expand_one_var (tree var, bool toplevel, bool really_expand) >> { >> unsigned int align = BIT

Re: [PATCH] v2 Re: [PATCH] RFC: Use Levenshtein spelling suggestions in Fortran FE

2015-12-10 Thread Tobias Burnus
David Malcolm wrote: > On Sat, 2015-12-05 at 20:53 +0100, Mikael Morin wrote: > > to get things moving again, a few comments on top of David Malcolm's: [...] > > It seems you are considering some candidates more than once here. [...] > > You have to start the lookup with the current namespace's sym

Re: [PATCH] PR c/68473: sanitize source range-printing within certain macro expansions

2015-12-10 Thread Martin Sebor
On 11/23/2015 10:52 AM, David Malcolm wrote: This patch fixes PR c/68473 by bulletproofing the new diagnostic_show_locus implementation against ranges that finish before they start (which can happen when using the C preprocessor), falling back to simply printing a caret. Successfully bootstrappe

Re: [PATCH] update -Wall and -Wextra documentation

2015-12-10 Thread Martin Sebor
On 12/10/2015 04:57 AM, Bernd Schmidt wrote: On 12/10/2015 12:07 AM, Martin Sebor wrote: * invoke.texi (Warning Options): Update -Wall options. Clarify when some -Wextra options are enabled. Add -Wplacement-new example. I tried to check this list against c.opt - I figure this should

Re: ipa-cp heuristics fixes

2015-12-10 Thread Jan Hubicka
> Is this really necessary, is it not enough to remove the assignment to > ret below? If the parameter is not used, devirtualization time bonus, > which you then rely on estimate_local_effects, should be zero for it. > > It is a very minor point, I suppose, but if the function gets cloned > for a

Re: [PATCH] update -Wall and -Wextra documentation

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 05:52 PM, Martin Sebor wrote: The updated patch is attached. Ok. Bernd

[PATCH 2/2] [graphite] update required isl versions

2015-12-10 Thread Sebastian Pop
we now check the isl version, as there are no real differences in existing files in between isl 0.14 and isl 0.15. --- config/isl.m4 | 41 +++-- configure | 112 -- 2 files changed, 123 insertions(+), 30 deletions(-) dif

[PATCH 1/2] [graphite] document minimal required version for isl

2015-12-10 Thread Sebastian Pop
also update ISL to isl as requested by its author Sven Verdoolaege. --- gcc/doc/install.texi | 9 + gcc/doc/invoke.texi | 4 ++-- 2 files changed, 7 insertions(+), 6 deletions(-) diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi index 0b71bef..b43a3ec 100644 --- a/gcc/doc/install.

Re: [PATCH] testsuite/lib/multline.exp: show test name and line numbers

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 03:56 PM, David Malcolm wrote: The following patch updates multiline.exp to use the global $testname_with_flags as a prefix in such results. I also dropped the printing of the index in favor of printing the line numbers enclosed within dg-{begin|end}-multiline-output. After the

Re: [RFA] [PATCH] [PR tree-optimization/68619] Avoid direct cfg cleanups in tree-ssa-dom.c [4/3] v2

2015-12-10 Thread Uros Bizjak
> Finally the mechanical changes necessary due to the API change in the walker. You forgot to change the graphite part, as in the attached patch. 2015-12-10 Uros Bizjak PR tree-optimization/68619 * graphite-scop-detection.c (gather_bbs::before_dom_children): Change return type to

Re: [RFA] [PATCH] [PR tree-optimization/68619] Avoid direct cfg cleanups in tree-ssa-dom.c [4/3] v2

2015-12-10 Thread Jeff Law
On 12/10/2015 10:05 AM, Uros Bizjak wrote: Finally the mechanical changes necessary due to the API change in the walker. You forgot to change the graphite part, as in the attached patch. 2015-12-10 Uros Bizjak PR tree-optimization/68619 * graphite-scop-detection.c (gather_bbs::be

Re: Free up bits in DECLs and TYPEs

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 04:04 PM, Michael Matz wrote: This isn't stage 3 material really, OTOH fairly low risk. Anyway, okay for trunk now or once stage 1 opens? This is cool and we want it, but not now. Ok for stage 1, with the formatting problems quoted below fixed. Bernd +#define TYPE_ALIGN(NOD

Re: [v3] avoid alignment of static variables affecting stack's

2015-12-10 Thread Bernd Schmidt
On 12/10/2015 05:07 PM, Jan Beulich wrote: If not reaching if (TREE_CODE (origvar) == SSA_NAME) { gcc_assert (TREE_CODE (var) != VAR_DECL || (!DECL_EXTERNAL (var) && !DECL_HAS_VALUE_EXPR_P (var) && !TREE_STATIC (var)

[PATCH] gcc: read -fdebug-prefix-map OLD from environment (improved reproducibility)

2015-12-10 Thread Daniel Kahn Gillmor
Work on the reproducible-builds project [0] has identified that build paths are one cause of output variation between builds. This changeset allows users to avoid this variation when building C objects with debug symbols, while leaving the default behavior unchanged. Background -- gcc in

Re: [hsa 1/10] Configury changes and new options

2015-12-10 Thread Martin Jambor
Hi, On Tue, Dec 08, 2015 at 10:43:15PM +, Richard Sandiford wrote: > [Sorry for the low-quality review, was just reading out of interest...] > > Martin Jambor writes: > > +If you configure GCC with HSA offloading but do not have the HSA > > +run-time library installed in a standard location

Re: [hsa 1/10] Configury changes and new options

2015-12-10 Thread Martin Jambor
Hi, On Mon, Dec 07, 2015 at 12:19:08PM +0100, Martin Jambor wrote: > Hi, > > this patch contains changes to the configuration mechanism and offload > bits, so that users can build compilers with HSA support. when writing up how to build an HSA-enabled GCC for the wiki page, and checking the proc

Re: [hsa 2/10] Modifications to libgomp proper

2015-12-10 Thread Martin Jambor
Hi, thanks for the feedback. I have incorporated most of it into the branch (the diff is below) but also have a few questions. On Wed, Dec 09, 2015 at 12:35:36PM +0100, Jakub Jelinek wrote: > On Mon, Dec 07, 2015 at 12:19:57PM +0100, Martin Jambor wrote: > > +/* Flag set when the subsequent ele

Re: [hsa 0/10] Merge of HSA branch

2015-12-10 Thread Martin Jambor
Hi, On Mon, Dec 07, 2015 at 12:46:45PM +0100, Jakub Jelinek wrote: > On Mon, Dec 07, 2015 at 12:17:58PM +0100, Martin Jambor wrote: > > Because I have not been able to come up with any solution to failing > > libgomp/testsuite/libgomp.c++/target-2.C, I have disabled use of > > dynamic parallelism

Patch to fix PR68691

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

[google][gcc-4_9] update hardreg costs only when conflict_costs[] < 0

2015-12-10 Thread Wei Mi
Hi, The patch is for google branch only. In r216697, when a hardreg is assigned to an allocno, a positive cost will be added to those conflict allocnos to reflect the disfavor of the hardreg. However, the fact that conflict allocno disfavors a hard_regno doesn't necessarily mean current allocno

Re: [PATCH 4/4] Add -Wmisleading-indentation to -Wall

2015-12-10 Thread David Malcolm
On Thu, 2015-10-29 at 11:38 -0600, Jeff Law wrote: > On 10/29/2015 10:49 AM, David Malcolm wrote: > > Our documentation describes -Wall as enabling "all the warnings about > > constructions that some users consider questionable, and that are easy to > > avoid > > (or modify to prevent the warning)

Request permission to delete gcc.dg/vect/costmodel/ppc/costmodel-fast-math-vect-pr29925.c

2015-12-10 Thread Bill Schmidt
Hi, The subject test case has been failing as follows: FAIL: gcc.dg/vect/costmodel/ppc/costmodel-fast-math-vect-pr29925.c scan-tree-dump-times vect "vectorization not profitable" 1 The test has been failing since r223528, which is: 2015-05-22 Richard Biener PR tree-optimization/657

Re: Request permission to delete gcc.dg/vect/costmodel/ppc/costmodel-fast-math-vect-pr29925.c

2015-12-10 Thread David Edelsohn
On Thu, Dec 10, 2015 at 2:23 PM, Bill Schmidt wrote: > Hi, > > The subject test case has been failing as follows: > > FAIL: gcc.dg/vect/costmodel/ppc/costmodel-fast-math-vect-pr29925.c > scan-tree-dump-times vect "vectorization not profitable" 1 > > The test has been failing since r223528, which

Re: [testsuite][ARM target attributes] Fix effective_target tests

2015-12-10 Thread Christophe Lyon
On 10 December 2015 at 14:14, Kyrill Tkachov wrote: > > On 10/12/15 13:04, Christophe Lyon wrote: >> >> On 10 December 2015 at 13:30, Kyrill Tkachov >> wrote: >>> >>> Hi Christophe, >>> >>> >>> On 08/12/15 11:18, Christophe Lyon wrote: On 8 December 2015 at 11:50, Kyrill Tkachov w

Re: [Patch,tree-optimization]: Add new path Splitting pass on tree ssa representation

2015-12-10 Thread Jeff Law
On 12/03/2015 07:38 AM, Richard Biener wrote: This pass is now enabled by default with -Os but has no limits on the amount of stmts it copies. The more statements it copies, the more likely it is that the path spitting will turn out to be useful! It's counter-intuitive. The primary benefit AF

[PATCH] Fix ICE with -fno-if-conversion (PR rtl-optimization/68730)

2015-12-10 Thread Jakub Jelinek
Hi! It seems some passes in between the combiner and ira aren't prepared to update dominance info. It usually is not a problem, because already before the combiner we call free_dominance_info. But we now have a new i?86 stv pass that is injected after the combiner that computes dominators but do

Re: [Patch,tree-optimization]: Add new path Splitting pass on tree ssa representation

2015-12-10 Thread Jeff Law
On 12/03/2015 07:45 AM, Richard Biener wrote: Ah, some EEMBC one. Btw, the testcase that was added shows if (xc < xm) { xk = (unsigned char) (xc < xy ? xc : xy); } else { xk = (unsigned char) (xm < xy ? xm : xy); }

[PTX] reorder function calling hooks

2015-12-10 Thread Nathan Sidwell
This patch reorders the function calling hooks so they come before write_one_arg. There's no change to their functionality, but it will allow them to be called from write_one_arg and write_result, rather than have those functions continue to independently perform essentially the same calculatio

Re: [PATCH] Fix ICE with -fno-if-conversion (PR rtl-optimization/68730)

2015-12-10 Thread Jeff Law
On 12/10/2015 01:11 PM, Jakub Jelinek wrote: Hi! It seems some passes in between the combiner and ira aren't prepared to update dominance info. It usually is not a problem, because already before the combiner we call free_dominance_info. But we now have a new i?86 stv pass that is injected aft

[PATCH] Fix PR c++/21802 (two-stage name lookup fails for operators)

2015-12-10 Thread Patrick Palka
This patch fixes name-lookup of operators in template definitions whose operands are non-dependent expressions, i.e. PR c++/21802 (and incidentally 53223). The approach that this patch takes is to detect when build_new_op() returns a call to an overloaded function and to store a call to this overl

Re: [PATCH, testsuite] Fix PR68629: attr-simd-3.c failure on arm-none-eabi targets

2015-12-10 Thread Jeff Law
On 12/09/2015 02:56 AM, Thomas Preud'homme wrote: c-c++-common/attr-simd-3.c fails to compile on arm-none-eabi targets due to -fcilkplus needing -pthread which is not available for those targets. This patch solves this issue by adding a condition to the cilkplus effective target that compiling

  1   2   >