Hi,
I checked the patch in trunk directly since it is obvious. Committed
revision 171206.
Thanks,
Mingjie
2011-03-21 Mingjie Xing mingjie.x...@gmail.com
* tree-dfa.c (add_referenced_var): Fix typo in comment.
Index: tree-dfa.c
Hi Mikael,
Here is the new version of the patch. Regression-tested. OK for trunk?
OK this time.
Thank you.
Sende fortran/ChangeLog
Sende fortran/dependency.c
Sende fortran/frontend-passes.c
Sende fortran/gfortran.h
Sende fortran/invoke.texi
On 2011/3/18 12:18 AM, Richard Henderson wrote:
On 03/16/2011 06:55 PM, Chung-Lin Tang wrote:
You have to use DeMorgan's Law to distribute the ~ operator:
Duh. Not sure where my head was yesterday.
Let's enhance the comment for someone else having an off day. Perhaps
/* Given
2011/3/18 Nicola Pero nicola.p...@meta-innovation.com:
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (revision 171155)
+++ gcc/ChangeLog (working copy)
@@ -1,3 +1,7 @@
+2011-03-18 Nicola Pero
2011/3/18 Jason Merrill ja...@redhat.com:
OK.
Jason
Applied first part at rev. 171209. Second part at rev. 171210.
Kai
On 03/20/2011 08:48 PM, Jakub Jelinek wrote:
Ok for trunk/4.6?
2011-03-20 Jakub Jelinek ja...@redhat.com
PR target/48213
* config/s390/s390.c (s390_delegitimize_address): Don't call
lowpart_subreg if orig_x has BLKmode.
* gcc.dg/pr48213.c: New test.
Ok.
On Sun, Mar 20, 2011 at 01:58:24AM +0100, Benjamin Kosnik wrote:
2011-03-19 Jakub Jelinek ja...@redhat.com
* config/abi/pre/gnu.ver (CXXABI_1.3): Don't export _ZT[IS][PK]*[no].
(CXXABI_1.3.5): Export _ZTI[PK]*[no].
This looks good to me, nice catch. I'll figure out what
On Mon, Mar 21, 2011 at 3:33 AM, Kai Tietz ktiet...@googlemail.com wrote:
2011/3/18 Jason Merrill ja...@redhat.com:
OK.
Jason
Applied first part at rev. 171209. Second part at rev. 171210.
thanks.
Hi!
This patch updates baseline_symbols.txt for a bunch of targets.
This is generated with the
http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01175.html
patch applied.
2011-03-21 Jakub Jelinek ja...@redhat.com
* config/abi/post/i386-linux-gnu/baseline_symbols.txt: Update.
*
I think with the patch validate_(unshare)_change becomes easier to use. You
can still decide locally if you want unsharing or not. Without taking into
account in which contexts your function gets called. I consider this a big
advantage.
You're supposed to know what you're doing though and
Hi,
with the attached patch dwarf2out tries to print the name of the
unspec together with the numeric value in case the back end uses
define_c_enum for the unspec constants.
Ok for mainline?
Bye,
-Andreas-
2011-03-21 Andreas Krebbel andreas.kreb...@de.ibm.com
* dwarf2out.c
On Mon, 21 Mar 2011, Rainer Orth wrote:
Richard Guenther rguent...@suse.de writes:
On Fri, 18 Mar 2011, Rainer Orth wrote:
Richard Guenther richard.guent...@gmail.com writes:
It seemed to have disabled linker-plugin support for old binutils
with
PING
2011/3/14 Kai Tietz ktiet...@googlemail.com:
Hello,
this patch fixes some DOS-filename comparision and directory-separator checks.
Changelog libcpp
2011-03-14 Kai Tietz
* files.c (file_hash_eq): Use filename_cmp
instead of strcmp.
(nonexistent_file_hash_eq):
While reviewing PR bootstrap/48135, it came up that the Perl
requirements in install.texi are inaccurate:
* On some platforms (Darwin, Solaris), Perl is a build-time requirement,
but documented with the development requirements only.
* On Solaris with symbol versioning enabled, it is always
On Thu, 17 Mar 2011, Rainer Orth wrote:
I've just reviewed install.texi for my platforms and found it mostly
accurate. A few minor changes are desirable for the 4.6 release, but
also apply to mainline since the file is identical there:
* Update recommendations to binutils 2.21 since this is
Hi,
the attached patch makes a couple of tweaks to the gimplifier in order to help
Ada, but I think that they are of general usefulness:
1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls. This is
correct since stack memory isn't considered as trapping in the IL.
2)
Prompted by PR bootstrap/48135, I had a closer look at make_sunver.pl
and noticed that it doesn't use File::Glob at all, but turns globs in
the input version script into a regular expression for matching.
Therefore, the requirement of a Perl version newer than what is bundled
in Solaris 8 should
On Mon, 21 Mar 2011, Rainer Orth wrote:
While reviewing PR bootstrap/48135, it came up that the Perl
requirements in install.texi are inaccurate:
* On some platforms (Darwin, Solaris), Perl is a build-time requirement,
but documented with the development requirements only.
* On Solaris
As reported in the PR, the toplevel configure test for libpwl doesn't
work since it fails to link with libstdc++. The following patch
corrects this.
There were a couple of issues:
* One cannot use LDFLAGS here since that is passed to $CC before -lpwl,
which of course fails with a static
Denis Chertykov schrieb:
2011/3/18 Georg-Johann Lay a...@gjlay.de:
Is it tested for regressions ?
Denis.
I ran tests against svn 170942 (latest 4.7.0 snapshot). Besides
timestamps, the diff looks like this:
1435a1436,1437
XPASS: gcc.dg/sibcall-3.c execution test
XPASS:
On 03/21/2011 12:48 PM, Rainer Orth wrote:
As reported in the PR, the toplevel configure test for libpwl doesn't
work since it fails to link with libstdc++. The following patch
corrects this.
There were a couple of issues:
* One cannot use LDFLAGS here since that is passed to $CC before
We still have support for --enable-threads=solaris (the old Unix
International thread library) in GCC. This is pretty pointless IMO:
* Both libthread and libpthread are only two different interfaces to the
same implementation.
* Most of our runtime libraries don't support libthread at all.
*
Gerald Pfeifer ger...@pfeifer.com writes:
On Thu, 17 Mar 2011, Rainer Orth wrote:
I've just reviewed install.texi for my platforms and found it mostly
accurate. A few minor changes are desirable for the 4.6 release, but
also apply to mainline since the file is identical there:
* Update
This avoids duplicating branches in the generated code at -O0 for complex loop
conditions involving boolean expressions.
Tested on i586-suse-linux, applied on the mainline.
2011-03-21 Eric Botcazou ebotca...@adacore.com
* gcc-interface/decl.c (elaborate_expression_1): When
On Mon, Mar 21, 2011 at 12:19 PM, Eric Botcazou ebotca...@adacore.com wrote:
Hi,
the attached patch makes a couple of tweaks to the gimplifier in order to help
Ada, but I think that they are of general usefulness:
1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls. This is
On 2011/3/20 08:11 PM, Richard Guenther wrote:
On Sun, Mar 20, 2011 at 12:01 PM, Chung-Lin Tang
clt...@codesourcery.com wrote:
Hi,
PR48183 is a case where ARM NEON instrinsics, under -O -g, produce debug
insns that tries to expand OImode (32-byte integer) zero constants, much
too large to
On 03/21/2011 01:40 PM, Rainer Orth wrote:
Paolo Bonzinibonz...@gnu.org writes:
On 03/21/2011 12:48 PM, Rainer Orth wrote:
As reported in the PR, the toplevel configure test for libpwl doesn't
work since it fails to link with libstdc++. The following patch
corrects this.
There were a
Mike Stump mikest...@comcast.net writes:
+catch { exec $libtool --mode=clean rm -f $basename.lo }
If libtool runs on the host, this doesn't do that, try:
[remote_exec host ... ]
see gcc.exp for examples.
+catch { exec $libtool --mode=clean rm -f [file rootname [file tail
Hello,
As noted in the PR audit trail, the scheduler crashes for this bug because
we see an SSE insn without proper reservation. Uros has pointed several
such insns to me. The following patch adding athlon_decode attributes is
bootstrapped and tested on x86_64.
Uros, you have preapproved
Richard Guenther richard.guent...@gmail.com writes:
Ok for 4.7, RMs should decide about 4.6.1.
Applied, thanks. It would be nice to get into 4.6 (even 4.6.1) since
otherwise install.texi which claims --disable-symvers as a workaround
lies :-)
Ok.
Just to make no mistake: ok for 4.6.0 or
22_locale/num_put/put/char/14220.cc XPASSes on 64-bit Solaris 10/SPARC
for quite some time, so I'm removing the corresponding xfail.
Tested with the appropriate runtest invocations, installed on 4.5 branch
and mainline. Ok for 4.6, too?
Thanks.
Rainer
2011-03-12 Rainer Orth
On Mon, 21 Mar 2011, Rainer Orth wrote:
Richard Guenther richard.guent...@gmail.com writes:
Ok for 4.7, RMs should decide about 4.6.1.
Applied, thanks. It would be nice to get into 4.6 (even 4.6.1) since
otherwise install.texi which claims --disable-symvers as a workaround
lies :-)
-mfdpic is not supported for the bfin-uclinux target. this patch defines a
macro in the uclinux header and then adds a check to bfin.c if the macro is
defined.
2011-03-21 Stuart Henderson stuart.hender...@analog.com
Originally from Bernd Schmidt
* config/bfin/uclinux.h
Pretty obvious, require at least 1 partition.
Richard.
2011-03-31 Richard Guenther rguent...@suse.de
PR lto/48210
* params.def (lto-partitions): Require at least 1 partition.
Index: gcc/params.def
===
---
Richard Guenther rguent...@suse.de writes:
Index: gcc/params.def
===
--- gcc/params.def(revision 171229)
+++ gcc/params.def(working copy)
@@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE,
DEFPARAM
We force all maybe-trapping conditions into separate stmts, but this
isn't necessary w/o -fno-call-exceptions. It also confuses our
branch prediction.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
The bug remains for -fnon-call-exceptions.
Richard.
2011-03-21
On Sat, Mar 12, 2011 at 01:08:23PM +0100, Eric Botcazou wrote:
gcc/ada/
* gcc-interface/ada-tree.h (union lang_tree_node): Check for
TS_COMMON before calling TREE_CHAIN.
* gcc-interface/misc.c (gnat_init_ts): New function.
(LANG_HOOKS_INIT_TS): Define.
I presume that
On Mon, 21 Mar 2011, Rainer Orth wrote:
Therefore I'll soon commit this patch to obsolete Solaris 8 support in
4.7. It will be removed in 4.8 then.
When obsoleting targets or other features, please update the release notes
(gcc-4.7/changes.html).
--
Joseph S. Myers
jos...@codesourcery.com
Hi Gerald, Hi Jospeh,
The patch below fixes a bunch of minor typos in the invoke.texi file.
I suspect that it might qualify as obvious, but I though that I ought
to run it past you first.
OK to apply ?
Cheers
Nick
gcc/ChangeLog
2011-03-21 Nick Clifton ni...@redhat.com
*
All,
I committed the obvious patch below after asking Jakub.
Andreas
2011-03-21 Andreas Tobler andre...@fgznet.ch
* config/rs6000/t-freebsd: Remove duplication from file.
Index: t-freebsd
===
--- t-freebsd (revision
On Mon, 21 Mar 2011, Joseph S. Myers wrote:
On Mon, 21 Mar 2011, Rainer Orth wrote:
Richard Guenther rguent...@suse.de writes:
Index: gcc/params.def
===
--- gcc/params.def(revision 171229)
+++
Richard Guenther rguent...@suse.de writes:
DEFPARAM (MIN_PARTITION_SIZE,
lto-min-partition,
- Size of minimal paritition for WHOPR (in estimated instructions),
+ Minimal size of a paritition for LTO (in estimated instructions),
Another typo :-)^
On Mon, 21 Mar 2011, Rainer Orth wrote:
Richard Guenther rguent...@suse.de writes:
DEFPARAM (MIN_PARTITION_SIZE,
lto-min-partition,
- Size of minimal paritition for WHOPR (in estimated instructions),
+ Minimal size of a paritition for LTO (in estimated instructions),
H.J. Lu hongjiu...@intel.com writes:
diff --git a/gcc/explow.c b/gcc/explow.c
index 460af1f..9b3c082 100644
--- a/gcc/explow.c
+++ b/gcc/explow.c
@@ -611,6 +611,8 @@ copy_to_reg (rtx x)
rtx
copy_addr_to_reg (rtx x)
{
+ if (GET_MODE (x) != VOIDmode GET_MODE (x) != Pmode)
+x =
This removes odd things from being a valid RHS and transitions the two
easy cases from single to ternary RHS. There are now two codes left
that should IMHO move over to ternary, COND_EXPR and VEC_COND_EXPR.
I'll tackle them separately.
Bootstrapped and tested on x86_64-unknown-linux-gnu,
On Sun, 20 Mar 2011, Jakub Jelinek wrote:
2011-03-20 Jakub Jelinek ja...@redhat.com
PR c/48197
* c-common.c (shorten_compare): If primopN is first sign-extended
to opN and then zero-extended to result type, set primopN to opN.
* gcc.c-torture/execute/pr48197.c:
This test is inspired by PR 48124.
[Jakub, let me know if you find any additional variations on this theme.]
In the C++ memory model, non contiguous bitfields are distinct memory
locations and cannot be accessed by and/or magic if we introduce data races.
In the test below, we can trigger
Joseph S. Myers jos...@codesourcery.com writes:
When obsoleting targets or other features, please update the release notes
(gcc-4.7/changes.html).
Like this? Or should I link to the UI threads removal patch, too?
I don't think such a link is needed.
Ok, I've checked in the patch.
You're right, though; they should probably be marked as such for
completeness. OK to commit with those additions and reordering as above
sight unseen, or would you like to see the patch?
Note that I didn't ask for that, only for a comment explaining why they were
set aside. So it's up to
Hi Eric, this looks like a nice improvement. I noticed this mysterious
undef though - what is it for?
+#undef MOVE_FROM_FIELD_LIST_TO
Not clear what kind of answer you're expecting. To cancel the previous define?
--
Eric Botcazou
1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls. This
is correct since stack memory isn't considered as trapping in the IL.
This is ok.
Thanks.
2) Improve gimplification of complex conditions in COND_EXPR. They are
naturally generated by the Ada compiler and the
On Mon, Mar 21, 2011 at 04:31:31PM +, Joseph S. Myers wrote:
On Sun, 20 Mar 2011, Jakub Jelinek wrote:
2011-03-20 Jakub Jelinek ja...@redhat.com
PR c/48197
* c-common.c (shorten_compare): If primopN is first sign-extended
to opN and then zero-extended to result
2011-03-21 Eric Botcazou ebotca...@adacore.com
* gcc-interface/decl.c (elaborate_expression_1): When optimization is
disabled, use the variable for bounds of loop iteration scheme.
This quite mysteriously triggers a bogus -Wuninitialized warning on IRIX.
Fixed thusly, tested on
On Mar 21, 2011, at 5:59 AM, Rainer Orth wrote:
I've applied those changes, and also managed to fix the problems I had
in Tru64 UNIX testing. Here's the patch I'd like to commit to 4.7.
+# Build shared library from LIB.
+proc boehm-gc-build-shlib { lib flags extra-flags } {
+# FIXME:
Mike Stump mikest...@comcast.net writes:
On Mar 21, 2011, at 5:59 AM, Rainer Orth wrote:
I've applied those changes, and also managed to fix the problems I had
in Tru64 UNIX testing. Here's the patch I'd like to commit to 4.7.
+# Build shared library from LIB.
+proc boehm-gc-build-shlib {
On Mon, Mar 14, 2011 at 12:28 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Jan 27, 2011 at 2:40 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Thu, Jan 27, 2011 at 12:12 AM, H.J. Lu hongjiu...@intel.com wrote:
On Tue, Dec 14, 2010 at 05:20:48PM -0800, H.J. Lu wrote:
This patch
Hello!
Attached patch fixes recent bootstrap failure on alpha-pc-linux-gnu in
in dwarf2out_var_location, at dwarf2out.c:21983.
2011-03-21 Uros Bizjak ubiz...@gmail.com
* config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop
between a call and its CALL_ARG_LOCATION
On Mar 21, 2011, at 11:33 AM, Paolo Bonzini wrote:
So, this still seems unportable to me?
These are libtool options.
Ah, that's the part I was missing... Thanks.
On Mon, Mar 21, 2011 at 07:32:14PM +0100, Benjamin Kosnik wrote:
Well, such stricter checking wouldn't be completely painless, e.g. currently
both current baseline_symbols.txt and the ones I've posted don't include the
two TLS: lines, because of the fear that not everyone has TLS enabled.
On Mar 12, 2011, at 1:01 PM, Jack Howarth wrote:
2011-03-12 Jack Howarth howa...@bromo.med.uc.edu
libstdc++-v3/
* testsuite/lib/prune.exp: Prune could not create compact unwind for
warnings.
gcc/
* testsuite/lib/prune.exp: Ditto.
Checked in r171263. I modified the
On 03/19/2011 12:52 PM, Richard Sandiford wrote:
Given the mode stuff above, I've tried to be quite draconian as far
as caller-provided modes go. I think the caller really should know
what mode they're dealing with. The one case where I had to hold
back a bit was create_convert_operand_from,
On 03/21/2011 11:40 AM, Uros Bizjak wrote:
Hello!
Attached patch fixes recent bootstrap failure on alpha-pc-linux-gnu in
in dwarf2out_var_location, at dwarf2out.c:21983.
2011-03-21 Uros Bizjak ubiz...@gmail.com
* config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop
On 21.03.2011 18:12, Uros Bizjak wrote:
2011/3/21 Andrey Belevantseva...@ispras.ru:
Hello,
As noted in the PR audit trail, the scheduler crashes for this bug because
we see an SSE insn without proper reservation. Uros has pointed several
such insns to me. The following patch adding
Hello!
2011-03-21 Uros Bizjak ubiz...@gmail.com
* config/alpha/alpha.md (unspec): New define_c_enum.
(unspecv): Ditto.
Tested on alphaev68-pc-linux-gnu, committed to mainline SVN.
Uros.
Index: alpha.md
===
---
This is my first patch submittal so constructive criticism is appreciated.
This patch causes PowerPC AltiVec local stack arrays 16 bytes to be
aligned at 16 bytes.
2011-01-27 Pat Wellander pwell...@codesourcery.com
* config/rs6000/rs6000.h (LOCAL_ALIGNMENT):
AltiVec local arrays = 16
On Mon, Mar 21, 2011 at 2:54 PM, Wellander, Pat - Code Sourcery
pwell...@codesourcery.com wrote:
This is my first patch submittal so constructive criticism is appreciated.
This patch causes PowerPC AltiVec local stack arrays 16 bytes to be
aligned at 16 bytes.
Wait I thought the alignment of
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
2011-03-20 Rainer Orth r...@cebitec.uni-bielefeld.de
* Makefile.am (CHECK): Use $(GOTESTFLAGS).
* Makefile.in: Regenerate.
Thanks.
Committed.
Ian
On Mon, Mar 21, 2011 at 3:21 PM, Wellander, Pat - Code Sourcery
pwell...@codesourcery.com wrote:
The stack is aligned at 16 bytes with AltiVec (if not, this patch would not
make sense).
Also, all char arrays are aligned at 16 bytes.
This patch causes other local stack arrays = 16 bytes to be
On 3/21/2011 3:26 PM, Andrew Pinski wrote:
On Mon, Mar 21, 2011 at 3:25 PM, Andrew Pinskipins...@gmail.com wrote:
On Mon, Mar 21, 2011 at 3:21 PM, Wellander, Pat - Code Sourcery
pwell...@codesourcery.com wrote:
The stack is aligned at 16 bytes with AltiVec (if not, this patch would not
make
On 03/18/2011 10:11 AM, Richard Guenther wrote:
This tries to extend the previously posted CCP folding patch by
introducing a generic interface for non-tree-building, GIMPLE SSA
aware folding. The low-level interface for folding regular
operations is
/* Fold the expression composed by *CODEP,
This patch fixes a problem discovered by compiling the boost libraries with VSX
enabled. In the middle of a template is the code:
namespace boost
{
// ...
template typename Iterator
struct apply
{
// ...
typedef
I noticed that dg.gcc/torture/vector-1.c and dg.gcc/torture/vector-2.c were
failing on 32-bit PowerPC Linux systems. The failure was due to the compiler
complaining that by default the ABI on 32-bit does not allow passing vector
items. I fixed the tests so that they pass -mabi=altivec on powerpc
On Mon, 21 Mar 2011, Nick Clifton wrote:
2011-03-21 Nick Clifton ni...@redhat.com
* doc/invoke.texi (Overall Options): Move closing brace to end of
options list.
(Optimization Options): Add missing @gol.
(Directory Options): Likewise.
(i386 and x86-64
Ping
On 03/19/2011 09:46 PM, Jerry DeLisle wrote:
Hi,
The attached patch re-implements read_x using fbuf_getc. Self explanatory.
Regression tested on x86-64 linux. No new test case needed.
Ok for trunk?
Regards,
Jerry
2011-03-19 Jerry DeLisle jvdeli...@gcc.gnu.org
PR libgfortran/48030
*
74 matches
Mail list logo