[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

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 jin...@google.com 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

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

[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 stdio.h include in GCC's tsystem.h. [ This is clearly a

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] 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

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

[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

[PATCH] Fix PR52349

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

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), it

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~

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 or the

[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 vmaka...@redhat.com * lra-constraints.c (narrow_reload_pseudo_class): New function.

[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 kti...@redhat.com * config/i386/i386.c (ix86_delegitimize_address): Handle

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 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 kti...@redhat.com * gcc.dg/pack-test-5.c: Add -mno-ms-bitfields option for mingw-targets. * gcc.dg/Wpadded.c: Likewise. *

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

2012-02-23 Thread Kai Tietz
2012/2/23 Jakub Jelinek ja...@redhat.com: 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 +        

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 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 ja...@redhat.com: 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    

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 kti...@redhat.com * config/i386/i386.c (ix86_delegitimize_address): Handle UNSPEC_PCREL plus displacement.

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 ktiet...@googlemail.com: 2012/2/23 Jakub Jelinek ja...@redhat.com: 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

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.

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 ja...@redhat.com: 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__

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.

[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

[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 al...@gcc.gnu.org * doc/passes.texi (Full redundancy elimination): Fix typo. Signed-off-by: Bernhard Reutner-Fischer rep.dot@gmail.com --- gcc/doc/passes.texi |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git

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

2012-02-23 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2012-02-23 Bernhard Reutner-Fischer al...@gcc.gnu.org * tree-if-conv (predicate_scalar_phi): Commentary typo fix. Signed-off-by: Bernhard Reutner-Fischer rep.dot@gmail.com --- gcc/tree-if-conv.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff

[PATCH 5/5] dump_file whitespace nitpicks

2012-02-23 Thread Bernhard Reutner-Fischer
gcc/ChangeLog: 2012-02-23 Bernhard Reutner-Fischer al...@gcc.gnu.org * tree-into-ssa (update_ssa): Avoid trailing whitespace in dump_file. * tree-ssa-sccvn.c (print_scc): Ditto. Signed-off-by: Bernhard Reutner-Fischer rep.dot@gmail.com --- gcc/tree-into-ssa.c |

[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 al...@gcc.gnu.org * doc/invoke.texi (-fdse, -fdce): Remove duplicate entries. Signed-off-by: Bernhard Reutner-Fischer rep.dot@gmail.com --- gcc/doc/invoke.texi |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff

[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 vmaka...@redhat.com * lra-eliminations.c (update_reg_eliminate): Make hard register unallocatable when

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)

[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

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

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 al...@gcc.gnu.org * doc/invoke.texi (-fdse, -fdce): Remove duplicate entries. OK (actually, you should commit this as obvious). Now we have the arrangements for docstring

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 al...@gcc.gnu.org * doc/passes.texi (Full redundancy elimination): Fix typo. Should be committed as obvious. -- Joseph S. Myers jos...@codesourcery.com

[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

[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 vmaka...@redhat.com *

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 patrick.marl...@gmail.com Jack Howarth howa...@bromo.med.uc.edu boehm-gc/ PR boehm-gc/52179 * include/gc_config.h.in:

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

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 jos...@codesourcery.com wrote: On Thu, 23 Feb 2012, Bernhard Reutner-Fischer wrote: gcc/ChangeLog 2012-02-23  Bernhard Reutner-Fischer  al...@gcc.gnu.org       * doc/passes.texi (Full redundancy elimination): Fix typo. Should be committed as

[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

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:

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

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

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

PATCH: PR target/52364: The unnecessary second form in *movabsmode_[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 *movabsmode_1 [(set (mem:SWI1248x (match_operand:DI 0

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

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 soni.sande...@gmail.com 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’: