On 08/01/2012 05:48 PM, Janus Weil wrote:
here is a follow-up patch to my recent fix for PR 42418. Shortly after
committing it, I realized that the same checks should be done not only
for plain PROCEDURE declarations, but also for procedure-pointer
components and (deferred) type-bound procedures.
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
* tree-flow.h (make_rename_temp): Remove.
* tree-dfa.c (make_rename_temp): Likewise.
* cgraphunit.c (thunk_adjust, assemble_thunk
On Wed, Aug 1, 2012 at 9:21 PM, Marc Glisse 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 that change.
Tha
On Tue, Jul 31, 2012 at 3:47 PM, Tobias Burnus 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. Thanks fo
On Wed, 1 Aug 2012 14:43:33 -0700
Roland McGrath 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 case, only a la
>> 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
>>
>> PR fortran/54147
>> * resolve.c (check_proc_interface): New routine for PROCEDURE
>> interface
On Tue, Jul 31, 2012 at 07:38:53PM -0700, Richard Henderson wrote:
> I've bootstrapped this on both ppc64 and x86_64. I'll leave some time
> for comment, but I can't immediately see how this could go wrong anywhere.
r190051 of a powerpc-linux bootstrap dies trying to compile garbage
in stage2 gty
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 n
On Thu, 2 Aug 2012 09:57:40 +0100
Julian Brown wrote:
> On Wed, 1 Aug 2012 14:43:33 -0700
> Roland McGrath 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 ch
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 libs
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 li
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 tri
Extended regcprop to check and remove for redundant move instructions
resulting from the pass.
Paulo.
2012-08-02 Paulo Matos
* regcprop.c (copy_value): remove check for redundant moves.
* regcprop.c (copy_value): add check for redundant moves,
remove instructions if
Forgot to mention: this is to fix PR 54154.
Updated changelog:
2012-08-02 Paulo Matos
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 redundant.
>
> 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 33c
On Thu, Aug 2, 2012 at 12:17 PM, Paulo J. Matos wrote:
> Extended regcprop to check and remove for redundant move instructions
> resulting from the pass.
>
> Paulo.
>
> 2012-08-02 Paulo Matos
>
> * regcprop.c (copy_value): remove check for redundant moves.
> * regcprop.c (copy_v
On Thu, Aug 2, 2012 at 12:19 PM, Paulo J. Matos wrote:
> Forgot to mention: this is to fix PR 54154.
>
> Updated changelog:
>
> 2012-08-02 Paulo Matos
>
> PR middle-end/54154
>
> * regcprop.c (copy_value): remove check for redundant moves.
> * regcprop.c (copy_value): add check for
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?
Sorr
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
* Makefile.tpl: Omit TARGET_LIB_PATH from RPATH_ENV
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
* pt.c (tsubst_copy_and_build): Handle VECTOR_TYPE like scalars.
gcc/testsuite/ChangeLog
2012-08-02 Marc Gliss
On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse 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 Glisse
>
>
On 08/02/12 13:08, Richard Guenther wrote:
On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse 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?
Not really, beca
On Thu, 2 Aug 2012, Richard Guenther wrote:
On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse 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?
I often aim fo
On Thu, 2 Aug 2012, Richard Guenther wrote:
On Wed, Aug 1, 2012 at 9:21 PM, Marc Glisse 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
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 wrote:
Forgot to mention: this is to fix PR 54154.
Updated changelog:
2012-08-02 Paulo Matos
PR middle-end/54154
On Thu, Aug 2, 2012 at 3:23 AM, Jan Hubicka 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 about...
>>
On Thu, Aug 2, 2012 at 2:48 PM, Marc Glisse wrote:
> On Thu, 2 Aug 2012, Richard Guenther wrote:
>
>> On Wed, Aug 1, 2012 at 9:21 PM, Marc Glisse wrote:
>>>
>>> Hello,
>>>
>>> an opinion on this?
>>>
>>> (I just noticed: I'll update the list in the comment visible at the top
>>> of
>>> the patch
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 reprodu
2012-08-02 Paulo Matos
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.
--- //depot/bc/main/devHost/gcc46/gcc/gcc/regcprop.c 2011-09-06 1
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
>
> PR tree-optimization/53805
> * fold-const.c (invert_tree_comparison): Do
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
* tree-into-ssa.c (struct common_info_d): New struct.
(struct var_info_d, struct ssa_name_info): Use it.
On Thu, Aug 2, 2012 at 7:08 AM, Richard Guenther
wrote:
> On Thu, Aug 2, 2012 at 1:42 PM, Marc Glisse 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_
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 get_ran
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:
Hell
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 som
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
PR tree-optimization/53805
* fold-cons
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:
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 spec
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 o
On Wed, Jul 25, 2012 at 2:33 PM, Uros Bizjak wrote:
> On Wed, Jul 25, 2012 at 1:45 PM, Richard Guenther
> wrote:
>> On Tue, Jul 24, 2012 at 9:32 PM, Uros Bizjak wrote:
>>> Hello!
>>>
>>> Attached patch backports recent LEA improvements to 4.7 branch.
>>
>> As in regression fixes, or wrong-code
> On Thu, Aug 2, 2012 at 3:23 AM, Jan Hubicka 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 concern
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 error_mark_
On Tue, Jul 31, 2012 at 11:06 AM, Sandeep Soni wrote:
> +/* The syntax of a function declaration is as follows:
> +
> + FUNCTION_DECL
> + <
> + function body
> + >
> +
> + Here, each of the PARMS in itself is a parameter declaration similar to a
> + variable declaration, TYPE is th
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 ab
On Tue, Jul 10, 2012 at 10:21 PM, Sandeep Soni 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);
>gl_consume_expected_token (parse
Looks good.
Original Message
From: Paolo Carlini
Sent: Thu, Aug 2, 2012 01:09 PM
To: Jason Merrill
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 Merrill wrote:
> On 08/02/2012 12:21 PM,
On Wed, Aug 1, 2012 at 12:14 PM, H.J. Lu wrote:
> On Wed, Aug 1, 2012 at 11:58 AM, Richard Sandiford
> wrote:
>> "H.J. Lu" writes:
>>> We have
>>>
>>> (gdb) r -fpreprocessed x.i -quiet -dumpbase x.i -mx32
>>> -maddress-mode=long -mtune=generic -march=x86-64 -auxbase x -O2 -version
>>> -ftree-vec
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 whic
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.
Re
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 t
This patch fixes a disagreement about the form of REG_BR_PRED.
I came across it as an rtl-checking ICE when inverting a jump
after dbr_schedule.
REG_BR_PRED used to be just a single int (and is still documented
that way, although I'm going to chicken out of fixing that).
dbr_schedule created the n
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 (sp
Sorry for the late reply. Been thinking it over a bit more.
"Moore, Catherine" 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 the fix originally sugge
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
Maxim Kuvyrkov
Julian Brown
gcc/
bootstrapped and regression tested on x86-64.
committed as revision 190105
2012-08-02 Kenneth Zadeck
* cfgexpand.c (expand_debug_locations): Encapsulate test for
CONST_DOUBLE in macro.
* combine.c (try_combine, gen_lowpart_for_combine): Ditto.
* cprop.c (implicit_set_cond_p):
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 post
On Thu, Aug 2, 2012 at 1:57 AM, Julian Brown 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 length of this insn.
As suggested a few weeks ago on the libstdc++ list, this patch tweaks
the include order in , allowing internal library headers to
use shared_ptr and unique_ptr without depending on
.
* include/std/memory: Include auto_ptr.h later.
* include/bits/shared_ptr.h (shared_ptr(auto_ptr<_U
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.
-Na
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
envi
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=revision&revision=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 standard
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 ever
> Revision http://gcc.gnu.org/viewcvs?view=revision&revision=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
Hello!
We can implement atomic_fetch_sub with atomic_fetch_add and inverted
operand 2. However, we have to prevent overflows with negation, so
only const_int operand 2 is allowed in the expander.
2012-08-02 Uros Bizjak
PR target/54087
* config/i386/sync.md (atomic_fetch_sub):
73 matches
Mail list logo
