This is one thing I noticed when removing referenced vars:
Index: trunk/gcc/tree-cfg.c
===
*** trunk.orig/gcc/tree-cfg.c 2012-08-02 10:13:52.0 +0200
--- trunk/gcc/tree-cfg.c2012-08-02 10:16:24.593266230 +0200
It's now a wrapper around create_tmp_reg.
Bootstrap and regtest pending on x86_64-unknown-linux-gnu.
Richard.
2012-08-02 Richard Guenther rguent...@suse.de
* tree-flow.h (make_rename_temp): Remove.
* tree-dfa.c (make_rename_temp): Likewise.
* cgraphunit.c
On Wed, Aug 1, 2012 at 9:21 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
an opinion on this?
(I just noticed: I'll update the list in the comment visible at the top of
the patch if this gets in).
It looks ok to me but I am no floating-point expert. Can you add a testcase?
Ok with
On Tue, Jul 31, 2012 at 3:47 PM, Tobias Burnus bur...@net-b.de wrote:
On 07/29/2012 11:24 AM, Thomas Koenig wrote:
here is an updated patch for PR 54033, this time with test cases.
Thanks to Janis for pointing me in the right direction with these.
Regression-tested. OK for trunk?
Ok.
On Wed, 1 Aug 2012 14:43:33 -0700
Roland McGrath mcgra...@google.com wrote:
Using e.g. -falign-labels=16 on ARM can confuse the constant-pool
layout code such that it places pool entries too far away from their
referring instructions. This change seems to fix it.
I don't have a small test
Patch was regtested on x86_64-unknown-linux-gnu. Ok for trunk?
Looks OK. Thanks!
Thanks for the review. Committed as r190069.
Cheers,
Janus
2012-08-01 Janus Weil ja...@gcc.gnu.org
PR fortran/54147
* resolve.c (check_proc_interface): New routine for PROCEDURE
Important: Your investment can get you an extra $10,000 by Friday with only
$300 purchase of blockbuster symbol!
V_ND_B is a pioneer in manufacturing of battaries for mobiles it gained a
lot of interest in the last couple of day, investors are spending.
We expect V_ND_B gainining 60% in the
On Thu, 2 Aug 2012 09:57:40 +0100
Julian Brown jul...@codesourcery.com wrote:
On Wed, 1 Aug 2012 14:43:33 -0700
Roland McGrath mcgra...@google.com wrote:
Using e.g. -falign-labels=16 on ARM can confuse the constant-pool
layout code such that it places pool entries too far away from their
Hi,
On 08/02/2012 01:53 AM, Peter Bergner wrote:
So just to be sure, like the patch below?
More or less. See comments below.
Also, is this ok for the 4.6 and 4.7 release branches?
I don't think this is a regression, thus I would say 4.7 only, to be safe.
Peter
libstdc++-v3/
PR
On 08/02/2012 10:54 AM, Richard Guenther wrote:
Seems to break testing, all testcases emit
Warning: Nonexistent include directory finclude^M
now and thus all testcases fail like
FAIL: gfortran.dg/alloc_comp_basics_1.f90 -O0 (test for excess errors)
First, I actually wonder why it only
On Thu, 2 Aug 2012, Tom de Vries wrote:
On 01/08/12 10:47, Richard Guenther wrote:
* tree-ssa-tail-merge.c (release_last_vdef): Remove.
(replace_block_by): Adjust.
(tail_merge_optimize): Use mark_virtual_operands_for_renaming.
Richard,
this caused PR50672 to trigger
Extended regcprop to check and remove for redundant move instructions
resulting from the pass.
Paulo.
2012-08-02 Paulo Matos paulo.ma...@csr.com
* regcprop.c (copy_value): remove check for redundant moves.
* regcprop.c (copy_value): add check for redundant moves,
Forgot to mention: this is to fix PR 54154.
Updated changelog:
2012-08-02 Paulo Matos paulo.ma...@csr.com
PR middle-end/54154
* regcprop.c (copy_value): remove check for redundant moves.
* regcprop.c (copy_value): add check for redundant moves,
remove instructions if
This patch works passed profiledbootstrap with LTO as well as LTO -O3
on 176.gcc in SPEC CPU 2000. I have to add 2 inline_edge_summary_vec
checks to avoid ICE. OK to install?
Thanks, it looks good. I am just concerned about...
diff --git a/gcc/ipa-split.c b/gcc/ipa-split.c
index
On Thu, Aug 2, 2012 at 12:17 PM, Paulo J. Matos paulo at matos-sorge
dot com wrote:
Extended regcprop to check and remove for redundant move instructions
resulting from the pass.
Paulo.
2012-08-02 Paulo Matos Paulo dot Matos at csr dot com
* regcprop.c (copy_value): remove check
On Thu, Aug 2, 2012 at 12:19 PM, Paulo J. Matos pa...@matos-sorge.com wrote:
Forgot to mention: this is to fix PR 54154.
Updated changelog:
2012-08-02 Paulo Matos paulo.ma...@csr.com
PR middle-end/54154
* regcprop.c (copy_value): remove check for redundant moves.
*
Ulrich Weigand wrote:
The following patch implements this idea; it passes a basic regression
test on arm-linux-gnueabi. (Obviously this would need a lot more
testing on various platforms before getting into mainline ...)
Can you have a look whether this fixes the problem you're seeing?
Omit TARGET_LIB_PATH from RPATH_ENVVAR in HOST_EXPORTS on bootstrap builds.
Discussion and rationale at: http://gcc.gnu.org/ml/gcc/2012-06/msg00314.html
For google/main. Tested for bootstrap and regression.
2012-08-02 Simon Baldwin sim...@google.com
* Makefile.tpl: Omit
Hello,
this patch allows p-~T() when T is (after substitution) a vector, which
is necessary for use in std::vector for instance.
gcc/cp/ChangeLog
2012-08-02 Marc Glisse marc.gli...@inria.fr
* pt.c (tsubst_copy_and_build): Handle VECTOR_TYPE like scalars.
gcc/testsuite/ChangeLog
On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
this patch allows p-~T() when T is (after substitution) a vector, which is
necessary for use in std::vector for instance.
Why not include VECTOR_TYPE in ARITHMETIC_TYPE_P?
gcc/cp/ChangeLog
2012-08-02 Marc
On 08/02/12 13:08, Richard Guenther wrote:
On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
this patch allows p-~T() when T is (after substitution) a vector, which is
necessary for use in std::vector for instance.
Why not include VECTOR_TYPE in
On Thu, 2 Aug 2012, Richard Guenther wrote:
On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
this patch allows p-~T() when T is (after substitution) a vector, which is
necessary for use in std::vector for instance.
Why not include VECTOR_TYPE in
On Thu, 2 Aug 2012, Richard Guenther wrote:
On Wed, Aug 1, 2012 at 9:21 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
an opinion on this?
(I just noticed: I'll update the list in the comment visible at the top of
the patch if this gets in).
It looks ok to me but I am no floating-point
Thanks for the comments, I will be sending a new patch and fixed changelog.
On 02/08/12 11:27, Richard Guenther wrote:
On Thu, Aug 2, 2012 at 12:19 PM, Paulo J. Matos pa...@matos-sorge.com wrote:
Forgot to mention: this is to fix PR 54154.
Updated changelog:
2012-08-02 Paulo Matos
On Thu, Aug 2, 2012 at 3:23 AM, Jan Hubicka hubi...@ucw.cz wrote:
This patch works passed profiledbootstrap with LTO as well as LTO -O3
on 176.gcc in SPEC CPU 2000. I have to add 2 inline_edge_summary_vec
checks to avoid ICE. OK to install?
Thanks, it looks good. I am just concerned
On Thu, Aug 2, 2012 at 2:48 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Thu, 2 Aug 2012, Richard Guenther wrote:
On Wed, Aug 1, 2012 at 9:21 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
an opinion on this?
(I just noticed: I'll update the list in the comment visible at the top
On 02/08/12 11:25, Steven Bosscher wrote:
Hello,
Thanks for working on this.
How did you test this?
This patch is for GCC46. The main problem is that I was not able to
reproduce it (yet) for any upstream backends. I therefore patched GCC46
and tested our backend (where I can easily
2012-08-02 Paulo Matos paulo.ma...@csr.com
PR middle-end/54154
* regcprop.c (copy_value): remove check for redundant moves.
* regcprop.c (copyprop_hardreg_forward_1): add check for redundant
moves, remove instructions if redundant.
---
On Thu, Aug 02, 2012 at 02:48:08PM +0200, Marc Glisse wrote:
I am redoing the bootstrap+regtest, then I'll commit if I don't hear
protests about the testcase.
gcc/ChangeLog
2012-06-15 Marc Glisse marc.gli...@inria.fr
PR tree-optimization/53805
* fold-const.c
This removes some of the redundant hash lookups for the decl aux info.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-08-02 Richard Guenther rguent...@suse.de
* tree-into-ssa.c (struct common_info_d): New struct.
(struct var_info_d, struct
On Thu, Aug 2, 2012 at 7:08 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
this patch allows p-~T() when T is (after substitution) a vector, which is
necessary for use in std::vector for instance.
Why not
Hello,
as promised, I submit the patch (split to 5 pieces) adding support for
assumed rank actual arguments to the LBOUND/UBOUND/SHAPE intrinsics in
the non-scalar case (without the DIM argument).
Patch 1: Disable shape setting and simplification for assumed rank.
Patch 2: Move and rename
On 08/02/2012 07:42 AM, Marc Glisse wrote:
+ if (SCALAR_TYPE_P (object_type)
+ || TREE_CODE (object_type) == VECTOR_TYPE)
You can use scalarish_type_p for this test. OK with that change.
Jason
Hello Mikael,
thanks for your patch series, which not only implements missing parts of
the assumed-rank support but also fixes some buglets such as the
set_loop_bounds setting or the shape setting for assumed ranks. The
patch is okay.
Tobias
On 08/02/2012 04:17 PM, Mikael Morin wrote:
nick clifton wrote:
Hi Johann,
Hi, just out of curiosity: Is there a special reason to add a
stormy16-specific implementation instead of using the code in
libgcc2.c? t-avr for example uses the C implementation in libgcc2.c
for HI. Are there disadvantages?
Apart from not using some slightly
On Thu, 2 Aug 2012, Nathan Froyd wrote:
On Thu, Aug 02, 2012 at 02:48:08PM +0200, Marc Glisse wrote:
I am redoing the bootstrap+regtest, then I'll commit if I don't hear
protests about the testcase.
gcc/ChangeLog
2012-06-15 Marc Glisse marc.gli...@inria.fr
PR
On Thu, 2 Aug 2012, Jason Merrill wrote:
On 08/02/2012 07:42 AM, Marc Glisse wrote:
+ if (SCALAR_TYPE_P (object_type)
+ || TREE_CODE (object_type) == VECTOR_TYPE)
You can use scalarish_type_p for this test. OK with that change.
Wow, I'd missed it, a function that
On Thu, Aug 02, 2012 at 05:20:24PM +0200, Marc Glisse wrote:
On Thu, 2 Aug 2012, Nathan Froyd wrote:
PR tree-optimization/53805
* fold-const.c (invert_tree_comparison): Do invert ORDERED_EXPR and
UNORDERED_EXPR for floating point.
Minor protest about the ChangeLog: I think you
On 08/01/2012 02:57 PM, Paolo Carlini wrote:
So, it is possible that when spec != NULL_TREE and we are once more in a
SFINAE context, we have to actually call perform_deferred_access_checks
(complain) and either return error_mark_node or the spec depending on
the return value?
I don't think we
Hi,
On 08/02/2012 05:33 PM, Jason Merrill wrote:
On 08/01/2012 02:57 PM, Paolo Carlini wrote:
So, it is possible that when spec != NULL_TREE and we are once more in a
SFINAE context, we have to actually call perform_deferred_access_checks
(complain) and either return error_mark_node or the
This patch adds the ability to spot and exploit the UBFX instruction
(bit-field extract) to perform AND operations with an immediate which is
2^N - 1. On some benchmarks this can lead to a significant win over the
existing behaviour on Cortex-A15.
Tested on arm-eabi with some additional testing
On Wed, Jul 25, 2012 at 2:33 PM, Uros Bizjak ubiz...@gmail.com wrote:
On Wed, Jul 25, 2012 at 1:45 PM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Jul 24, 2012 at 9:32 PM, Uros Bizjak ubiz...@gmail.com wrote:
Hello!
Attached patch backports recent LEA improvements to 4.7
On Thu, Aug 2, 2012 at 3:23 AM, Jan Hubicka hubi...@ucw.cz wrote:
This patch works passed profiledbootstrap with LTO as well as LTO -O3
on 176.gcc in SPEC CPU 2000. I have to add 2 inline_edge_summary_vec
checks to avoid ICE. OK to install?
Thanks, it looks good. I am just concerned
On 08/02/2012 12:21 PM, Paolo Carlini wrote:
+ if (complain tf_error)
+ recheck_decl_substitution (spec, gen_tmpl, targ_ptr);
+ else
+ return error_mark_node;
I think we want to be consistent with the end of the function about
whether we return
On Tue, Jul 31, 2012 at 11:06 AM, Sandeep Soni soni.sande...@gmail.com wrote:
+/* The syntax of a function declaration is as follows:
+
+ FUNCTION_DECLName, Type, Parms
+
+ function body
+
+
+ Here, each of the PARMS in itself is a parameter declaration similar to a
+
On 08/02/2012 06:53 PM, Jason Merrill wrote:
On 08/02/2012 12:21 PM, Paolo Carlini wrote:
+ if (complain tf_error)
+recheck_decl_substitution (spec, gen_tmpl, targ_ptr);
+ else
+return error_mark_node;
I think we want to be consistent with the end of the function
On Tue, Jul 10, 2012 at 10:21 PM, Sandeep Soni soni.sande...@gmail.com wrote:
-static void
+static tree
gp_parse_expect_false_label (gimple_parser *parser)
{
gl_consume_expected_token (parser-lexer, CPP_LESS);
gl_consume_expected_token (parser-lexer, CPP_NAME);
Looks good.
Original Message
From: Paolo Carlini paolo.carl...@oracle.com
Sent: Thu, Aug 2, 2012 01:09 PM
To: Jason Merrill ja...@redhat.com
CC: gcc-patches@gcc.gnu.org
Subject: Re: [RFC / RFH] Re-opened C++/51213 (access control under SFINAE)
On 08/02/2012 06:53 PM, Jason
On Wed, Aug 1, 2012 at 12:14 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Wed, Aug 1, 2012 at 11:58 AM, Richard Sandiford
rdsandif...@googlemail.com wrote:
H.J. Lu hongjiu...@intel.com writes:
We have
(gdb) r -fpreprocessed x.i -quiet -dumpbase x.i -mx32
-maddress-mode=long -mtune=generic
Hi,
this patch uses the aggregate jump functions created by the previous
patch in the series to determine benefits of inlining a particular
call graph edge. It has not changed much since the last time I posted
it, except for the presence of by_ref flags and removal of checks
required by TBAA
Hi Tobias,
I am not sure whether it is the best solution, but one possibility would
be to ignore -fintrinsic-modules-path for the warning. (That assumes
that the warning is (almost) never needed for an installed compiler.)
I think this is the right approach. The attached patch does this.
On Thu, 2 Aug 2012, Jason Merrill wrote:
On 08/02/2012 07:42 AM, Marc Glisse wrote:
+ if (SCALAR_TYPE_P (object_type)
+ || TREE_CODE (object_type) == VECTOR_TYPE)
You can use scalarish_type_p for this test. OK with that change.
That function was static, so I had to
I hit a case where we were splitting a set of (const (unspec [...]))
into something that still referred to the original expression.
We reused that same expression as a REG_EQUAL note, leading to
an rtx-sharing violation.
Tested on mips64-linux-gnu and applied.
Richard
gcc/
* recog.c
Sorry for the late reply. Been thinking it over a bit more.
Moore, Catherine catherine_mo...@mentor.com writes:
Now that we are in the window for 4.8, I'd like to discuss the
possibility of applying this patch. Have you had a chance to think
about it?
In the end, I think we should go for
Hello,
here is a patch handling multiplication of wrapping integer types in VRP.
It passed bootstrap+testsuite limited to c,c++ and without the testcase,
so I'll retest it better during the night. For some reason the test
libgomp.graphite/force-parallel-6.c which used to fail passed with the
On 1 August 2012 20:34, François Dumont wrote:
Ok for trunk ?
OK, thanks.
This patch changes the 24k/74k scheduling descriptions to use the existing
mips_store_data_bypass_p predicate instead of treating cprestore as a special
case. OK for mainline?
-Sandra
2012-08-02 Sandra Loosemore san...@codesourcery.com
Maxim Kuvyrkov ma...@codesourcery.com
bootstrapped and regression tested on x86-64.
committed as revision 190105
2012-08-02 Kenneth Zadeck zad...@naturalbridge.com
* cfgexpand.c (expand_debug_locations): Encapsulate test for
CONST_DOUBLE in macro.
* combine.c (try_combine, gen_lowpart_for_combine): Ditto.
*
This is another patch that has been present in our local source base for some
years now. It originally came from MIPS; I've verified that we have legal
permission to contribute it to the FSF.
The 74k.md parts of this patch depend on the not-yet-reviewed 74k madd
scheduler
tweaks patch I
On Thu, Aug 2, 2012 at 1:57 AM, Julian Brown jul...@codesourcery.com wrote:
FWIW, I've hit this issue in the past, and used a patch as follows to
fix it:
@@ -12015,7 +12025,10 @@ create_fix_barrier (Mfix *fix, HOST_WIDE
gcc_assert (GET_CODE (from) != BARRIER);
/* Count the
As suggested a few weeks ago on the libstdc++ list, this patch tweaks
the include order in memory, allowing internal library headers to
use shared_ptr and unique_ptr without depending on
backward/auto_ptr.h.
* include/std/memory: Include auto_ptr.h later.
*
mep uses {constraints,predicates}.md and #if 0'd out the old macro versions.
In the interest of not keeping dead code around, I'd like to delete these.
I think this falls under the obvious rule, so I'll commit this in a couple of
days after waiting to hear from DJ.
Tested by building mep-elf.
That's fine with me.
As $SUBJECT says. There's not too much interesting here. I did a
fairly literal-minded conversion, so it's possible there's smarter ways
to do some things.
Compiled with cross to mmix-knuth-mmixware and spot-checked by comparing
libgcc object files. I have no idea how to set up a simulator
On 2012-08-02 02:03, Alan Modra wrote:
r190051 of a powerpc-linux bootstrap dies trying to compile garbage
in stage2 gtype-desc.c, r190050 is OK.
It took a while, but I found the problem. The fact that the prologue
generation code didn't use the same alignment that we applied at in
function.c.
Revision http://gcc.gnu.org/viewcvs?view=revisionrevision=189772
added constant Support_Atomic_Primitives to the target specific
system-*.ads files. The default system.ads was not updated.
The included patch
- adds the missing constant
- removes constants not present in other system-*.ads files
Patch courtesy Richard Smith at Google:
Fix bug in the implementation of std::map's operator[]. Construct an
object of type value_type, rather than using std::make_pair, so as to
allow mapped_type to have an *explicit* copy constructor.
See [map.access] (23.4.4.3)/5 for the corresponding
On Thu, 02 Aug 2012 11:36:42 Paolo Carlini wrote:
Also, is this ok for the 4.6 and 4.7 release branches?
I don't think this is a regression, thus I would say 4.7 only, to be safe.
Ok.
we don't have a separate ChangeLog in libstdc++-v3/testsuite/. Thus a
single ChangeLog entry for
68 matches
Mail list logo