Re: [PATCH] MIPS16 TLS support for GCC

2012-07-05 Thread Chung-Lin Tang
Hi Richard, picking up a yet uncommitted part of the MIPS16 changes, see below: On 2012/2/3 11:28 PM, Richard Sandiford wrote: Chung-Lin Tang clt...@codesourcery.com writes: (2) is interesting if there is also a way to build those MIPS16 libraries out of the box. I'd like such a mechanism to

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-05 Thread Tristan Gingold
On Jul 4, 2012, at 8:31 PM, Magnus Fromreide wrote: Hello. Ever since the ISL patch went in my builds have failed. Which ISL patch ? I am building with local copies of all the libraries, so I have added gmp, mpfr, mpc, isl and cloog from ftp://gcc.gnu.org/pub/gcc/infrastructure to my

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-05 Thread Richard Guenther
On Thu, 5 Jul 2012, Tristan Gingold wrote: On Jul 4, 2012, at 8:31 PM, Magnus Fromreide wrote: Hello. Ever since the ISL patch went in my builds have failed. Which ISL patch ? I am building with local copies of all the libraries, so I have added gmp, mpfr, mpc, isl and cloog

Re: [ARM Patch 2/3]PR53189: optimizations of 64bit logic operation with constant

2012-07-05 Thread Carrot Wei
Hi I updated this patch to use const_ok_for_dimode_op to decide if a const is legal for xordi3 insn. The special cases of all 0 and all 1 are also handled. Tested on arm qemu with all thumb/arm neon/non-neon combination. The testcase save-temps-1.c failed in arm/non-neon mode, it should be

Re: [RFA/ARM 1/3] Add VFP support for VFMA and friends

2012-07-05 Thread Matthew Gretton-Dann
On 26/06/12 14:44, Richard Earnshaw wrote: On 25/06/12 15:59, Matthew Gretton-Dann wrote: All, This patch adds support to the ARM backend for generating floating-point fused multiply-accumulate. OK? gcc/ChangeLog: 2012-06-25 Matthew Gretton-Dann matthew.gretton-d...@arm.com *

Re: [patch] Add block debug info to phi_arg_d

2012-07-05 Thread Steven Bosscher
On Thu, Jul 5, 2012 at 11:23 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jul 5, 2012 at 11:08 AM, Dehao Chen de...@google.com wrote: Hi, This patch added block field to phi_arg_d to make sure the associated source locus is consistent with its block info. Bootstrapped and

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-05 Thread Richard Guenther
On Thu, 5 Jul 2012, Richard Guenther wrote: On Thu, 5 Jul 2012, Tristan Gingold wrote: On Jul 4, 2012, at 8:31 PM, Magnus Fromreide wrote: Hello. Ever since the ISL patch went in my builds have failed. Which ISL patch ? I am building with local copies of all the

Re: [PATCH] Move Graphite from using PPL over to ISL

2012-07-05 Thread Richard Guenther
On Thu, 5 Jul 2012, Richard Guenther wrote: On Thu, 5 Jul 2012, Richard Guenther wrote: On Thu, 5 Jul 2012, Tristan Gingold wrote: On Jul 4, 2012, at 8:31 PM, Magnus Fromreide wrote: Hello. Ever since the ISL patch went in my builds have failed. Which ISL

Re: [patch] Add block debug info to phi_arg_d

2012-07-05 Thread Richard Guenther
On Thu, Jul 5, 2012 at 11:42 AM, Steven Bosscher stevenb@gmail.com wrote: On Thu, Jul 5, 2012 at 11:23 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jul 5, 2012 at 11:08 AM, Dehao Chen de...@google.com wrote: Hi, This patch added block field to phi_arg_d to make sure the

Re: [patch] Add block debug info to phi_arg_d

2012-07-05 Thread Dehao Chen
On Thu, Jul 5, 2012 at 5:58 PM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jul 5, 2012 at 11:42 AM, Steven Bosscher stevenb@gmail.com wrote: On Thu, Jul 5, 2012 at 11:23 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jul 5, 2012 at 11:08 AM, Dehao Chen

Re: [Patch, fortran] PR fortran/53732

2012-07-05 Thread Dominique Dhumieres
Mikael, After the previous fix, the test fails with real*8 arr(4, 4, 4, 4) 1 Warning: Nonstandard type declaration REAL*8 at (1) Replacing real*8 arr(4, 4, 4, 4) with real(8) :: arr(4, 4, 4, 4) fixes the failures (due to the compilation with -pedantic-errors). Dominique

[Committed] S/390: Enable ifunc function attribute by default

