2011-07-21 Tobias Grosser tob...@grosser.es
* 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
---
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
---
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 @@
2011-07-26 Sebastian Pop sebastian@amd.com
* 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
2011-07-21 Tobias Grosser tob...@grosser.es
* 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
2011-07-21 Tobias Grosser tob...@grosser.es
* configure: Regenerated.
* config/cloog.m4: Do not define CLOOG_ORG
and in gcc/
2011-07-21 Tobias Grosser tob...@grosser.es
* Makefile.in (graphite-clast-to-gimple.o, graphite-cloog-util.o):
Remove
* 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
---
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
---
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
---
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
---
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
---
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
+++
---
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
---
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/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
---
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
---
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
---
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
---
gcc/graphite-dependences.c | 585 +---
gcc/graphite-dependences.h | 39 +--
gcc/graphite-poly.c|3 -
.../testsuite/libgomp.graphite/force-parallel-1.c |4 +-
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
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
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);
+
+
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: [wwwdocs]
On Mon, 15 Aug 2011, Hans-Peter Nilsson wrote:
This patch:
2011-08-11 Uros Bizjak ubiz...@gmail.com
* 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):
ping
On Mon, Aug 8, 2011 at 11:00 AM, Guozhi Wei car...@google.com 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
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
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
Uros Bizjak ubiz...@gmail.com 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
Date: Mon, 15 Aug 2011 10:43:35 +0200 (CEST)
From: Richard Guenther rguent...@suse.de
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
On Sun, Aug 14, 2011 at 7:24 PM, Uros Bizjak ubiz...@gmail.com 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
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 rguent...@suse.de
PR tree-optimization/50058
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.
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
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
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
On Mon, Aug 15, 2011 at 02:36, Sven Verdoolaege sk...@kotnet.org 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
On Mon, Aug 15, 2011 at 02:23, Tobias Grosser tob...@grosser.es 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.
On Mon, Aug 15, 2011 at 07:40, Joseph S. Myers jos...@codesourcery.com 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
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
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 - GNU GCC
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 (bundled)
On Fri, Aug 12, 2011 at 4:03 AM, Artem Shinkarov
artyom.shinkar...@gmail.com 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
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 rguent...@suse.de
* tree-vrp.c (value_range_nonnegative_p): Fix anti-range case.
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
addsd
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
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
On Mon, Aug 15, 2011 at 4:56 PM, Richard Sandiford
richard.sandif...@linaro.org 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
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, %xmm1
orpd
On Sun, Aug 7, 2011 at 23:00, Guozhi Wei car...@google.com wrote:
2011-08-08 Guozhi Wei car...@google.com
Backport r174965 from trunk.
2011-06-12 Ira Rosen ira.ro...@linaro.org
* tree-vect-data-refs.c (vect_peeling_hash_get_most_frequent):
Richard Guenther richard.guent...@gmail.com writes:
On Mon, Aug 15, 2011 at 5:15 PM, Richard Sandiford
richard.sandif...@linaro.org 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
On Mon, Aug 15, 2011 at 5:34 PM, Richard Sandiford
richard.sandif...@linaro.org wrote:
Richard Guenther richard.guent...@gmail.com writes:
On Mon, Aug 15, 2011 at 5:15 PM, Richard Sandiford
richard.sandif...@linaro.org wrote:
rtx_cost knows that the expression under test (X) is an operand to
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
On Mon, Aug 15, 2011 at 3:24 PM, Richard Guenther
richard.guent...@gmail.com wrote:
On Fri, Aug 12, 2011 at 4:03 AM, Artem Shinkarov
artyom.shinkar...@gmail.com wrote:
Hi
Here is a completed version of the vector comparison patch we
discussed a long time ago here:
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
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,
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
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 max
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 could be
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
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
[+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
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
From: Rainer Orth r...@cebitec.uni-bielefeld.de
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: Rainer Orth r...@cebitec.uni-bielefeld.de
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
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 you do
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 the
Gabriel Charette gch...@google.com 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
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 quite
On Mon, Aug 15, 2011 at 1:39 PM, Ian Lance Taylor i...@google.com wrote:
Gabriel Charette gch...@google.com 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
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
Forwarding this patch to pph (which was just committed to trunk in
issue4874043).
Committed to pph branch.
2011-08-11 Gabriel Charette gch...@google.com
libcpp/ChangeLog
* include/line-map.h (LINEMAP_POSITION_FOR_COLUMN): Remove.
Update all users to use
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
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 @@
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:
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
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.
Gabriel Charette gch...@google.com writes:
On Mon, Aug 15, 2011 at 1:39 PM, Ian Lance Taylor i...@google.com wrote:
Gabriel Charette gch...@google.com 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
77 matches
Mail list logo