Re: [GSoC] generation of Gimple code from isl_ast_node_if

2014-07-27 Thread Roman Gareev
Can you explain why you believe it is hard/impossible to generate a test case without reduction? I don't believe this. I only know that all the test cases considered by me have the same problem. Could you please explain what is 'reduction'? I've found out that, according to the comment of the

Re: [GSoC] generation of Gimple code from isl_ast_node_if

2014-07-27 Thread Tobias Grosser
On 27/07/2014 08:12, Roman Gareev wrote: Can you explain why you believe it is hard/impossible to generate a test case without reduction? I don't believe this. I only know that all the test cases considered by me have the same problem. Could you please explain what is 'reduction'? I've found

[Patch] PR 61692 - Fix for inline asm ICE

2014-07-27 Thread David Wohlferd
I'm not sure which maintainer to cc for inline asm stuff? I have a release on file with the FSF, but don't have SVN write access. Problem: extract_insn() in recog.c will ICE if (noperands MAX_RECOG_OPERANDS). Normally this isn't a problem since expand_asm_operands() in cfgexpand.c catches

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Alan Modra
On Sat, Jul 26, 2014 at 01:45:12PM +0200, Matthias Klose wrote: Am 17.07.2014 02:41, schrieb Ulrich Weigand: Hello, this is the variant intended for the 4.8/4.9 branches of the patch: https://gcc.gnu.org/ml/gcc-patches/2014-07/msg01072.html As discussed, it does *not* actually

[MIPS, committed] Reverse argument order in const-anchor tests

2014-07-27 Thread Richard Sandiford
gcc.target/mips/const-anchor-[12].c started failing after: 2014-04-29 James Greenhalgh james.greenha...@arm.com * calls.c (initialize_argument_information): Always treat PUSH_ARGS_REVERSED as 1, simplify code accordingly. (expand_call): Likewise.

Re: [GSoC] generation of Gimple code from isl_ast_node_if

2014-07-27 Thread Roman Gareev
Thank you! I've attached patches with a test case (it is located in patch1.txt), which generates the following ISL AST: for (int c1 = 0; c1 = 49; c1 += 1) { if (c1 = 24) S_4(c1); S_5(c1); } I think that it doesn't contain reduction and doesn't yield several bbs. I've also checked that

Re: [GSoC] generation of Gimple code from isl_ast_node_if

2014-07-27 Thread Tobias Grosser
On 27/07/2014 12:48, Roman Gareev wrote: Thank you! I've attached patches with a test case (it is located in patch1.txt), which generates the following ISL AST: for (int c1 = 0; c1 = 49; c1 += 1) { if (c1 = 24) S_4(c1); S_5(c1); } I think that it doesn't contain reduction and

[PATCH] Fix ICE with -Wodr (PR middle-end/61913)

2014-07-27 Thread Marek Polacek
Wodr in common.opt was missing a Var, which means: 1) we ICE with -Wodr, since -Wodr isn't handled in opts.c; 2) -Wno-odr wouldn't work. Thus fixed. I'd think this doesn't need a testcase... Bootstrapped/regtested on x86_64-linux, ok for trunk? 2014-07-27 Marek Polacek pola...@redhat.com

Re: update address taken: don't drop clobbers

