[PATCH 01/20] Make CLooG-ISL the only supported CLooG version.

2011-08-15 Thread Sebastian Pop
2011-07-21 Tobias Grosser * configure: Regenerated. * config/cloog.m4: Remove support for CLooG-ppl and CLooG-parma, both cloog.org and legacy versions. The only supported version will be CLooG with the isl backend. --- ChangeLog |7 ++ config/cloog.m4

[PATCH 07/20] fix memory leak

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-sese-to-poly.c |1 + 1 files changed, 1 insertions(+), 0 deletions(-) diff --git a/gcc/graphite-sese-to-poly.c b/gcc/graphite-sese-to-poly.c index 7e23c9d..05280e7 100644 --- a/gcc/graphite-sese-to-poly.c +++ b/gcc/graphite-sese-to-poly.c @@ -1996,6 +1996,7 @@ build_scop_drs

[PATCH 06/20] Remove ATTRIBUTE_UNUSED

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-cloog-util.c | 10 +- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/gcc/graphite-cloog-util.c b/gcc/graphite-cloog-util.c index 9bc24a0..83cfb54 100644 --- a/gcc/graphite-cloog-util.c +++ b/gcc/graphite-cloog-util.c @@ -236,7 +236,7 @@ new_C_Polyhedron_f

[PATCH 04/20] Document CLooG-ISL requirement for Graphite

2011-08-15 Thread Sebastian Pop
2011-07-26 Sebastian Pop * doc/invoke.texi: Document CLooG-ISL requirement for Graphite. --- gcc/doc/install.texi | 24 1 files changed, 8 insertions(+), 16 deletions(-) diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi index 9b1b037..368221f 100644 -

[PATCH 02/20] Require cloog 0.16.3

2011-08-15 Thread Sebastian Pop
2011-07-21 Tobias Grosser * configure: Regenerated. * configure.ac: Require cloog isl 0.16.3 --- ChangeLog|5 + configure|6 +++--- configure.ac |2 +- 3 files changed, 9 insertions(+), 4 deletions(-) diff --git a/ChangeLog b/ChangeLog index aeb3f14..0c

[PATCH 03/20] Remove code that supported legacy CLooG.

2011-08-15 Thread Sebastian Pop
2011-07-21 Tobias Grosser * configure: Regenerated. * config/cloog.m4: Do not define CLOOG_ORG and in gcc/ 2011-07-21 Tobias Grosser * Makefile.in (graphite-clast-to-gimple.o, graphite-cloog-util.o): Remove graphite-cloog-util.h. * graphite-clast-to

[PATCH 05/20] Move to new Cloog interface.

2011-08-15 Thread Sebastian Pop
* graphite-clast-to-gimple.c (new_clast_name_index): Store a copy of the string, no just a reference. (clast_name_index): Add a new field, that specifies if we need to free the name. (free_clast_name_index): If necessary, free the name string. (clast_name_index_elt_info): Calculate the

[PATCH 11/20] document ISL requirement for GCC installation

