From: Hans-Peter Nilsson h...@bitrange.com
Date: Wed, 21 Sep 2011 23:27:38 -0400 (EDT)
Minor inconsistency spotted there: the same goes for the fpack
insns but you now set TREE_READONLY for them. (Not claiming I
caught all of them.)
Thanks a lot for pointing this out, I'll remove the
Eric, could you give this a quick spin on Solaris? Even if you
don't have a FMAF capable cpu available, at least make sure the
configury assembler feature detection bits work properly.
And any other form of feedback is appreciated as well. This patch
works fine for me with current binutils
On 9/02/2011, at 6:53 AM, Martin Jambor wrote:
This patch basically disables all intraprocedural devirtualization
simply because that transformation relies on assumptions that no
longer hold true. That leaves only devirtualization within inlining
and IPA-CP but those do not work
Backport mainline r178852, r172697 to ARM/embedded-4_6-branch
Committed.
2011-09-22 Joey Ye joey...@arm.com
Backport r178852 from mainline
2011-09-14 Julian Brown jul...@codesourcery.com
* config/arm/arm.c (arm_override_options): Add unaligned_access
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
---
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
On Thu, Sep 22, 2011 at 12:25 AM, Joseph S. Myers
jos...@codesourcery.com 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
2011/9/21 Georg-Johann Lay a...@gjlay.de:
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
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 assigment at
2011/9/21 Jason Merrill ja...@redhat.com:
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
+
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,
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
Ramana Radhakrishnan ramana.radhakrish...@linaro.org 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
a typo fixed.
K
On Thu, Sep 22, 2011 at 2:28 PM, Kirill Yukhin kirill.yuk...@gmail.com 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
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):
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
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.
2011/9/22 Georg-Johann Lay a...@gjlay.de:
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.
Hello,
Here is the updated patch. OK now?
Thanks,
Bingfeng
2011-09-22 Bingfeng Mei b...@broadcom.com
* alias.c (mems_in_disjoint_address_spaces_p) New function.
(nonoverlapping_memrefs_p): Use mems_in_disjoint_address_sapces_p
to derive alias information.
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
Andreas Krebbel wrote:
2011-09-21 Andreas Krebbel andreas.kreb...@de.ibm.com
* reload.c (find_reloads): Set operand_mode to Pmode for address
operands consisting of just a CONST_INT.
2011-09-21 Andreas Krebbel andreas.kreb...@de.ibm.com
*
Hello Romain,
Sorry for my late reply to this thread.
Romain Geissler romain.geiss...@st.com 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
OK.
Jason
Romain Geissler romain.geiss...@gmail.com 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
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
On 11-09-22 09:40 , Dodji Seketeli wrote:
Romain Geisslerromain.geiss...@gmail.com 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
Hi Tristan,
On Thu, Sep 22, 2011 at 6:59 AM, Tristan Gingold ging...@adacore.com 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
On Sep 22, 2011, at 4:21 PM, Bruce Korb wrote:
Hi Tristan,
On Thu, Sep 22, 2011 at 6:59 AM, Tristan Gingold ging...@adacore.com 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
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 do what
Le 22 sept. 2011 à 16:18, Diego Novillo a écrit :
On 11-09-22 09:40 , Dodji Seketeli wrote:
Romain Geisslerromain.geiss...@gmail.com 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
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
Romain Geissler romain.geiss...@gmail.com 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
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 ging...@adacore.com
* inclhack.def
Ayal Zaks ayal.z...@gmail.com writes:
Richard Sandiford richard.sandif...@linaro.org wrote on 30/08/2011
03:10:50 PM:
From: Richard Sandiford richard.sandif...@linaro.org
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
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)
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.
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...
Ian Lance Taylor i...@google.com 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
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...
On 09/21/2011 09:34 AM, Dodji Seketeli wrote:
Jason Merrillja...@redhat.com 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
I have merged mainline revision 179076 onto the gccgo branch.
Ian
Richard Sandiford richard.sandif...@linaro.org wrote on 30/08/2011
03:29:26 PM:
From: Richard Sandiford richard.sandif...@linaro.org
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
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
Rainer Orth r...@cebitec.uni-bielefeld.de writes:
Ian Lance Taylor i...@google.com 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
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 paolo.carl...@oracle.com
PR c++/50371
* pt.c (invalid_nontype_parm_type_p): Handle
Hello!
2011-09-22 Uros Bizjak ubiz...@gmail.com
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
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
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
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 ka...@gcc.gnu.org
PR testsuite/50487
* gfortran.dg/bessel_6.f90: Fix tolerance test.
Index: gfortran.dg/bessel_6.f90
On 09/01/11 15:20, Bernd Schmidt wrote:
The following change
2011-05-26 Rainer Orth r...@cebitec.uni-bielefeld.de
PR gcov-profile/48845
causes testsuite failures on C6X. Specifically,
@@ -495,11 +495,16 @@ proc check_profiling_available { test_wh
# These conditions
On Wed, Sep 21, 2011 at 05:01:31PM -0400, David Miller wrote:
From: Michael Meissner meiss...@linux.vnet.ibm.com
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
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:
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 we
From: Joseph S. Myers jos...@codesourcery.com
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
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
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
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
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
Thanks as always for the review.
Ayal Zaks ayal.z...@gmail.com writes:
Richard Sandiford richard.sandif...@linaro.org wrote on 30/08/2011
03:29:26 PM:
From: Richard Sandiford richard.sandif...@linaro.org
To: gcc-patches@gcc.gnu.org
Cc: Ayal Zaks/Haifa/IBM@IBMIL
Date: 30/08/2011 03:29
2011/9/22 Jason Merrill ja...@redhat.com:
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
On Wed, Sep 7, 2011 at 10:01 PM, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Sep 07, 2011 at 09:54:03PM +0200, Uros Bizjak wrote:
2011-09-07 Jakub Jelinek ja...@redhat.com
PR target/50310
* config/i386/i386.c (ix86_prepare_sse_fp_compare_args): For
TARGET_AVX
On 09/22/2011 05:11 PM, Fabien Chêne wrote:
2011/9/22 Jason Merrillja...@redhat.com:
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
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
===
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
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.
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.
On 9/21/11, dnovi...@google.com 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:
On Sep 1, 2011, at 6:20 AM, Bernd Schmidt wrote:
The following change
2011-05-26 Rainer Orth r...@cebitec.uni-bielefeld.de
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 for trunk and 4.6.
Jason
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): Add new
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 intends to
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
to
2011/9/22 Georg-Johann Lay a...@gjlay.de:
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
73 matches
Mail list logo