2014-07-27 Thread Marc Glisse
On Thu, 10 Jul 2014, Richard Biener wrote: --- gcc/tree-into-ssa.c (revision 212109) +++ gcc/tree-into-ssa.c (working copy) @@ -1831,26 +1831,38 @@ maybe_register_def (def_operand_p def_p, { tree def = DEF_FROM_PTR (def_p); tree sym = DECL_P (def) ? def : SSA_NAME_VAR (def); /* If

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Alan Modra
On Sun, Jul 27, 2014 at 07:16:07PM +0930, Alan Modra wrote: On Sat, Jul 26, 2014 at 01:45:12PM +0200, Matthias Klose wrote: Am 17.07.2014 02:41, schrieb Ulrich Weigand: Hello, this is the variant intended for the 4.8/4.9 branches of the patch:

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread pinskia
On Jul 27, 2014, at 4:53 AM, Alan Modra amo...@gmail.com wrote: On Sun, Jul 27, 2014 at 07:16:07PM +0930, Alan Modra wrote: On Sat, Jul 26, 2014 at 01:45:12PM +0200, Matthias Klose wrote: Am 17.07.2014 02:41, schrieb Ulrich Weigand: Hello, this is the variant intended for the 4.8/4.9

[PATH wwwdocs] Mention avx-512vlbwdq branch in svn.html

2014-07-27 Thread Kirill Yukhin
Hello, Patch below introduces mention of avx-512vlbwdq SVN branch in htdocs/svn.html Same prefix for e-mail (w/ avx-512) put intentionally. Is it ok to install? -- Thanks, K === RCS file: /cvs/gcc/wwwdocs/htdocs/svn.html,v

[PATCH i386 AVX512] [1/n] Introduce `-mavx512dq' switch

2014-07-27 Thread Kirill Yukhin
Hello, With this patch we'd like to start merge process of avx-512vlbwdq branch into main trunk. This patch introduces new switch `-mavx512dq' Bootstrapped. Is it ok for trunk? * common/config/i386/i386-common.c (OPTION_MASK_ISA_AVX512DQ_SET): Define.

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Matthias Klose
Am 27.07.2014 13:59, schrieb pins...@gmail.com: On Jul 27, 2014, at 4:53 AM, Alan Modra amo...@gmail.com wrote: On Sun, Jul 27, 2014 at 07:16:07PM +0930, Alan Modra wrote: On Sat, Jul 26, 2014 at 01:45:12PM +0200, Matthias Klose wrote: Am 17.07.2014 02:41, schrieb Ulrich Weigand: Hello,

[Patch, moxie] Add moxiebox target

2014-07-27 Thread Anthony Green
The following binutils patches introduce a new moxie-based target called moxiebox. Moxiebox is a VM developed by the bitcoin community to act as a secure, sandboxed execution environment for bitcoin automation. See http://github.com/jgarzik/moxiebox for more information. I'm checking these

[Patch, Fortran] Add first coarray ABI documentation to gfortran.texi

2014-07-27 Thread Tobias Burnus
Dear all, attached is a first patch to gfortran.texi which add documentation about libcaf*.c. The things are still a bit in a flux and it is incomplete (atomics, locking, error stop and vector subscripts are still missing), but one has to start somewhere … OK for the trunk? Comments on the

Re: Patch for constexpr variable templates

2014-07-27 Thread Adam Butcher
On 2014-07-26 17:14, Jason Merrill wrote: On 07/26/2014 12:11 PM, Jason Merrill wrote: On 07/26/2014 03:04 AM, Braden Obrzut wrote: On 07/25/2014 05:24 PM, Jason Merrill wrote: Fair enough, but in that case let's use 'sorry' rather then 'error' to be clear that it's a missing feature.

Re: update address taken: don't drop clobbers

2014-07-27 Thread Marc Glisse
On Thu, 10 Jul 2014, Richard Biener wrote: --- gcc/tree-into-ssa.c (revision 212109) +++ gcc/tree-into-ssa.c (working copy) @@ -1831,26 +1831,38 @@ maybe_register_def (def_operand_p def_p, { tree def = DEF_FROM_PTR (def_p); tree sym = DECL_P (def) ? def : SSA_NAME_VAR (def); /* If

PR61919: Invalid rtx sharing in tree-outof-ssa.c

2014-07-27 Thread Richard Sandiford
PR 61919 is another ripple from the patch to take advantage of rtx sharing rules when instantiating virtual registers. In this case the invalid sharing is coming from tree-outof-ssa.c, where the same MEM rtx is being used in several moves. (Note that despite the name, partition_to_pseudo maps to

Re: Patch for constexpr variable templates

2014-07-27 Thread Andrew Sutton
In the 59638 case, the declarations void (*a)(auto); void (*b)(auto) = 0; are shorthand for template typename T void (*a)(T); template typename T void (*b)(T) = 0; which, unless there's some constraint with variable templates that I'm not aware of, ought to define two variable

[committed] Remove my MIPS maintainer entry

2014-07-27 Thread Richard Sandiford
as per https://gcc.gnu.org/ml/gcc/2014-07/msg00231.html Richard * MAINTAINERS: Remove my MIPS maintainer entry. Index: MAINTAINERS === --- MAINTAINERS 2014-07-24 16:13:22.686714267 +0100 +++ MAINTAINERS 2014-07-27

Re: Warn when returning the address of a temporary (middle-end) v2

2014-07-27 Thread Richard Sandiford
Marc Glisse marc.gli...@inria.fr writes: Hello, I followed the advice in this discussion: https://gcc.gnu.org/ml/gcc-patches/2014-04/msg00269.html and here is a new patch. I made an effort to isolate a path in at least one subcase so it doesn't look too strange that the warning is in this

Re: [PATH wwwdocs] Mention avx-512vlbwdq branch in svn.html

2014-07-27 Thread Gerald Pfeifer
Hi Kirill, On Sun, 27 Jul 2014, Kirill Yukhin wrote: Patch below introduces mention of avx-512vlbwdq SVN branch in htdocs/svn.html Same prefix for e-mail (w/ avx-512) put intentionally. Is it ok to install? you don't need to get explicit approval for release notes related to your areas

Re: Warn when returning the address of a temporary (middle-end) v2

2014-07-27 Thread Marc Glisse
On Sun, 27 Jul 2014, Richard Sandiford wrote: Marc Glisse marc.gli...@inria.fr writes: Hello, I followed the advice in this discussion: https://gcc.gnu.org/ml/gcc-patches/2014-04/msg00269.html and here is a new patch. I made an effort to isolate a path in at least one subcase so it doesn't

Re: Patch for constexpr variable templates

2014-07-27 Thread Adam Butcher
On 2014-07-27 19:01, Andrew Sutton wrote: In the 59638 case, the declarations void (*a)(auto); void (*b)(auto) = 0; are shorthand for template typename T void (*a)(T); template typename T void (*b)(T) = 0; which, unless there's some constraint with variable templates that I'm not

Convert more incremental hash users to inchash

2014-07-27 Thread Andi Kleen
This patchkit converts more incremental hash users to the new inchash class. The only larger change is for rtl hashing, which I had to move to a new file to avoid problems with the generator program. All changes should only minimally change behavior. Bootstrapped and tested on x86_64-linux. Ok

[PATCH 2/6] Convert asan.c to inchash

2014-07-27 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com gcc/: 2014-07-25 Andi Kleen a...@linux.intel.com * asan.c (asan_mem_ref_hasher::hash): Convert to inchash. --- gcc/asan.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/gcc/asan.c b/gcc/asan.c index 475dd82..f7fa55f

[PATCH 3/6] Convert ipa-devirt to inchash

2014-07-27 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com gcc/: 2014-07-25 Andi Kleen a...@linux.intel.com * ipa-devirt.c (polymorphic_call_target_hasher::hash): Convert to inchash. --- gcc/ipa-devirt.c | 20 +--- 1 file changed, 9 insertions(+), 11 deletions(-) diff --git

[PATCH 4/6] Convert tree-ssa-dom to inchash

2014-07-27 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com gcc/: 2014-07-25 Andi Kleen a...@linux.intel.com * tree-ssa-dom.c (iterative_hash_exprs_commutative): Convert to inchash. (iterative_hash_hashable_expr): Dito. (avail_expr_hash): Dito. --- gcc/tree-ssa-dom.c | 79

[PATCH 6/6] Convert tree-ssa-tail-merge to inchash

2014-07-27 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com gcc/: 2014-07-25 Andi Kleen a...@linux.intel.com * tree-ssa-tail-merge.c (same_succ_hash): Convert to inchash. --- gcc/tree-ssa-tail-merge.c | 22 ++ 1 file changed, 10 insertions(+), 12 deletions(-) diff --git

[PATCH 5/6] Convert tree-ssa-sccvn to inchash

2014-07-27 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com gcc/: 2014-07-25 Andi Kleen a...@linux.intel.com * tree-ssa-sccvn.c (vn_reference_op_compute_hash): (vn_reference_compute_hash): (vn_nary_op_compute_hash): (vn_phi_compute_hash): * tree-ssa-sccvn.h

[PATCH 1/6] RTL dwarf2out changes

2014-07-27 Thread Andi Kleen
From: Andi Kleen a...@linux.intel.com Convert dwarf2out and rtl.c to the new inchash interface. I moved the rtl hash code to another file to avoid having to link all the hash code into the generator functions. gcc/: 2014-07-25 Andi Kleen a...@linux.intel.com * Makefile.in (OBJS):

Re: Warn when returning the address of a temporary (middle-end) v2

2014-07-27 Thread Andreas Schwab
Marc Glisse marc.gli...@inria.fr writes: On Sun, 27 Jul 2014, Richard Sandiford wrote: Marc Glisse marc.gli...@inria.fr writes: + if (always_executed) + msg = function returns address of local variable; + else + msg =

[GSoC][match-and-simplify] sanitize option checking

2014-07-27 Thread Prathamesh Kulkarni
Added checks to see if either cmmand-line options is not repeated, and generates match-and-simplify code on both GENERIC and GIMPLE if both -generic and -gimple are specified. * genmatch.c (cmd_options): New struct. (check_repeated_arg): New function. (parse_cmd_arg): Likewise.

Re: Warn when returning the address of a temporary (middle-end) v2

2014-07-27 Thread Marc Glisse
On Sun, 27 Jul 2014, Andreas Schwab wrote: Marc Glisse marc.gli...@inria.fr writes: On Sun, 27 Jul 2014, Richard Sandiford wrote: Marc Glisse marc.gli...@inria.fr writes: + if (always_executed) + msg = function returns address of local variable; +

[GSoC][match-and-simplify] include is-a.h

2014-07-27 Thread Prathamesh Kulkarni
Is it okay to include is-a.h ? I have adjusted print_operand to use is_a and as_a in this patch. * genmatch.c (is-a.h): Include. (is_a_helper::test): Specialize for operand subclasses. (print_operand): Adjust to use is_a and as_a. Thanks, Prathamesh. Index: genmatch.c

Aw: RE: Re: [MIPS r5900] libgcc floating point fixes

2014-07-27 Thread Jürgen Urban
Jürgen Urban juergenur...@gmx.dewrites: Jürgen Urban juergenur...@gmx.de writes: Hello Richard, Jürgen Urban juergenur...@gmx.de writes: Is this something you have recently developed outside of the mainline kernel or does it already exist. I'm not aware of such logic in

Re: [PATCH, rs6000, 4.8/4.9] Fix alignment of non-Altivec vector struct fields

2014-07-27 Thread Alan Modra
On Sun, Jul 27, 2014 at 03:16:03PM +0200, Matthias Klose wrote: Blah, that won't work of course. The macro needs to take two parameters. #define rs6000_special_adjust_field_align_p(FIELD, COMPUTED) false This is pre-approved if it works. I really should finish off the branch I

Re: Patch for constexpr variable templates

2014-07-27 Thread Braden Obrzut
So given this, should I leave the test cases that fail for this reason alone or should I still change them to dg-message? It sounds like GCC's behavior with auto in function parameters needs to be changed, but that definitely sounds like a separate patch to me. - Braden Obrzut