2012-07-05 Thread Andreas Krebbel
Hi, the attached patch enables the ifunc function attribute by default on s390 and s390x. Committed to mainline. Bye, -Andreas- 2012-07-05 Andreas Krebbel andreas.kreb...@de.ibm.com * config.gcc: Enable ifunc attribute by default on s390 and s390x. --- gcc/config.gcc |2 ++ 1

[Committed] S/390: Use the tuned glibc versions of memcpy and memcmp

2012-07-05 Thread Andreas Krebbel
Hi, with this patch the inline versions of memcpy and memcmp will only be used for constant size parameters up to 64k. In the other cases glibc hopefully will be able to come up with an CPU (z10 and z196) optimized version dynamically via ifunc. Committed to mainline. Bye, -Andreas-

[PATCH] Fix edge probabilities of loop guards

2012-07-05 Thread Richard Guenther
This fixes edge probabilities of loop guards the vectorizer inserts for prologue/epilogue peeling. Formerly we'd end up with prolog_loop_niters.72_165 = D.2000_146 1; if (prolog_loop_niters.72_165 == 0) goto bb 7; else goto bb 4; # SUCC: 4 [100.0%] (false) 7 (true) which is

Re: [PATCH] Fix for PR52009 - Another missed tail merging opportunity

2012-07-05 Thread Richard Guenther
On Wed, Jul 4, 2012 at 8:07 PM, Tom de Vries tom_devr...@mentor.com wrote: On 31/01/12 22:07, Tom de Vries wrote: On 31/01/12 22:05, Tom de Vries wrote: Richard, Sorry, with patch this time. this patch fixes PR52009. Consider this test-case: ... int z; void foo (int y) { if (y

Re: [patch] Add block debug info to phi_arg_d

2012-07-05 Thread Richard Guenther
On Thu, Jul 5, 2012 at 12:15 PM, Dehao Chen de...@google.com wrote: On Thu, Jul 5, 2012 at 5:58 PM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jul 5, 2012 at 11:42 AM, Steven Bosscher stevenb@gmail.com wrote: On Thu, Jul 5, 2012 at 11:23 AM, Richard Guenther

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-05 Thread Michael Matz
Hi, On Wed, 4 Jul 2012, Steven Bosscher wrote: On Wed, Jul 4, 2012 at 7:02 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Any suggestions how to fix this? Should tail merging detect __builtin_unreachable and not merge such block? That seems to be the most straight-forward thing to do. I

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-05 Thread Richard Guenther
On Thu, Jul 5, 2012 at 2:44 PM, Michael Matz m...@suse.de wrote: Hi, On Wed, 4 Jul 2012, Steven Bosscher wrote: On Wed, Jul 4, 2012 at 7:02 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Any suggestions how to fix this? Should tail merging detect __builtin_unreachable and not merge such

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-05 Thread Tom de Vries
On 04/07/12 19:02, Ulrich Weigand wrote: Any suggestions how to fix this? Should tail merging detect __builtin_unreachable and not merge such block? Or else, should the CFG optimizer be extended (how?) to handle unreachable blocks with multiple predecessors better? Ulrich, I extended the

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-05 Thread Michael Matz
Hi, On Thu, 5 Jul 2012, Richard Guenther wrote: On Wed, Jul 4, 2012 at 7:02 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Any suggestions how to fix this? Should tail merging detect __builtin_unreachable and not merge such block? That seems to be the most straight-forward thing

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-05 Thread Richard Guenther
On Thu, Jul 5, 2012 at 2:49 PM, Tom de Vries tom_devr...@mentor.com wrote: On 04/07/12 19:02, Ulrich Weigand wrote: Any suggestions how to fix this? Should tail merging detect __builtin_unreachable and not merge such block? Or else, should the CFG optimizer be extended (how?) to handle

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-05 Thread Michael Matz
Hi, On Thu, 5 Jul 2012, Tom de Vries wrote: The asserts allow the return result to be optimized, but not the cfg conditions. AFAIU, we can insert the asserts earlier. F.i., we can insert aD.1711_6 = ASSERT_EXPR aD.1711_1(D), aD.1711_1(D) 0 before the GIMPLE_COND in bb2. Nope. That

Re: [Patch, Fortran] Add parsing support for assumed-rank array

2012-07-05 Thread Mikael Morin
On 24.06.2012 17:34, Tobias Burnus wrote: Tobias Burnus wrote: To cleanup my local trees; I had the patch lingering there for a many weeks. User visible, it only adds parsing support for dimension(..) and a sorry message. I have now updated the patch. Changes: Hello, some commen^Wbike

New Croatian PO file for 'gcc' (version 4.7.1)

2012-07-05 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Croatian team of translators. The file is available at: http://translationproject.org/latest/gcc/hr.po (This file, 'gcc-4.7.1.hr.po', has

Re: [Patch, Fortran] CLASS handling for assumed-rank arrays

2012-07-05 Thread Mikael Morin
On 26.06.2012 19:12, Tobias Burnus wrote: Build and regtested on x86-64-linux. OK for the trunk? Tobias OK once the preliminary patch gets approved. Mikael

Re: [patch] Fix debug info of nested inline functions

2012-07-05 Thread H.J. Lu
On Wed, May 16, 2012 at 2:29 PM, Eric Botcazou ebotca...@adacore.com wrote: Right, and that's why we want your change to split the nested function into abstract and concrete instances.  But then it should be fine to attach the abstract instance to the abstract parent normally, I would think.

[PATCH] Fix thinko in int type pretty printing

2012-07-05 Thread Richard Guenther
I should have consulted docs. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2012-07-05 Richard Guenther rguent...@suse.de * tree-pretty-print.c (dump_generic_node): Properly test the result of exact_log2. Index: gcc/tree-pretty-print.c

Re: [Patch, fortran] PR fortran/53732

2012-07-05 Thread Mikael Morin
On 05.07.2012 12:32, Dominique Dhumieres wrote: Mikael, After the previous fix, the test fails with real*8 arr(4, 4, 4, 4) 1 Warning: Nonstandard type declaration REAL*8 at (1) Replacing real*8 arr(4, 4, 4, 4) with real(8) :: arr(4, 4, 4, 4) fixes the failures (due to the

RFA: MN10300: Call default_encode_section_info

2012-07-05 Thread Nick Clifton
Hi Jeff, Hi Alex, This may count as an obvious fix, but I felt that I ought to check, just in case... Currently mn10300_encode_section_info() does not call default_encode_section_info(). This means that it misses out on possibly setting some important flags in the symbol, such as

Re: RFA: MN10300: Call default_encode_section_info

2012-07-05 Thread Jeff Law
On 07/05/2012 08:29 AM, Nick Clifton wrote: Hi Jeff, Hi Alex, This may count as an obvious fix, but I felt that I ought to check, just in case... Currently mn10300_encode_section_info() does not call default_encode_section_info(). This means that it misses out on possibly setting

Re: [PATCH] Merge the ISL optimizer from the graphite branch

2012-07-05 Thread Jack Howarth
On Tue, Jul 03, 2012 at 02:45:29PM +0200, Tobias Grosser wrote: On 07/03/2012 02:24 PM, Richard Guenther wrote: On Tue, 3 Jul 2012, Tobias Grosser wrote: On 07/03/2012 01:56 PM, Richard Guenther wrote: On Tue, 3 Jul 2012, Tobias Grosser wrote: On 07/03/2012 01:15 PM, Richard Guenther wrote:

Re: [PATCH, rs6000] Fix insertion of nop[s] to force dependent load into new dispatch group.

2012-07-05 Thread Pat Haugen
On 05/23/2012 10:48 AM, Pat Haugen wrote: The following patch fixes existing code that tried to prevent load-hit-store (LHS) from being in the same dispatch group. The main problem was use of the wrong dependency list in is_costly_group(), but I also added code to verify the memory refs

Re: [PATCH] Fix for PR51879 - Missed tail merging with non-const/pure calls

2012-07-05 Thread Tom de Vries
On 03/05/12 12:21, Richard Guenther wrote: On Wed, May 2, 2012 at 4:06 PM, Tom de Vries tom_devr...@mentor.com wrote: On 27/04/12 11:01, Richard Guenther wrote: SNIP I see you do not handle SNIP struct S { int i; }; struct S foo (void); struct S bar (void) { struct S s1, s2; if

Re: [PATCH, rs6000] Fix insertion of nop[s] to force dependent load into new dispatch group.

2012-07-05 Thread David Edelsohn
On Thu, Jul 5, 2012 at 12:16 PM, Pat Haugen pthau...@linux.vnet.ibm.com wrote: Ok for backport to 4.7 branch now that that is open again? Yes. Thanks, David

[patch mingw]: Fix PR bootstrap/52947

2012-07-05 Thread Kai Tietz
Hello, This patch fixes an long outstanding bootstrap-issue for mingw-native targets by using msys' make. The problem is that this make *auto-magical* modifies POSIX-paths to DOS-style paths. This leads to the issue that two DOS-style paths getting merged together for native build. This patch

Re: [PATCH] PR c++/50852 - loose template parameter comparison

2012-07-05 Thread Jason Merrill
On 04/12/2012 05:08 PM, Jason Merrill wrote: The fixup stuff is a solution to the problem of typedefs with incompatible template parameters leading to crashes, but it seems to me that this is just a symptom of the problem of replacing one typedef with another, and we should attack that

Re: Ping*4: [RFA:] fix bug in configure header-probing for stack protector support in target C library

2012-07-05 Thread Nathanael Nerode (GCC)
On 06/29/2012 09:15 AM, Hans-Peter Nilsson wrote: From: Joseph S. Myers jos...@codesourcery.com Date: Fri, 22 Jun 2012 13:09:42 +0200 On Fri, 22 Jun 2012, Hans-Peter Nilsson wrote: Fix configure test for stack protector support in target C library. * configure.ac (test_prefix,

Re: Tree tail merging breaks __builtin_unreachable optimization

2012-07-05 Thread Tom de Vries
On 05/07/12 15:30, Michael Matz wrote: Hi, On Thu, 5 Jul 2012, Tom de Vries wrote: The asserts allow the return result to be optimized, but not the cfg conditions. AFAIU, we can insert the asserts earlier. F.i., we can insert aD.1711_6 = ASSERT_EXPR aD.1711_1(D), aD.1711_1(D) 0

C++ PATCH for c++/50852, 53039 (problems with typedefs in templates)

2012-07-05 Thread Jason Merrill
Since Dodji's work to support typedef access checking in templates for GCC 4.6, there have been problems with typedefs used in template arguments; if the typedefs compare as equivalent but come from different templates, they expect different numbers/kinds of template arguments, and so if one

Re: [Patch ARM] PR51980 / PR49801 Improve Neon permute intrinsics.

2012-07-05 Thread Ramana Radhakrishnan
On 5 July 2012 17:51, Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote: 2012-07-05 Ramana Radhakrishnan ramana.radhakrish...@linaro.org Julian Brown jul...@codesourcery.com PR target/49891 It is PR target/48941 and not the number as above or in the subject

[PATCH][Cilkplus] Patch to fix a couple testsuite results

2012-07-05 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilk Plus branch mainly affecting the C compiler. It will fix tests failing in gcc.c-torture and some -Wpadding tests. Thanking You, Yours sincerely, Balaji V. Iyer. Index: cilk.c ===

RE: [PATCH][Cilkplus] Patch to fix a couple testsuite results

2012-07-05 Thread Iyer, Balaji V
Sorry, I created the patch in the wrong directory. Please see the corrected patch. Thanks, Balaji V. Iyer. -Original Message- From: Iyer, Balaji V Sent: Thursday, July 05, 2012 5:09 PM To: 'gcc-patches@gcc.gnu.org' Subject: [PATCH][Cilkplus] Patch to fix a couple testsuite results

Re: C++ PATCH for c++/50852, 53039 (problems with typedefs in templates)

2012-07-05 Thread Jason Merrill
On 07/05/2012 03:37 PM, Jason Merrill wrote: have not done that for non-type arguments so far. This patch implements typedef stripping for non-type template arguments, and thereby avoids the need for any fixup of sibling information. The patch also fixes 53039, which is another case of

Re: C++ PATCH for c++/50852, 53039 (problems with typedefs in templates)

2012-07-05 Thread Paolo Carlini
On 07/05/2012 09:37 PM, Jason Merrill wrote: The patch also fixes 53039, which is another case of treating things as interchangeable that turn out not to be: in this case they are template parameter packs that are compared for identity in arg_from_parm_pack_p, which was running into problems

Re: [patch][RFC] Remove tree/gimple/rtl dependencies from core CFG code

2012-07-05 Thread Joseph S. Myers
My only comment is that you add a coretypes.h include to tree-flow.h and remove one from tree-ssa-alias.h - but since the GCC convention is that all .c files start with includes of config.h, system.h and coretypes.h in that order, the include in tree-flow.h shouldn't be needed - any source

Re: C++ PATCH for c++/50852, 53039 (problems with typedefs in templates)

2012-07-05 Thread Jason Merrill
On 07/05/2012 06:33 PM, Paolo Carlini wrote: Thanks! I suppose nothing simple and safe can be applied to 4.7... I'm not sure. On the one hand, it is a significant change; on the other hand, it's pretty straightforward and fixes several regressions. Let's leave it on the trunk for a while

Re: [RFC, ivopts] fix bugs in ivopts address cost computation

2012-07-05 Thread Jiangning Liu
Hi, For the following code change, @@ -4212,11 +4064,6 @@ get_computation_cost_at (struct ivopts_d cost.cost += adjust_setup_cost (data, add_cost (TYPE_MODE (ctype), speed)); - /* Having offset does not affect runtime cost in case it is added to -

[gimplefe] Fixing the bug for gimple_assign statement with ternary operands

2012-07-05 Thread Sandeep Soni
The following patch fixes the issues that were faced by the introduction of the previous patch Diego, the crux was using gimple_build_assign_with_ops3 for ternary operands. Thanks for your help in finding it. I am halfway through the patch for building gimple_cond statements. I will be able to