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):
> 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
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
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
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
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
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
That's fine with me.
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
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
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.
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
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 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/
On 1 August 2012 20:34, François Dumont wrote:
>
> Ok for trunk ?
OK, thanks.
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
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
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
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
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
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
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
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
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 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
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 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 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 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 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
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
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
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
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 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, 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
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
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
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,
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 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_
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 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
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 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
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 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...
>>
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, 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
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 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, 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
>
>
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
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
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
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
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
>
> 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
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.
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
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
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
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 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
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 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
>> 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 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
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, 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
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
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
**
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.
73 matches
Mail list logo