2011-08-15 Thread Sebastian Pop
--- gcc/doc/install.texi |8 +++- 1 files changed, 7 insertions(+), 1 deletions(-) diff --git a/gcc/doc/install.texi b/gcc/doc/install.texi index 368221f..f2b2fd9 100644 --- a/gcc/doc/install.texi +++ b/gcc/doc/install.texi @@ -368,6 +368,11 @@ It can be downloaded from @uref{http://www.

[PATCH 10/20] use cloog_isl_state_malloc

2011-08-15 Thread Sebastian Pop
--- gcc/graphite.c | 11 ++- 1 files changed, 6 insertions(+), 5 deletions(-) diff --git a/gcc/graphite.c b/gcc/graphite.c index b2cf7c6..7d124c7 100644 --- a/gcc/graphite.c +++ b/gcc/graphite.c @@ -41,6 +41,7 @@ along with GCC; see the file COPYING3. If not see #include #include

[PATCH 13/20] add pbb->schedule

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-poly.c |6 gcc/graphite-poly.h |6 gcc/graphite-sese-to-poly.c | 55 -- 3 files changed, 64 insertions(+), 3 deletions(-) diff --git a/gcc/graphite-poly.c b/gcc/graphite-poly.c index ce0649b..3d81372 1006

[PATCH 15/20] strip-mine with isl

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-blocking.c | 32 +++- 1 files changed, 31 insertions(+), 1 deletions(-) diff --git a/gcc/graphite-blocking.c b/gcc/graphite-blocking.c index 429b405..0741f82 100644 --- a/gcc/graphite-blocking.c +++ b/gcc/graphite-blocking.c @@ -27,6 +27,7 @@ along w

[PATCH 16/20] interchange with isl

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-interchange.c | 17 + gcc/graphite-poly.h| 16 2 files changed, 33 insertions(+), 0 deletions(-) diff --git a/gcc/graphite-interchange.c b/gcc/graphite-interchange.c index b819ece..68b9c1e 100644 --- a/gcc/graphite-interchange.c +++ b/

[PATCH 12/20] add pbb->domain

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-poly.c | 12 +++- gcc/graphite-poly.h |6 ++ gcc/graphite-sese-to-poly.c | 205 +++ 3 files changed, 187 insertions(+), 36 deletions(-) diff --git a/gcc/graphite-poly.c b/gcc/graphite-poly.c index fd2703b..ce0649b 100

[PATCH 17/20] use isl's datadep analysis

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-dependences.c | 169 ++- gcc/graphite-poly.c |8 + gcc/graphite-poly.h |3 + gcc/testsuite/gcc.dg/graphite/interchange-14.c |3 +- gcc/testsuite/gcc.dg/graphite/interchan

[PATCH 14/20] add pdr->accesses and pdr->extent

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-poly.c |7 ++- gcc/graphite-poly.h |4 +- gcc/graphite-sese-to-poly.c | 149 --- 3 files changed, 148 insertions(+), 12 deletions(-) diff --git a/gcc/graphite-poly.c b/gcc/graphite-poly.c index 3d81372..2835311 1006

[PATCH 18/20] remove dead code

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-dependences.c | 298 --- gcc/graphite-poly.c | 47 --- gcc/graphite-poly.h |9 -- gcc/graphite-sese-to-poly.c |8 +- 4 files changed, 2 insertions(+), 360 deletions(-) diff --git a/gcc/graphite-dependences.c

[PATCH 20/20] disable loop flattening

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-flattening.c | 289 + 1 files changed, 4 insertions(+), 285 deletions(-) diff --git a/gcc/graphite-flattening.c b/gcc/graphite-flattening.c index c58d8a3..c8c8531 100644 --- a/gcc/graphite-flattening.c +++ b/gcc/graphite-flattening.c @

[PATCH 08/20] Add ISL data structures

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-blocking.c| 12 +++- gcc/graphite-clast-to-gimple.c |9 + gcc/graphite-cloog-util.c | 11 ++- gcc/graphite-cloog-util.h |2 -- gcc/graphite-dependences.c | 11 ++- gcc/graphite-flattening.c | 11 ++-

[PATCH 09/20] Add scop->context

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-clast-to-gimple.c | 55 ++-- gcc/graphite-cloog-util.c |1 + gcc/graphite-poly.c|8 + gcc/graphite-poly.h|3 + gcc/graphite-sese-to-poly.c| 299 +++- gcc/graphite.c |4 + 6

[PATCH 19/20] remove old data dependence analysis, xfail graphite-force-parallel

2011-08-15 Thread Sebastian Pop
--- gcc/graphite-dependences.c | 585 +--- gcc/graphite-dependences.h | 39 +-- gcc/graphite-poly.c|3 - .../testsuite/libgomp.graphite/force-parallel-1.c |4 +- .../testsuite/libgomp.graphi

[PATCH] add configure --with-isl

2011-08-15 Thread Sebastian Pop
Hi, This patch still needs some changes to ISL for the compile time version check to work: [#if ISL_VERSION_MAJOR != $1 \ || ISL_VERSION_MINOR != $2 \ || ISL_VERSION_REVISION < $3 I am providing this patch for reference, just in case ISL 0.08 will contain these defines. I have not tes

Re: [PATCH] add configure --with-isl

2011-08-15 Thread Tobias Grosser
On 08/15/2011 08:19 AM, Sebastian Pop wrote: Hi, This patch still needs some changes to ISL for the compile time version check to work: [#if ISL_VERSION_MAJOR != $1 \ || ISL_VERSION_MINOR != $2 \ || ISL_VERSION_REVISION< $3 I am providing this patch for reference, just in case IS

Re: [PATCH 15/20] strip-mine with isl

2011-08-15 Thread Sven Verdoolaege
On Mon, Aug 15, 2011 at 02:12:54AM -0500, Sebastian Pop wrote: > @@ -160,6 +182,14 @@ pbb_strip_mine_time_depth (poly_bb_p pbb, int > time_depth, int stride) > ppl_delete_Linear_Expression (expr); > ppl_Polyhedron_add_constraint (res, new_cstr); > ppl_delete_Constraint (new_cstr); >

RE: [wwwdocs] Announce new ARM/embedded-4_6-branch branch

2011-08-15 Thread Terry Guo
Hello Gerald, Could you please review my updated patch? BR, Terry > -Original Message- > From: Terry Guo [mailto:terry@arm.com] > Sent: Sunday, August 14, 2011 10:27 AM > To: 'Gerald Pfeifer' > Cc: 'gcc-patches@gcc.gnu.org'; Richard Earnshaw; Matthew Gretton-Dann > Subject: RE: [wwwd

Re: Fix spurious match testsuite regressions from "[PATCH, middle end]: Introduce BUILT_IN_I{CEIL_FLOOR_ROUND_RINT} FP-to-int conversion functions"

2011-08-15 Thread Richard Guenther
On Mon, 15 Aug 2011, Hans-Peter Nilsson wrote: > This patch: > > > 2011-08-11 Uros Bizjak > > > > * builtins.def (BUILT_IN_ICEIL{,F,L}, BUILT_IN_IFLOOR{,F,L}, > > BUILT_IN_IRINT{,F,L}, BUILT_IN_IROUND{,F,L}: New builtin definitions. > > * convert.c (convert_to_integer): Convert to

Re: [google] Backport r174965 from trunk to google/gcc-4_6 (issue4852046)

2011-08-15 Thread Carrot Wei
ping On Mon, Aug 8, 2011 at 11:00 AM, Guozhi Wei wrote: > > Hi > > I want to backport r174965 from trunk to google/gcc-4_6, which fixed vect-72.c > failure in target arm, as described in > http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00927.html > > Tested with buildit and regression test on arm q

[PATCH, testsuite, ARM] Divide by zero test no necessarily trap

2011-08-15 Thread Joey Ye
gcc.c/torture/execute/20101011-1.c always fails on ARM, because it needs the target architecture trap in division by zero. It is implementation defined on ARM to trap division by zero. So this case should be skipped on ARM. There are two ways to skip this test. One is using standard dejagnu op

Re: [PATCH] [JAVA] patch for Java on RTEMS

2011-08-15 Thread Andrew Haley
On 08/10/2011 06:00 PM, Jie Liu wrote: > For the previous analysis "libjava patches for RTEMS"[1], there are 6 > cases need to pay more attention. PR18699, TLtest, Thread_Interrupt, > Thread_Sleep_2, Throw_2 and bclink. > > After add patch to bdwgc for RTEMS pthread support[2]: > PR18699

Re: [x86] Use match_test for .md attributes

2011-08-15 Thread Richard Sandiford
Uros Bizjak writes: > Hello! >> Following on from the two patches I've just posted, this one makes >> config/i386/*.md use match_test for .md attributes. Tested as >> described here: > >> http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01182.html > >> * config/i386/i386.md: Use (match_test

Re: Fix spurious match testsuite regressions from "[PATCH, middle end]: Introduce BUILT_IN_I{CEIL_FLOOR_ROUND_RINT} FP-to-int conversion functions"

2011-08-15 Thread Hans-Peter Nilsson
> Date: Mon, 15 Aug 2011 10:43:35 +0200 (CEST) > From: Richard Guenther > On Mon, 15 Aug 2011, Hans-Peter Nilsson wrote: > > So, ok as is? > > If not, would you preapprove tree-dumping decl_uids only at a > > higher dump verbosity level? > > Ok. Yes, definitely, a patch to do decl_uids dumping

Re: [PATCH, i386]: Expand round(a) = sgn(a) * floor(fabs(a) + 0.5) using SSE4 ROUND insn

2011-08-15 Thread Richard Guenther
On Sun, Aug 14, 2011 at 7:24 PM, Uros Bizjak wrote: > Hello! > > We can use ROUNDSP/ROUNDSD in round(a) expansion. Currently, we expand > round(a) as (-O2 -ffast-math): > > .LFB0: >        .cfi_startproc >        movsd   .LC1(%rip), %xmm1 >        movapd  %xmm0, %xmm2 >        movsd   .LC0(%rip),

[PATCH] Fix PR50058

2011-08-15 Thread Richard Guenther
This fixes the FAIL of PR50058 - I tightened the struct copy matching too much and somehow the FAIL got lost during testing. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-08-15 Richard Guenther PR tree-optimization/50058 * tree-ssa-sccv

Re: [Patch ARM] Fix PR50022

2011-08-15 Thread Ramana Radhakrishnan
> > fails to build, rename it everywhere if needed. Yes - sorry . I had fixed this up on Friday after accidentally sending a wrong version. The final version is attached. Bootstrap with C and Ada completed ok with this patch. Regression tests on a cross-compiler showed no new failures. Committed

Re: [PATCH 01/20] Make CLooG-ISL the only supported CLooG version.

2011-08-15 Thread Joseph S. Myers
I can't tell if this is meant to be a trunk submission or a branch submission. But if it's a trunk submission, my previous comments still apply: changes to prerequisites need raising on the gcc list for the benefit of the wider audience there, and with as much detail (about what hosts and conf

Re: [PATCH 01/20] Make CLooG-ISL the only supported CLooG version.

2011-08-15 Thread Joseph S. Myers
I'd also like to point out that this is simply not a proper patch submission at all, independent of the specific issues about changes to prerequisites that require things to be raised and explained in a self-contained way on the gcc list. Every patch submission needs an explanation of the purp

[PATCH] Remove "bogus" g++.dg/init/copy7.C testcase

2011-08-15 Thread Richard Guenther
The g++.dg/init/copy7.C testcase checks whether the C++ frontend guards memcpy it emits via a conditional verifying that src != dst because calling memcpy with overlapping source / destination is not supported. The testcase is misguided though (and the C++ frontend was, until recently) - the midd

Re: [PATCH 11/20] document ISL requirement for GCC installation

2011-08-15 Thread Sebastian Pop
On Mon, Aug 15, 2011 at 02:36, Sven Verdoolaege wrote: >> +It can be downloaded from @uref{http://www.kotnet.org/~skimo/isl/}. > > Why don't you want to use the official URL? Changed now. From 93d892edaa303736cd0ca89662bec979aec0485c Mon Sep 17 00:00:00 2001 From: Sebastian Pop Date: Fri, 12 Aug

Re: [PATCH] add configure --with-isl

2011-08-15 Thread Sebastian Pop
On Mon, Aug 15, 2011 at 02:23, Tobias Grosser wrote: >>  # Check for CLOOG >> >>  dnl Provide configure switches and initialize clooginc&  clooglibs >>  dnl with user input. >>  CLOOG_INIT_FLAGS >> +if test "x$with_isl" = "xno"; then >> +  dnl Only execute fail-action, if CLooG has been requested.

Re: [PATCH 01/20] Make CLooG-ISL the only supported CLooG version.

2011-08-15 Thread Sebastian Pop
On Mon, Aug 15, 2011 at 07:40, Joseph S. Myers wrote: > I'd also like to point out that this is simply not a proper patch > submission at all, independent of the specific issues about changes to > prerequisites that require things to be raised and explained in a > self-contained way on the gcc lis

Re: [PATCH] [JAVA] patch for Java on RTEMS

2011-08-15 Thread Jie Liu
> Looks OK, but there is no ChangeLog.  Do you have copyright > assignment? Have added ChangeLog to the patch, please see the attachment. And I think I have copyright assignment, because I have Free Software Foundation paperwork, as "ASSIGNMENT - GNU GCC ... JIE RT688742" Best Regards, Jie gcj-

Re: [PATCH] [JAVA] patch for Java on RTEMS

2011-08-15 Thread Andrew Haley
On 08/15/2011 03:07 PM, Jie Liu wrote: >> Looks OK, but there is no ChangeLog. Do you have copyright >> assignment? > > Have added ChangeLog to the patch, please see the attachment. And I > think I have copyright assignment, because I have Free Software > Foundation paperwork, as "ASSIGNMENT - GN

Re: [PATCH 2/2] document ISL requirement for GCC installation

2011-08-15 Thread Michael Matz
Hi, On Sun, 14 Aug 2011, Richard Guenther wrote: > > which would be exactly the way no distribution would use it. So please > > just don't bundle ISL with CLoog. > > Well, I would simply have linked the bundled ISL statically into > libcloog. Which would still require not exporting the (bundl

Re: Vector Comparison patch

2011-08-15 Thread Richard Guenther
On Fri, Aug 12, 2011 at 4:03 AM, Artem Shinkarov wrote: > Hi > > Here is a completed version of the vector comparison patch we > discussed a long time ago here: > http://gcc.gnu.org/ml/gcc-patches/2010-08/msg01184.html > > The patch implements vector comparison according to the OpenCL > standard,

[PATCH] VRP TLC

2011-08-15 Thread Richard Guenther
This restructures extract_range_from_unary_expr_1 so it is easier to follow. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2011-08-15 Richard Guenther * tree-vrp.c (value_range_nonnegative_p): Fix anti-range case. (extract_range_from_unary_expr_1): R

Re: [PATCH, i386]: Expand round(a) = sgn(a) * floor(fabs(a) + 0.5) using SSE4 ROUND insn

2011-08-15 Thread Michael Matz
Hi, On Mon, 15 Aug 2011, Richard Guenther wrote: > > Adding -msse4, we now generate branchless code using roundsd: > > > > .LFB0: > >        .cfi_startproc > >        movsd   .LC0(%rip), %xmm2 > >        movapd  %xmm0, %xmm1 > >        andpd   %xmm2, %xmm1 > >        andnpd  %xmm0, %xmm2 > >    

Add a set_src_cost wrapper function

2011-08-15 Thread Richard Sandiford
I'm about to post a patch that adds an extra parameter to rtx_cost. Since most callers to rtx_cost are for SET_SRCs, it seemed a shame to have to add an extra boiler-plate parameter to each of them. The patch below therefore adds a set_src_cost wrapper for this common case. Tested on x86_64-linux

[PATCH] Fix PR50082

2011-08-15 Thread Richard Guenther
This "fixes" PR50082 by passing comparison operands stripped off useless conversions to the canonicalization routine. This makes sure to fold the testcase early from a point where emitting a overflow warning is properly constrained. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to

Re: Add a set_src_cost wrapper function

2011-08-15 Thread Richard Guenther
On Mon, Aug 15, 2011 at 4:56 PM, Richard Sandiford wrote: > I'm about to post a patch that adds an extra parameter to rtx_cost. > Since most callers to rtx_cost are for SET_SRCs, it seemed a shame to > have to add an extra boiler-plate parameter to each of them.  The patch > below therefore adds a

Re: [PATCH, i386]: Expand round(a) = sgn(a) * floor(fabs(a) + 0.5) using SSE4 ROUND insn

2011-08-15 Thread Michael Matz
Hi, On Mon, 15 Aug 2011, Michael Matz wrote: > > > .LFB0: > > >        .cfi_startproc > > >        movsd   .LC0(%rip), %xmm2 > > >        movapd  %xmm0, %xmm1 > > >        andpd   %xmm2, %xmm1 > > >        andnpd  %xmm0, %xmm2 > > >        addsd   .LC1(%rip), %xmm1 > > >        roundsd $1, %xmm1,

Re: [google] Backport r174965 from trunk to google/gcc-4_6 (issue4852046)

2011-08-15 Thread Diego Novillo
On Sun, Aug 7, 2011 at 23:00, Guozhi Wei wrote: > 2011-08-08  Guozhi Wei   > >        Backport r174965 from trunk. > >        2011-06-12  Ira Rosen   > >                * tree-vect-data-refs.c (vect_peeling_hash_get_most_frequent): >                Take number of iterations to peel into account f

Re: Add an operand number argument to rtx_cost

2011-08-15 Thread Richard Sandiford
Richard Guenther writes: > On Mon, Aug 15, 2011 at 5:15 PM, Richard Sandiford > wrote: >> rtx_cost knows that the expression under test (X) is an operand to an >> expression of a particular code (OUTER_CODE).  However, nothing says >> which operand it actually is.  This can make a difference for

Re: Add an operand number argument to rtx_cost

2011-08-15 Thread Richard Guenther
On Mon, Aug 15, 2011 at 5:34 PM, Richard Sandiford wrote: > Richard Guenther writes: >> On Mon, Aug 15, 2011 at 5:15 PM, Richard Sandiford >> wrote: >>> rtx_cost knows that the expression under test (X) is an operand to an >>> expression of a particular code (OUTER_CODE).  However, nothing says

Re: [PATCH] Remove "bogus" g++.dg/init/copy7.C testcase

2011-08-15 Thread Mike Stump
On Aug 15, 2011, at 5:42 AM, Richard Guenther wrote: > The argument still holds that no reasonable memcpy implementation > will reject the src == dest case. Hum... Sounds like if that's the case that we should document it in the manual as something we expect (requirement) of the memcpy implement

Re: Vector Comparison patch

2011-08-15 Thread Artem Shinkarov
On Mon, Aug 15, 2011 at 3:24 PM, Richard Guenther wrote: > On Fri, Aug 12, 2011 at 4:03 AM, Artem Shinkarov > wrote: >> Hi >> >> Here is a completed version of the vector comparison patch we >> discussed a long time ago here: >> http://gcc.gnu.org/ml/gcc-patches/2010-08/msg01184.html >> >> The pa

CFT: [build] Move libgcc_tm_file to toplevel libgcc

2011-08-15 Thread Rainer Orth
This patch almost completes the build side of the libgcc moves. It moves the libgcc_tm_file support to libgcc/config.host (tm_file), where it belongs. It builds on a patch Paolo posted some time ago, with some minor corrections (introducing the required dependencies on the generated libgcc_tm.h).

Re: [patch, fortran] PR 46659 - extend conversion warnings to assignments

2011-08-15 Thread Thomas Koenig
Am 14.08.2011 22:54, schrieb Tobias Burnus: Thomas Koenig wrote: the attached patch extends conversion warnings to assignments. OK for trunk? I get now two warnings for: complex(8), parameter :: z = cmplx (0.5, 0.5) r = z end The problem is that gfc_check_assign is called twice for this, fr

Fixes for debug/50006

2011-08-15 Thread Richard Henderson
Ada on PA-RISC turned up two different problems with my patches for REG_ARGS_SIZE. (1) When we emit_stack_restore, that also zaps any accumulated args_size. Emit the proper notes. (2) The code at the bottom of alloca was suppressing the creation of args_size notes, except when the target provid

Re: [C++0x] contiguous bitfields race implementation

2011-08-15 Thread Aldy Hernandez
Some comments. /* If we have a bit-field with a bitsize> 0... */ if (DECL_BIT_FIELD_TYPE (fld) && tree_low_cst (DECL_SIZE (fld), 1)> 0) I think we can check bitsize != 0, thus && !integer_zerop (DECL_SIZE (fld)) Done /* Short-circuit out if we have the

Re: RFC: [build, ada] Centralize PICFLAG configuration

2011-08-15 Thread Rainer Orth
Paolo, > On 08/10/2011 01:42 PM, Rainer Orth wrote: >> * Centralize the determination of PICFLAG. Currently, three libraries >>inside the gcc tree are built PIC without libtool: libgcc, libiberty, >>and libgnat/libgnarl. >> >>libiberty/configure.ac has a hardcoded list of PICFLAG that

Re: CFT: [build] Move libgcc_tm_file to toplevel libgcc

2011-08-15 Thread Paolo Bonzini
On 08/15/2011 10:30 AM, Rainer Orth wrote: This patch almost completes the build side of the libgcc moves. It moves the libgcc_tm_file support to libgcc/config.host (tm_file), where it belongs. It builds on a patch Paolo posted some time ago, with some minor corrections (introducing the require

Re: RFC: [build, ada] Centralize PICFLAG configuration

2011-08-15 Thread Steve Ellcey
On Mon, 2011-08-15 at 19:53 +0200, Rainer Orth wrote: > * For IA64 HP-UX, there's a claim that -fPIC is necessary despite PIC > code being the default. I could find no hint in trunk that this is > true any longer. Rainer, If I recall correctly the issue for -fPIC on IA64 HP-UX is to turn on

Re: Remove LINEMAP_POSITION_FOR_COLUMN macro (issue4874043)

2011-08-15 Thread Gabriel Charette
[+iant] Ian: can you approve the go changes in this patch? It's a change in the linemap that reflects in functions used by go, but that shouldn't have any effect on the go compiler's behaviour. As a side thought, I'm getting random failures in the go test suite, in my last run the difference was

Re: RFC: [build, ada] Centralize PICFLAG configuration

2011-08-15 Thread Paolo Bonzini
On 08/15/2011 10:53 AM, Rainer Orth wrote: * The general approach between libtool and libiberty differs. Unless otherwise specified (PIC is the default or doesn't work for some reason), libtool defaults to -fPIC, while libiberty has a strange mixture of -fPIC/-fpic and nothing, without

Re: CFT: [build] Move crtstuff support to toplevel libgcc

2011-08-15 Thread Hans-Peter Nilsson
> From: Rainer Orth > Date: Mon, 15 Aug 2011 19:01:39 +0200 > * Unlike any other target, cris has crisv32-*-none and cris-*-none > targets which seem to be aliases for their *-elf counterparts. Do we > really need to keep those? While I can inherit e.g. extra_parts from > *-*-elf, there's

Re: CFT: [build] Move libgcc2 to toplevel libgcc

2011-08-15 Thread Hans-Peter Nilsson
> From: Rainer Orth > Date: Mon, 15 Aug 2011 19:19:23 +0200 > On top of the crtstuff and libgcc1 patches (and requiring the PICFLAG > centralization still under discussion), here's the bulk of the remaining > moves: it moves all LIBGCC2*, LIB2* macros and referenced files. > > Despite its size,

Re: Remove LINEMAP_POSITION_FOR_COLUMN macro (issue 4874043)

2011-08-15 Thread dnovillo
On 2011/08/15 18:05:51, Gabriel Charette wrote: 2011-08-11 Gabriel Charette libcpp/ChangeLog * include/line-map.h (LINEMAP_POSITION_FOR_COLUMN): Remove. Update all users to use linemap_position_for_column instead. gcc/go/Change

Re: [PATCH 04/20] Document CLooG-ISL requirement for Graphite

2011-08-15 Thread Gerald Pfeifer
On Mon, 15 Aug 2011, Sebastian Pop wrote: > 2011-07-26 Sebastian Pop > > * doc/invoke.texi: Document CLooG-ISL requirement for Graphite. That looks like a nice simplification (where in my mind this has been a bit of a mine field in the past). This is approved once the code changes are

Re: [PATCH 11/20] document ISL requirement for GCC installation

2011-08-15 Thread Gerald Pfeifer
On Mon, 15 Aug 2011, Sebastian Pop wrote: > --- > gcc/doc/install.texi |8 +++- > 1 files changed, 7 insertions(+), 1 deletions(-) This needs a ChangeLog entry. With one, this is fine. While you are at it... > -If you do not have PPL (the Parma Polyhedra Library) and the CLooG > +If yo

Announcing the Port of Intel(r) Cilk (TM) Plus into GCC

2011-08-15 Thread Iyer, Balaji V
Hello Everyone, This letter describes the recently created GCC branch called "cilkplus" that ports the Intel(R) Cilk(TM) Plus language extensions to the C and C++ front-ends of gcc-4.7. We are looking for collaborators and advice as we proceed - both on this open-source gcc project, and on th

Re: Remove LINEMAP_POSITION_FOR_COLUMN macro (issue4874043)

2011-08-15 Thread Ian Lance Taylor
Gabriel Charette writes: > Ian: can you approve the go changes in this patch? The changes to the Go frontend are fine, but they need to be applied to the master repository for the Go frontend. I can take care of that when you are ready to commit the patch. Note that changes to the files in gcc

Re: CFT: [build] Move libgcc2 to toplevel libgcc

2011-08-15 Thread Joseph S. Myers
On Mon, 15 Aug 2011, Rainer Orth wrote: > On top of the crtstuff and libgcc1 patches (and requiring the PICFLAG > centralization still under discussion), here's the bulk of the remaining > moves: it moves all LIBGCC2*, LIB2* macros and referenced files. > > Despite its size, the patch should be q

Re: Remove LINEMAP_POSITION_FOR_COLUMN macro (issue4874043)

2011-08-15 Thread Gabriel Charette
On Mon, Aug 15, 2011 at 1:39 PM, Ian Lance Taylor wrote: > Gabriel Charette writes: > >> Ian: can you approve the go changes in this patch? > > The changes to the Go frontend are fine, but they need to be applied to > the master repository for the Go frontend.  I can take care of that when > you

Re: CFT: [build] Move libgcc_tm_file to toplevel libgcc

2011-08-15 Thread Joseph S. Myers
On Mon, 15 Aug 2011, Rainer Orth wrote: > This patch almost completes the build side of the libgcc moves. It > moves the libgcc_tm_file support to libgcc/config.host (tm_file), where > it belongs. It builds on a patch Paolo posted some time ago, with some > minor corrections (introducing the req

[pph] Forwarding trunk linemap changes (issue4898050)

2011-08-15 Thread Gabriel Charette
Forwarding this patch to pph (which was just committed to trunk in issue4874043). Committed to pph branch. 2011-08-11 Gabriel Charette libcpp/ChangeLog * include/line-map.h (LINEMAP_POSITION_FOR_COLUMN): Remove. Update all users to use linemap_position_for_column inst

Re: CFT: [build] Move libgcc_tm_file to toplevel libgcc

2011-08-15 Thread Paolo Bonzini
On 08/15/2011 02:05 PM, Joseph S. Myers wrote: > This patch almost completes the build side of the libgcc moves. It > moves the libgcc_tm_file support to libgcc/config.host (tm_file), where > it belongs. It builds on a patch Paolo posted some time ago, with some > minor corrections (introdu

PATCH: PR testsuite/50085: [4.7 Regression] FAIL: g++.dg/opt/life1.C scan-assembler GLOBAL_OFFSET_TABLE on x86_64-apple-darwin10

2011-08-15 Thread H.J. Lu
Hi, g++.dg/opt/life1.C is supposed to be run only on Linux. I checked in this patch as an obvious fix. Thanks. H.J. --- Index: ChangeLog === --- ChangeLog (revision 11) +++ ChangeLog (working copy) @@ -1,3 +1,8 @@ +2011-08

Re: [Patch, Fortran] Coarrays: Add/fix check for no coarrays as result value

2011-08-15 Thread H.J. Lu
On Thu, Aug 4, 2011 at 8:00 AM, Tobias Burnus wrote: > This patch fixes the result check for coarrays / variables with coarray > subcomponents. It was working with a separate RESULT() variable - but not if > the function name was the result variable. > > Build and regtested on x86-64-linux. > OK f

Re: [PATCH, SMS] Support instructions with REG_INC_NOTE (re-submisson)

2011-08-15 Thread Ayal Zaks
Ok, so this extends the infrastructure to support insns which set an arbitrary number of registers, but currently specifically handles only REG_INC situations (which set two registers). I'm not against {0,1,infinity}, but wonder if this case really deserves the complexity: post/pre-inc/decrementing

Linemap force location and remove LINEMAP_POSITION_FOR_COLUMN (issue4801090)

2011-08-15 Thread Gabriel Charette
Here is the updated patch. It nows exposes two libcpp functions to force the source_location for tokens when desired. The lexer then checks for a value set by these functions in cpp_reader and acts accordingly when needing a location for a new token (either using the forced_location or calling

[pph] Support for references to symbols in other PPH images (issue4873051)

2011-08-15 Thread Diego Novillo
This patch finishes the support for external references to symbols in other PPH files. This is used when one PPH image includes another. The symbols in the included image should not be emitted from the parent, and references to them should go to the correct location in the child's pickle cache.

Re: Remove LINEMAP_POSITION_FOR_COLUMN macro (issue4874043)

2011-08-15 Thread Ian Lance Taylor
Gabriel Charette writes: > On Mon, Aug 15, 2011 at 1:39 PM, Ian Lance Taylor wrote: >> Gabriel Charette writes: >> >>> Ian: can you approve the go changes in this patch? >> >> The changes to the Go frontend are fine, but they need to be applied to >> the master repository for the Go frontend.