On 09/22/2011 08:18 PM, Rainer Orth wrote:
[build] Move gthr to toplevel libgcc
http://gcc.gnu.org/ml/gcc-patches/2011-08/msg00762.html
Can you post an updated patch for this one? I'll try to review the
others as soon as possible.
Paolo
2011/9/22 Georg-Johann Lay :
> This patch adds the PLUS part to fix the PR.
>
> addsi3 has a 8-bit scratch register now so that constants that are not covered
> by the constraints won't force a reload of the constant.
>
> The output routine tries adding the constant and subtracting the negated
> co
2011/9/23 Richard Sandiford
>
> Thanks as always for the review.
Sure, always a pleasure.
>
> Ayal Zaks writes:
> >> Richard Sandiford wrote on 30/08/2011
> >> 03:29:26 PM:
> >>
> >> > From: Richard Sandiford
> >> > To: gcc-patches@gcc.gnu.org
> >> > Cc: Ayal Zaks/Haifa/IBM@IBMIL
> >> > Date
On Fri, Sep 16, 2011 at 01:08:13AM +0200, Mikael Morin wrote:
> Hello,
>
> the scalarizer is there to generate loops for assignments over more than
> one element. Tobias extended it at various places to support coarrays,
> but this should not be necessary as coarrays in assignments either refer
>
Ping.
BR,
Terry
> -Original Message-
> From: Terry Guo [mailto:terry@arm.com]
> Sent: Tuesday, September 20, 2011 9:52 AM
> To: gcc-patches@gcc.gnu.org
> Subject: [PATCH] Make the pass of 941014-1.c as a normal pass.
>
> Hello,
>
> This is an old case written in 1997. Perhaps it int
> Ping?
>
> http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01626.html
>
> -cary
>
>
> include/ChangeLog:
>
> PR 40831
> * demangle.h (enum demangle_component_type): Add
> DEMANGLE_COMPONENT_CLONE.
>
> libiberty/ChangeLog:
>
> PR 40831
> * cp-demangle.c (d_make_comp)
OK for trunk and 4.6.
Jason
On Sep 1, 2011, at 6:20 AM, Bernd Schmidt wrote:
> The following change
>
> 2011-05-26 Rainer Orth
> PR gcov-profile/48845
>
> causes testsuite failures on C6X.
I was hoping that Rainer would comment... or a profiling person, or an avr
person
> Ok?
Ok. Watch for any hate
On 9/21/11, dnovi...@google.com wrote:
> http://codereview.appspot.com/5090041/diff/1/gcc/cp/pph-streamer-in.c
> File gcc/cp/pph-streamer-in.c (right):
>
> http://codereview.appspot.com/5090041/diff/1/gcc/cp/pph-streamer-in.c#newcode2146
> gcc/cp/pph-streamer-in.c:2146: pph_read_namespace_chain (p
On 09/22/11 08:00, Tristan Gingold wrote:
Hi,
this patch adds some fixes in fixincludes specific to VMS.
Until now we were using a manually modified subset of the VMS headers, but
using fix includes is the right way.
Ok for mainline ?
Looks good to me. Please, thank you.
Hi,
the below one-liner appears to fix this regression: to be honest, I
cannot say to have fully analyzed the issue, but testsuite is fine on
x86_64-linux, both mainline and 4_6-branch (indeed the latter is also
affected, 4_4-branch isn't)
Is it Ok? In case, 4_6-branch too?
Thanks,
Paolo.
ok for google branches.
(Did a little digging -- the remove pass is added because ipa-inline
did not do a good job updating the call graph so there might be some
inconsistency. However the affinity information needs to be computed
after inline transformation, so some more work may be needed in the
Forgot to mention the changes:
* cgraphbuild.c (remove_cgraph_callee_edges): Preserve callee edges if
callgraph profiles are needed.
* final.c (rest_of_handle_final): Rename .note.callgraph sections as
.gnu.callgraph sections.
* tree-optimize.c (gate_all_opt
This patch preserves cgraph callee edges till pass_final if callgraph
edge profiles sections are requested. It also renames callgraph edge
profile sections to be .gnu.callgraph instead of .note.callgraph
Index: cgraphbuild.c
===
---
On 09/22/2011 05:11 PM, Fabien Chêne wrote:
2011/9/22 Jason Merrill:
I don't, it just seemed strange to handle functions differently from other
decls here. But when I look more closely I see that we're in
lookup_field_1, which isn't interested in functions, so I guess we do want
to ignore fun
On Wed, Sep 7, 2011 at 10:01 PM, Jakub Jelinek wrote:
> On Wed, Sep 07, 2011 at 09:54:03PM +0200, Uros Bizjak wrote:
>> > 2011-09-07 Jakub Jelinek
>> >
>> > PR target/50310
>> > * config/i386/i386.c (ix86_prepare_sse_fp_compare_args): For
>> > TARGET_AVX return code for LTG
Hi!
The PR50482 testcase shows that we don't optimize logical vector ops
with all ones vectors very well.
The following patch implements that optimization it, bootstrapped/regtested
on x86_64-linux and i686-linux, ok for trunk?
2011-09-22 Jakub Jelinek
* rtlanal.c (all_ones_cst): New
2011/9/22 Jason Merrill :
> On 09/22/2011 04:22 AM, Fabien Chêne wrote:
>>
>> I would have thought that we want to do something with OVERLOAD here,
>> in order to get rid of PR c++/30195 and c++/25994 (removing a wrong
>> diagnostic additionaly)... But those PRs are already fixed by this
>> patch w
Thanks as always for the review.
Ayal Zaks writes:
>> Richard Sandiford wrote on 30/08/2011
>> 03:29:26 PM:
>>
>> > From: Richard Sandiford
>> > To: gcc-patches@gcc.gnu.org
>> > Cc: Ayal Zaks/Haifa/IBM@IBMIL
>> > Date: 30/08/2011 03:29 PM
>> > Subject: [4/4] Make SMS schedule register moves
>>
On 09/21/11 19:33, Richard Henderson wrote:
> Why, then, is this the only place in dwarf2cfi that needs to handle
> registers via a loop over nregs? It seems to me that we should either
> be handling multi-register spans everywhere or nowhere.
>
> Because alternately, this could be a bug in your
Hello,
sorry for the slow (yet faster than anyone else ;) review.
I'm a bit surprised that there is no resolve.c or iresolve.c change.
intrinsic.c may cerainly need some modification too.
Same goes for trans-intrinsic.c, but perhaps resolution time support is
sufficient in the library call case
Committed to trunk.
gcc/
* config/sparc/sparc.md (G[0-7]_REG, O[0-7]_REG, L[0-7]_REG,
I[0-7]_REG, F[0-62]_REG, FCC[0-3]_REG, CC_REG, SFP_REG): New
constants. Use them everywhere.
diff --git a/gcc/config/sparc/sparc.md b/gcc/config/sparc/sparc.md
index 812ae7b..588caf3 1
I've committed the following trivial patch to fix typos in ipa-prop.c
dump/debug print function.
--
Maxim Kuvyrkov
CodeSourcery / Mentor Graphics
fsf-gcc-fixes-1.patch
Description: Binary data
fsf-gcc-fixes-1.ChangeLog
Description: Binary data
From: "Joseph S. Myers"
Date: Thu, 22 Sep 2011 19:31:07 + (UTC)
> On Wed, 21 Sep 2011, David Miller wrote:
>
>> Unfortunately, we can't use these for comparisons on vectors as the
>> compiler wants them. Instead of producing a vector of comparison
>> results, these instructions produce a sm
On Wed, 21 Sep 2011, David Miller wrote:
> Unfortunately, we can't use these for comparisons on vectors as the
> compiler wants them. Instead of producing a vector of comparison
> results, these instructions produce a small bitmask of those results
> in a destination integer register. Therefore
> The only failure I had was pr50433, now fixed by revision 179046. With it,
> all the ADA tests pass (including gnat.dg/opt19.adb) pass without failure
> on x86_64-apple-darwin10 (thanks Jan for the quick fix).
That's what I meant. Note that it's Ada, not ADA, because it's a first name:
http:/
On Wed, Sep 21, 2011 at 05:01:31PM -0400, David Miller wrote:
> From: Michael Meissner
> Date: Thu, 15 Sep 2011 14:16:45 -0400
>
> > On the powerpc, we have an issue with Spec 2006 and calculix when FMAs are
> > generated and -ffast-math is used, where line 307 of rubber.f is:
> >
> > tt=dat
On 09/01/11 15:20, Bernd Schmidt wrote:
> The following change
>
> 2011-05-26 Rainer Orth
> PR gcov-profile/48845
>
> causes testsuite failures on C6X. Specifically,
>
> @@ -495,11 +495,16 @@ proc check_profiling_available { test_wh
> # These conditions depend on the argument
It seems that after 9 to 12 months, glibc has finally
got up with FreeBSD's libm. Time to fix the tolerance
test in bessel_6.f90.
2011-09-22 Steven G. Kargl
PR testsuite/50487
* gfortran.dg/bessel_6.f90: Fix tolerance test.
Index: gfortran.dg/bessel_6.f90
==
The following libgcc patches haven't been reviewed for about a month:
CFT: [build] Move crtstuff support to toplevel libgcc
http://gcc.gnu.org/ml/gcc-patches/2011-08/msg01273.html
CFT: [build] Move libgcc1 to toplevel libgcc
http://gcc.gnu.org/ml/gcc-patches/2011-0
OK.
Jason
the __sync expanders were defaulting to issuing normal loads and stores
surrounded by barriers when no pattern existed.
If the type is larger than the word size of the architecture, the load
or store would expand to 2 (or more) loads/stores and not be atomic.
This patch corrects that. In the
This patch adds the PLUS part to fix the PR.
addsi3 has a 8-bit scratch register now so that constants that are not covered
by the constraints won't force a reload of the constant.
The output routine tries adding the constant and subtracting the negated
constant and then chooses the shortest sequ
Hello!
2011-09-22 Uros Bizjak
PR target/50482
* config/i386/i386.c (ix86_expand_sse_movcc): When generating
blendv, force op_true to register if it doesn't satisfy
nonimmediate_operand predicate.
testsuite/ChangeLog:
2011-09-22 Uros Bizjak
PR targe
Hi,
I have this patchlet for a simple C++11 rejects-valid, reported by
Daniel. Tested x86_64-linux.
Ok for mainline?
Thanks,
Paolo.
/cp
2011-09-22 Paolo Carlini
PR c++/50371
* pt.c (invalid_nontype_parm_type_p): Handle NULLPTR_TYPE.
/testsuite
2011-0
Rainer Orth writes:
> Ian Lance Taylor writes:
>
>> History shows that Go library updates sometimes break the Go build on
>> non-GNU/Linux targets. I tried to be careful this time, but please let
>> me know about any new problems.
>
> As expected, this did break Solaris bootstrap and will also
This fixes a reload problem found by Tom de Vries while testing another
patch. An analysis of the problem is in PR50249. Essentially, we have
multiple reloads with the same type, for the same operand, feeding each
other and reusing the same register:
Reload 1: reload_in (SI) = (reg/f:SI 10 sl [588
> Richard Sandiford wrote on 30/08/2011
> 03:29:26 PM:
>
> > From: Richard Sandiford
> > To: gcc-patches@gcc.gnu.org
> > Cc: Ayal Zaks/Haifa/IBM@IBMIL
> > Date: 30/08/2011 03:29 PM
> > Subject: [4/4] Make SMS schedule register moves
> >
> > This is the move-scheduling patch itself. It should be
I have merged mainline revision 179076 onto the gccgo branch.
Ian
On 09/21/2011 09:34 AM, Dodji Seketeli wrote:
Jason Merrill writes:
clobbering the last token when the buffer is full sounds like it's
unlikely to be what the caller wants; should we abort instead?
abort () added in that case.
Please update the comment as well.
+/* An iterator over toke
off the top of my head, maybe something like one of the following:
step-through
gdb-invoked
side-effects
simulate-thread
simulate-thread or step-through sound fine. I'd vote for
simulate-thread, but really-- whatever...
Ian Lance Taylor writes:
> History shows that Go library updates sometimes break the Go build on
> non-GNU/Linux targets. I tried to be careful this time, but please let
> me know about any new problems.
As expected, this did break Solaris bootstrap and will also break IRIX
bootstrap:
* The So
I have rewritten and adapted all the current bitfield tests that are
store data race dependent.
I am open to a better name than "memmodel". If any one has a better
idea, I'm sed competent.
I'd definately suggest something more generic which gives an indication
of whats being done... memmo
On 09/22/2011 04:22 AM, Fabien Chêne wrote:
I would have thought that we want to do something with OVERLOAD here,
in order to get rid of PR c++/30195 and c++/25994 (removing a wrong
diagnostic additionaly)... But those PRs are already fixed by this
patch without doing anything with OVERLOAD. Cons
>> Only request is to document that the register moves are
>> placed/assigned-id's in a specific order.
>
>I suppose this is the downside of splitting the patches up, sorry,
>but the ids are only ordered for the throw-away loop:
>
> FOR_EACH_VEC_ELT_REVERSE (ps_reg_move_info, ps->reg_moves, i, move
Ayal Zaks writes:
> Richard Sandiford wrote on 30/08/2011
> 03:10:50 PM:
>
>> From: Richard Sandiford
>> To: gcc-patches@gcc.gnu.org
>> Cc: Ayal Zaks/Haifa/IBM@IBMIL
>> Date: 30/08/2011 03:10 PM
>> Subject: [3/4] SMS: Record moves in the partial schedule
>
>>
>> This patch adds infrastructure th
Hi,
this patch adds some fixes in fixincludes specific to VMS.
Until now we were using a manually modified subset of the VMS headers, but
using fix includes is the right way.
Ok for mainline ?
Tristan.
fixincludes/
2011-09-22 Tristan Gingold
* inclhack.def (ms_define_can_use_extern
Romain Geissler a écrit:
> The deprecated attribute was just an example. I may need to apply any type
> attribute
> when a new type is parsed, which is currently not possible with the current
> C++
> front-end.
So then it seems to me like you might need to emit an event when a new
declared nam
On 09/21/2011 02:32 PM, Dodji Seketeli wrote:
FWIW, I'd like the LRK_MACRO_PARM_REPLACEMENT name (or its
replacement. ha ha) to hint at the fact that it really has to do with
a token that is an /argument/ for a function-like macro.
I disagree; arguments are the situation when the two locations
Le 22 sept. 2011 à 16:18, Diego Novillo a écrit :
> On 11-09-22 09:40 , Dodji Seketeli wrote:
>> Romain Geissler a écrit:
>>
>>> I tried to fix PLUGIN_FINISH_DECL as well to include typedefs in C++.
>>>
>>> The followings does not currently trigger the PLUGIN_FINISH_DECL
>>> (or not in all cas
Hello,
Le 22 sept. 2011 à 15:22, Dodji Seketeli a écrit :
>
>> So i may need a PLUGIN_FINISH_TYPE_DECLARATION triggered when the type
>> is declared but before it is finally defined.
>
> Hmmh. For this specific case, maybe just setting the TREE_DEPRECATED
> flag on the tree node of type could d
On Sep 22, 2011, at 4:21 PM, Bruce Korb wrote:
> Hi Tristan,
>
> On Thu, Sep 22, 2011 at 6:59 AM, Tristan Gingold wrote:
>> Hi,
>>
>> currently alpha-vms is listed as a particular target in
>> fixincludes/mkfixinc.sh but ia64-vms isn't.
>> As I will submit a patch to add some rules for both a
Hi Tristan,
On Thu, Sep 22, 2011 at 6:59 AM, Tristan Gingold wrote:
> Hi,
>
> currently alpha-vms is listed as a particular target in
> fixincludes/mkfixinc.sh but ia64-vms isn't.
> As I will submit a patch to add some rules for both alpha and ia64 VMS, first
> fixincludes must be enabled on bo
On 11-09-22 09:40 , Dodji Seketeli wrote:
Romain Geissler a écrit:
I tried to fix PLUGIN_FINISH_DECL as well to include typedefs in C++.
The followings does not currently trigger the PLUGIN_FINISH_DECL
(or not in all cases), but should them ?
- function parameters (in the function prototype
Hi,
currently alpha-vms is listed as a particular target in fixincludes/mkfixinc.sh
but ia64-vms isn't.
As I will submit a patch to add some rules for both alpha and ia64 VMS, first
fixincludes must be enabled on both.
Committed on trunk.
Tristan.
fixincludes/
2011-09-22 Tristan Gingold
Romain Geissler a écrit:
> I tried to fix PLUGIN_FINISH_DECL as well to include typedefs in C++.
>
> The followings does not currently trigger the PLUGIN_FINISH_DECL
> (or not in all cases), but should them ?
> - function parameters (in the function prototype)
> - definition (with a function bo
OK.
Jason
Hello Romain,
Sorry for my late reply to this thread.
Romain Geissler a écrit:
>> Just to be sure I understand, do you need to be notified about *uses* of
>> types and decls as well? If so, maybe a new kind of event should
>> probably be defined, because PLUGIN_FINISH_DECL and PLUGIN_FINISH_TY
Andreas Krebbel wrote:
> 2011-09-21 Andreas Krebbel
>
> * reload.c (find_reloads): Set operand_mode to Pmode for address
> operands consisting of just a CONST_INT.
>
>
> 2011-09-21 Andreas Krebbel
>
> * gcc.target/s390/addr-constraints-1.c: New testcase.
This is OK.
T
Hi,
comitted as obvoius.
Honza
* ipa-inline-analysis.c: Fix overly long lines.
Index: ipa-inline-analysis.c
===
--- ipa-inline-analysis.c (revision 179083)
+++ ipa-inline-analysis.c (working copy)
@@ -269,7 +269,8
Hello,
Here is the updated patch. OK now?
Thanks,
Bingfeng
2011-09-22 Bingfeng Mei
* alias.c (mems_in_disjoint_address_spaces_p) New function.
(nonoverlapping_memrefs_p): Use mems_in_disjoint_address_sapces_p
to derive alias information.
(nonoverlapping_memrefs
2011/9/22 Georg-Johann Lay :
> This adds rotate left/right by 1 for HI and SI and rotates by QI.
>
> It's all straight forward. The test case in the PR leads to generation of the
> new rotate patterns.
>
> Passed without regressions.
>
> Ok?
Ok.
Denis.
This adds rotate left/right by 1 for HI and SI and rotates by QI.
It's all straight forward. The test case in the PR leads to generation of the
new rotate patterns.
Passed without regressions.
Ok?
Johann
PR target/50446
* config/avr/avr.md (rotlqi3): Support all offsets 0..7.
Hi,
this patch solves some problems related to jump function computation and
maintenance.
In particular:
1) inliner used to compute jump functions only when indirect inlining is
enabled.
We now use them for predicate evaulation and thus we need to compute them
always
up-to date (when op
Hi:
Unless I'm missing something, the mingw CPP_SPEC changes introduced in
r171833 have a typo: -D_REENTRANCE should read -D_REENTRANT . Patchlet
below. Please review, and apply if it's OK.
config/i386/mingw-w64.h (CPP_SPEC): Rename _REENTRANCE to _REENTRANT.
config/i386/mingw32.h (CPP_SPEC): L
a typo fixed.
K
On Thu, Sep 22, 2011 at 2:28 PM, Kirill Yukhin wrote:
> Hi,
> I've perepared a list of IA-32/x86-64 related changes (for changes.html).
>
> Could you please have a look and if there're no objections commit?
>
> Thanks, K
>
4.7.0-x86-changes.html.www.patch
Description: Binary da
Ramana Radhakrishnan writes:
> On 20 September 2011 09:38, Richard Sandiford
>> Otherwise, all expanders use expandable_comparison_operator instead.
>> This restores the previous behaviour for them, and I went through each
>> one to try to make sure that it was handled correctly.
>>
>> Tested on a
Hi,
I've perepared a list of IA-32/x86-64 related changes (for changes.html).
Could you please have a look and if there're no objections commit?
Thanks, K
4.7.0-x86-changes.html.www.patch
Description: Binary data
>> The patch was regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>>
>
> OK.
Committed as r179080.
Cheers,
Janus
On Thu, 22 Sep 2011, David Miller wrote:
> Positive feedback for the fact that someone is at least working on
> this stuff at all would be appreciated as well.
Using it or working on it? Not that much of either, sorry, but
what I found when using it, I put in PR48974 (well, besides the
ICE's, whi
2011/9/21 Jason Merrill :
> On 09/21/2011 01:59 PM, Fabien Chêne wrote:
if (!DECL_DEPENDENT_P (field))
- continue;
+ {
+ tree using_decl = USING_DECL_DECLS (field);
+ if ((TREE_CODE (using_decl) == FIELD_DECL
+
Hi Steve,
>> The patch was regtested on x86_64-unknown-linux-gnu. Ok for trunk?
>>
>
> OK.
thanks for the review!
> The only nitpicking that I have is with Nonintrinsic in
>
> + gfc_error ("Nonintrinsic elemental procedure '%s' is invalid "
> + "in procedure pointer a
2011/9/21 Georg-Johann Lay :
> This patch improves IOR, XOR, AND for HI and SI.
>
> There is room for improvement in the current implementation:
>
> - better analysis if a scratch is actually needed
> - add clobber operands to some insns
> - use 8-bit scratch instead of reload the constant
> - reus
On Thu, Sep 22, 2011 at 12:25 AM, Joseph S. Myers
wrote:
> Similar to various previous such patches, this patch adds dg-skip-if
> markers to various gcc.target/i386 tests to skip them if the multilib
> options with which the testsuite is run contain -march= options
> conflicting with those in the
Hi,
This patch adds a missing support of constant operands in reduction in SLP.
Bootstrapped and tested on powerpc64-suse-linux.
Committed.
Ira
ChangeLog:
PR tree-optimization/50451
* tree-vect-slp.c (vect_get_constant_vectors): Don't fail for
constant operands in reduc
As pointed out by Hans-Peter Nilsson.
Committed to trunk.
gcc/
* config/sparc/sparc.c (sparc_vis_init_builtins): Do not mark
fpack16, fpack32, fpackfix as const.
diff --git a/gcc/config/sparc/sparc.c b/gcc/config/sparc/sparc.c
index a4917da..d62d5a1 100644
--- a/gcc/config/spar
76 matches
Mail list logo
