Re: [gimplefe][patch] The symbol table for declarations

2012-02-23 Thread Sandeep Soni
On Wed, Nov 30, 2011 at 9:48 AM, Sandeep Soni wrote: > Sorry. Wrong patch. New patch attached. > > I am getting the following error in the new patch though. > > ../../gimple-front-end/gcc/gimple/parser.c: In function ‘gp_parse_var_decl’: > ../../gimple-front-end/gcc/gimple/parser.c:927:3: error: i

any chance of further wrong-code fixes for 4.6.3?

2012-02-23 Thread Kenny Simpson
issue w/ cmov+volatile (very small) PR rtl-optimization/47698 (2 line code move) PR target/45771 (small change to a conditional) PR c++/41449 PR middle-end/52314 (one expression change) thanks, -Kenny

PATCH: PR target/52364: The unnecessary second form in *movabs_[12]

2012-02-23 Thread H.J. Lu
Hi, The second form is redundant in ;; Stores and loads of ax to arbitrary constant address. ;; We fake an second form of instruction to force reload to load address ;; into register when rax is not available (define_insn "*movabs_1" [(set (mem:SWI1248x (match_operand:DI 0 "x86_64_movabs_operan

PR 52060 - fixed in 4.6.3?

2012-02-23 Thread Kenny Simpson
PR rtl-optimization/52060 is marked as fixed with a target of 4.6.4, but looks like its been backported to 4.6 a couple weeks ago. Should the target be adjusted so that it gets listed in the bugs fixed for 4.6.3? -Kenny

Re: v3 PATCH to include/bits/locale_facets.h to revert reordering of virtual functions

2012-02-23 Thread Benjamin Kosnik
Thanks Jason. -benjamin

v3 PATCH to include/bits/locale_facets.h to revert reordering of virtual functions

2012-02-23 Thread Jason Merrill
Benjamin's patch of 2011-08-06 (r177542) to clean up doxygen markup changed the order of the num_get::do_get virtual functions. This breaks ABI compatibility, so I'm reverting the change. Tested x86_64-pc-linux-gnu, applied to trunk. commit f8566d7934d5ab7006548fa67d596cac2c346d3e Author: Jaso

[pph] Write tree headers for mutated trees (issue5699055)

2012-02-23 Thread Lawrence Crowl
This patch writes the tree header for mutated trees. This write is necessary to get updated attribute information from the info bits. On read, these bits are then merged into the existing tree. Test x2incomplete4.cc is now generating assembly. However, it erroneously generates an inline default

Re: [PATCH 2/5] passes.texi: Fix typo in Full redundancy elimination

2012-02-23 Thread Bernhard Reutner-Fischer
On 23 February 2012 22:33, Joseph S. Myers wrote: > On Thu, 23 Feb 2012, Bernhard Reutner-Fischer wrote: > >> gcc/ChangeLog >> >> 2012-02-23  Bernhard Reutner-Fischer   >> >>       * doc/passes.texi (Full redundancy elimination): Fix typo. > > Should be committed as obvious. Presumably the same f

Fix PR bootstrap/52287

2012-02-23 Thread Eric Botcazou
This fixes the recent bootstrap failure of the mainline on SPARC/Solaris 8 by stabilizing the sort of the ready list of the Haïfa scheduler in the presence of debug insns. Bootstrapped/regtested on SPARC/Solaris 8 and x86-64/Linux, pre-approved by Bernd, applied on the mainline. 2012-02-23 E

Re: [PATCH]: Fix PR52179 and remove hack from PR48299

2012-02-23 Thread Mike Stump
On Feb 23, 2012, at 1:03 PM, Jack Howarth wrote: > Okay for gcc trunk for gcc 4.7 since this is target specific? Ok. > 2012-02-23 Patrick Marlier > Jack Howarth > > boehm-gc/ > > PR boehm-gc/52179 > * include/gc_config.h.in: Undefine HAVE_PTHREAD_GET_STACKADDR_NP. >

[lra] a path to fix GCC crash on s390

2012-02-23 Thread Vladimir Makarov
The following patch fixes a compiler crash on s390 during the bootstrap. Unfortunately it is not enough to fix the current s390 bootstrap failure. The patch was successfully bootstrapped on x86/x86-64. Committed as rev. 184524. 2012-02-23 Vladimir Makarov * lra-constraints.c (inh

[PATCH][ARM] core -> NEON extend

2012-02-23 Thread Andrew Stubbs
Hi All, This patch converts SImode to DImode extends that also move from core registers to VFP/NEON registers. Currently, the compiler does extends in core registers first, and then does the move. This adds to register pressure, which I would imagine to be a bad thing. If the value is not in

Re: [PATCH 2/5] passes.texi: Fix typo in Full redundancy elimination

2012-02-23 Thread Joseph S. Myers
On Thu, 23 Feb 2012, Bernhard Reutner-Fischer wrote: > gcc/ChangeLog > > 2012-02-23 Bernhard Reutner-Fischer > > * doc/passes.texi (Full redundancy elimination): Fix typo. Should be committed as obvious. -- Joseph S. Myers jos...@codesourcery.com

Re: [PATCH 1/5] invoke.texi: remove duplicate pass-flag entries

2012-02-23 Thread Joseph S. Myers
On Thu, 23 Feb 2012, Bernhard Reutner-Fischer wrote: > gcc/ChangeLog > > 2012-02-23 Bernhard Reutner-Fischer > > * doc/invoke.texi (-fdse, -fdce): Remove duplicate entries. OK (actually, you should commit this as obvious). Now we have the arrangements for docstring relicensing it may

Re: [PR51752] publication safety violations in loop invariant motion pass

2012-02-23 Thread Aldy Hernandez
On 02/23/12 12:19, Aldy Hernandez wrote: about hit me. Instead now I save all loads in a function and iterate through them in a brute force way. I'd like to rewrite this into a hash of some sort, but before I go any further I'm interested to know if the main idea is ok. For the record, it may

[PATCH]: Fix PR52179 and remove hack from PR48299

2012-02-23 Thread Jack Howarth
The attached patch implements the fix for supporting aslr on darwin11 and later which exists in current upstream boehm-gc, as discussed in https://github.com/ivmai/bdwgc/issues/13; see https://github.com/ivmai/bdwgc/commit/faef04e7cb3741163dfdf65900ef5d2a0530be0f. This change eliminates the tes

4.6 branch now frozen

2012-02-23 Thread Jakub Jelinek
Hi! 4.6.3-rc1 snapshot is being built right now, the 4.6 branch is from now on frozen, all commits require an extra ack from one of the RMs. I hope we can release 4.6.3 late next week. Jakub

Re: [PATCH][ARM] 64-bit shifts in NEON.

2012-02-23 Thread Andrew Stubbs
On 21/02/12 15:23, Andrew Stubbs wrote: On 06/02/12 13:13, Andrew Stubbs wrote: This patch adds DImode shift support in NEON registers/instructions. The patch causes delays any lowering until the split2 pass, after the register allocator has chosen whether to do the shift in NEON (VFP) register

[lra] a patch to fix a testsuite regression on SPARC.

2012-02-23 Thread Vladimir Makarov
The following patch fixes a few SPARC GCC testsuite regressions. The patch was successfully bootstrapped on x86/x86-64. Committed as rev. 184512. 2012-02-23 Vladimir Makarov * lra-eliminations.c (update_reg_eliminate): Make hard register unallocatable when we make it a result

Backport a patch to 4.6 branch

2012-02-23 Thread Jakub Jelinek
Hi! I've backported this to 4.6 branch, after bootstrapping/regtesting it on x86_64-linux and i686-linux. 2012-02-23 Jakub Jelinek Backported from trunk 2012-02-20 Georg-Johann Lay * gcc.c-torture/execute/pr52286.c: Fix FAIL on 16-bit int platforms. 2012-0

Re: [PATCH] Fix PR52298

2012-02-23 Thread Ulrich Weigand
Richard Guenther wrote: > PR tree-optimization/52298 > * tree-vect-stmts.c (vectorizable_store): Properly use > STMT_VINFO_DR_STEP instead of DR_STEP when vectorizing > outer loops. > (vectorizable_load): Likewise. > * tree-vect-data-refs.c (vect_analyze_data_re

[PATCH 0/5] misc janitorial tweaks

2012-02-23 Thread Bernhard Reutner-Fischer
Hi, A few tweaks i found i an old local branch. Ok for trunk? The duplicate pass-flag entries in invoke.texi is something that should have been warned about in one of the generator-scripts, ideally. Perhaps we should remember this as a feature bug.. Bernhard Reutner-Fischer (5): invoke.texi: r

[PATCH 1/5] invoke.texi: remove duplicate pass-flag entries

2012-02-23 Thread Bernhard Reutner-Fischer
gcc/ChangeLog 2012-02-23 Bernhard Reutner-Fischer * doc/invoke.texi (-fdse, -fdce): Remove duplicate entries. Signed-off-by: Bernhard Reutner-Fischer --- gcc/doc/invoke.texi |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/gcc/doc/invoke.texi b/gcc/doc/inv

[PATCH 4/5] make_phi_node can be static

2012-02-23 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2012-02-23 Bernhard Reutner-Fischer * tree-phinodes.c (make_phi_node): Mark static. * tree-flow.h (make_phi_node): Remove extern decl. * doc/gimple.texi (make_phi_node): Remove documentation. Signed-off-by: Bernhard Reutner-Fischer --- gcc/doc/gimple.t

[PATCH 5/5] dump_file whitespace nitpicks

2012-02-23 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2012-02-23 Bernhard Reutner-Fischer * tree-into-ssa (update_ssa): Avoid trailing whitespace in dump_file. * tree-ssa-sccvn.c (print_scc): Ditto. Signed-off-by: Bernhard Reutner-Fischer --- gcc/tree-into-ssa.c |4 ++-- gcc/tree-ssa-sccvn.c |4 +

[PATCH 3/5] tree-if-conv: Commentary typo fix

2012-02-23 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2012-02-23 Bernhard Reutner-Fischer * tree-if-conv (predicate_scalar_phi): Commentary typo fix. Signed-off-by: Bernhard Reutner-Fischer --- gcc/tree-if-conv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/gcc/tree-if-conv.c b/gcc/tree-if-co

[PATCH 2/5] passes.texi: Fix typo in Full redundancy elimination

2012-02-23 Thread Bernhard Reutner-Fischer
gcc/ChangeLog 2012-02-23 Bernhard Reutner-Fischer * doc/passes.texi (Full redundancy elimination): Fix typo. Signed-off-by: Bernhard Reutner-Fischer --- gcc/doc/passes.texi |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/gcc/doc/passes.texi b/gcc/doc/passes

[PR51752] publication safety violations in loop invariant motion pass

2012-02-23 Thread Aldy Hernandez
In this PR we have a publication safety violation in a transaction. The loop invariant motion pass hoists a load out of a loop, creating a load data race. Those unfamiliar with load data races in transactions, please see the PR, as this has been confusing to most (me included). In the snippe

Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets

2012-02-23 Thread Mike Stump
On Feb 23, 2012, at 9:25 AM, Jakub Jelinek wrote: > Anyway, will defer the rest to the testsuite maintainers. Ok.

Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets

2012-02-23 Thread Kai Tietz
2012/2/23 Jakub Jelinek : > On Thu, Feb 23, 2012 at 06:23:39PM +0100, Kai Tietz wrote: >> And the underlying issue is that __extension__ can't be use before >> __SIZE_TYPE__ here. gcc lacks to parse that proper.  You would get > > Then please use > __extension__ typedef __SIZE_TYPE__ size_t; > and

Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets

2012-02-23 Thread Jakub Jelinek
On Thu, Feb 23, 2012 at 06:23:39PM +0100, Kai Tietz wrote: > And the underlying issue is that __extension__ can't be use before > __SIZE_TYPE__ here. gcc lacks to parse that proper. You would get Then please use __extension__ typedef __SIZE_TYPE__ size_t; and use size_t in the prototype. __exten

Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets

2012-02-23 Thread Kai Tietz
2012/2/23 Kai Tietz : > 2012/2/23 Jakub Jelinek : >> Why?  I don't see extensions anywhere.  Or is __SIZE_TYPE__ on mingw >> long long and requires __extension__?  If yes, it should at least go >> right before the argument, but perhaps then it should be already >> in the __SIZE_TYPE__ macro. > > Th

Re: [patch i386]: Add support of delegitimize of UNSPEC_PCREL plus displacement

2012-02-23 Thread Kai Tietz
Hi, So tests are complete (with full test on x86_64-unknown-linux-gnu and testfailure testing on i686-w64-mingw32 and x86_64-w64-mingw32). ChangeLog 2012-02-23 Kai Tietz * config/i386/i386.c (ix86_delegitimize_address): Handle UNSPEC_PCREL plus displacement. Regression teste

Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets

2012-02-23 Thread Kai Tietz
2012/2/23 Jakub Jelinek : > On Thu, Feb 23, 2012 at 05:56:11PM +0100, Kai Tietz wrote: >> --- gcc.dg/pack-test-5.c      (revision 184486) >> +++ gcc.dg/pack-test-5.c      (working copy) >> @@ -1,6 +1,7 @@ >>  /* PR c/11446: packed on a struct takes precedence over aligned on the type >>     of a fi

Re: [patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets

2012-02-23 Thread Jakub Jelinek
On Thu, Feb 23, 2012 at 05:56:11PM +0100, Kai Tietz wrote: > --- gcc.dg/pack-test-5.c (revision 184486) > +++ gcc.dg/pack-test-5.c (working copy) > @@ -1,6 +1,7 @@ > /* PR c/11446: packed on a struct takes precedence over aligned on the type > of a field. */ > /* { dg-do run } */ >

Re: [patch i386]: Add support of delegitimize of UNSPEC_PCREL plus displacement

2012-02-23 Thread Kai Tietz
2012/2/23 Jakub Jelinek : > On Thu, Feb 23, 2012 at 05:38:37PM +0100, Kai Tietz wrote: >> --- config/i386/i386.c        (revision 184486) >> +++ config/i386/i386.c        (working copy) >> @@ -13241,6 +13241,22 @@ >> >>    if (TARGET_64BIT) >>      { >> +      if (GET_CODE (x) == CONST >> +        

[patch testsuite]: Adjust some tests of gcc.dg and gcc.c-torture for mingw targets

2012-02-23 Thread Kai Tietz
Hello, this patch corrects some testcases in gcc.dg and gcc.c-torture for x64 and x86 Windows targets. ChangeLog 2012-02-23 Kai Tietz * gcc.dg/pack-test-5.c: Add -mno-ms-bitfields option for mingw-targets. * gcc.dg/Wpadded.c: Likewise. * gcc.dg/bf-ms-layout-2.

Re: [patch i386]: Add support of delegitimize of UNSPEC_PCREL plus displacement

2012-02-23 Thread Jakub Jelinek
On Thu, Feb 23, 2012 at 05:38:37PM +0100, Kai Tietz wrote: > --- config/i386/i386.c(revision 184486) > +++ config/i386/i386.c(working copy) > @@ -13241,6 +13241,22 @@ > >if (TARGET_64BIT) > { > + if (GET_CODE (x) == CONST > + && GET_CODE (XEXP (x, 0)) == PLUS

[patch i386]: Add support of delegitimize of UNSPEC_PCREL plus displacement

2012-02-23 Thread Kai Tietz
Hi, This patch adds a missing nit about delegitimize of CONST (UNSPEC_PCREL + displacement). Some testcases - like gcc.c-torture/execute/ 930930-1.c - were failing due this. ChangeLog 2012-02-23 Kai Tietz * config/i386/i386.c (ix86_delegitimize_address): Handle UNSPEC_PCREL p

[lra] patch to fix SPARC bootstrap failure

2012-02-23 Thread Vladimir Makarov
The following patch fixes SPARC64 bootstrap failure finally. The patch was successfully bootstrapped on x86/x86-64, SPARC64, and ARM. Committed as rev. 184510. 2012-02-22 Vladimir Makarov * lra-constraints.c (narrow_reload_pseudo_class): New function. (match_reload): Call it.

Re: [PATCH, c]: Fix PR52290, [4.4/4.5/4.6/4.7 Regression] internal compiler error: tree check: expected function_decl, have var_decl in start_function, at c-decl.c:7712

2012-02-23 Thread Joseph S. Myers
On Thu, 23 Feb 2012, Uros Bizjak wrote: > 2012-02-23 Uros Bizjak > > * c-decl.c (start_function): Exit early if decl1 is not FUNTION_DECL. > > testsuite/ChangeLog: > > 2012-02-23 Uros Bizjak > > * gcc.dg/noncompile/pr52290.c: New test. > > Patch was bootstrapped and regressi

Re: [RFC, 4.8] Magic matching for flags clobbering and setting

2012-02-23 Thread Richard Henderson
On 02/22/12 17:16, Hans-Peter Nilsson wrote: >> What I know is missing off the top of my head are: > >> (2) Can't be usefully used with define_insn_and_split, and no way to tell. >> This problem should simply be documented in the .texi file as user >> error. > > Not sure I see the problem

Re: libitm: Use ml_wt as default TM methods for >1 thread.

2012-02-23 Thread Richard Henderson
On 02/23/12 05:34, Torvald Riegel wrote: > libitm: Use ml_wt as default TM methods for >1 thread. > > libitm/ > * retry.cc (GTM::gtm_thread::number_of_threads_changed): Change > default dispatch for more than 1 thread to ml_wt. Ok. r~

[PATCH][RFC] Preserve loop info from tree loop opts to after RTL loop opts (PR44688)

2012-02-23 Thread Richard Guenther
The attached patch blob makes us preserve loop information (the loop tree) from the start of tree loop optimizations until the end of RTL loop optimizations. The motivation for this is to fix excessive prefetching and loop unrolling we perform on (for example) prologue loops created by the vector

Re: Simulator testing for sh and sh64

2012-02-23 Thread Kaz Kojima
[I've moved to gcc-patches because this includes a patch anyway.] Thomas Schwinge wrote: > /scratch/tschwing/FM_sh64-elf/src/gcc-mainline/libgcc/libgcc2.c: In > function '__powisf2': > /scratch/tschwing/FM_sh64-elf/src/gcc-mainline/libgcc/libgcc2.c:1779:1: > error: unrecognizable insn:

Re: [Patch,AVR]: Tweak xor similar to ior

2012-02-23 Thread Denis Chertykov
2012/2/23 Georg-Johann Lay : > This mini-patch extends some combine patterns that used IOR to also match > for XOR in a similar way. > > Bottom line is always that when performing IOR or XOR, only portions > in non-zero bytes matter and zero-bytes that come from zero-extends > or shifts need not to

Re: [PATCH] Ignore CLOBBER stmts in ipa-split (PR tree-optimization/52019)

2012-02-23 Thread Richard Guenther
On Thu, 23 Feb 2012, Jakub Jelinek wrote: > Hi! > > IMHO we should treat CLOBBER stmts in various places like debug stmts, > they shouldn't affect the decisions of the optimization phases, but > if the pass does some transformation that somehow invalidates them or > moves them (not in this pass),

[PATCH] Fix PR52349

2012-02-23 Thread Richard Guenther
Approved in the PR, tested by Jakub, applied. Richard. 2012-02-23 Richard Guenther go/ * go-gcc.cc (Gcc_backend::placeholder_pointer_type): Use build_distinct_type_copy. Index: go-gcc.cc === --- go-gcc.c

[PATCH] Ignore CLOBBER stmts in ipa-split (PR tree-optimization/52019)

2012-02-23 Thread Jakub Jelinek
Hi! IMHO we should treat CLOBBER stmts in various places like debug stmts, they shouldn't affect the decisions of the optimization phases, but if the pass does some transformation that somehow invalidates them or moves them (not in this pass), it needs to be adjusted. Otherwise we end up with the

libitm: Use ml_wt as default TM methods for >1 thread.

2012-02-23 Thread Torvald Riegel
This patch changes the default TM method that is used if more than one thread is registered to ml_wt (from serialirr_onwrite previously). For one registered thread, it's still serialirr. Right now, this will not be a performance advantage for every workload and number of threads. However, of the

[Patch,AVR]: Tweak xor similar to ior

2012-02-23 Thread Georg-Johann Lay
This mini-patch extends some combine patterns that used IOR to also match for XOR in a similar way. Bottom line is always that when performing IOR or XOR, only portions in non-zero bytes matter and zero-bytes that come from zero-extends or shifts need not to be XORed/IORed. The patch just replace

[PATCH] PR libffi/52223: Define FLAGS_TO_PASS for libffi

2012-02-23 Thread Mikael Pettersson
This fixes a "make prefix=... mandir=... install" installation error in libffi on multilib targets like {x86_64,m68k}-linux: make prefix=/tmp/buildroot/usr mandir=/tmp/buildroot/usr/share/man install ... Making install in man make[5]: Entering directory `/tmp/objdir/x86_64-unknown-linux-gnu/32/lib

Re: [PATCH] Re: Serious regressions due to newlib's HAVE_INITFINI_ARRAY

2012-02-23 Thread Jakub Jelinek
On Thu, Feb 23, 2012 at 11:03:47AM +0100, Ulrich Weigand wrote: > OK for mainline? Yes, thanks. Jakub

[PATCH] Re: Serious regressions due to newlib's HAVE_INITFINI_ARRAY

2012-02-23 Thread Ulrich Weigand
Jakub Jelinek wrote: > On Wed, Feb 22, 2012 at 03:55:34PM +0100, Ulrich Weigand wrote: > > However, the macro HAVE_INITFINI_ARRAY is defined anyway; this > > definition is done by an internal "newlib.h" header that is pulled > > in via the include in GCC's "tsystem.h". [ This is clearly > > a vio

RE: [Ping] RE: CR16 Port addition

2012-02-23 Thread Gerald Pfeifer
On Mon, 20 Feb 2012, Jayant R. Sonar wrote: > PFA, the patch modified as per your suggestion. This is cool, please go ahead. (It would have been fine to commit this based on my previous mail. :-) > I have also attached here another patch for contrib.texi file changes. Very well. +Sumanth Gunda

Re: [PATCH, i386, Android] Enable exceptions and RTTI by default for Android

2012-02-23 Thread Richard Guenther
On Wed, Feb 22, 2012 at 8:20 PM, Jing Yu wrote: > So far, Android ARM toolchain, which builds Android platform for ARM > boards, does not enable RTTI and exceptions by default. There are > license concerns with the use of GNU libstdc++ and libsupc++. That, of course, does not answer my question w

[PATCH, c]: Fix PR52290, [4.4/4.5/4.6/4.7 Regression] internal compiler error: tree check: expected function_decl, have var_decl in start_function, at c-decl.c:7712

2012-02-23 Thread Uros Bizjak
Hello! With invalid code, we can trick grokdeclarator to return VAR_DECL, even when FUNCDEF context is requested. Attached one-liner detects this situation and exits early from start_function. The new error stream looks correct to me, with following invalid testcase we get: $ cat pr52290.c int f(