On Mon, Jul 23, 2012 at 07:30:23PM -0400, David Edelsohn wrote:
This is okay
Committed revision 189801.
PR target/53914
PR target/54009
* config/rs6000/constraints.md (Y): Use mem_operand_gpr.
* config/rs6000/predicates.md (word_offset_memref_operand): Delete.
On Mon, 23 Jul 2012, Andrew Pinski wrote:
This patch improves ifcombine by doing two things. First tries to see
if the then and else are swapped and handles that case.
Just a few hours after I posted:
http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01160.html
Could you check that your patch
On Mon, Jul 23, 2012 at 7:01 PM, Steven Bosscher stevenb@gmail.com wrote:
Hello,
This large patch makes GATHER_STATISTICS always take a value, 0
(disabled) or 1 (enabled), and turns all related #ifdef code into
conditional branches.
This slightly increases the data section of cc1, but
On Mon, Jul 23, 2012 at 10:58 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Wed, 20 Jun 2012, Richard Guenther wrote:
On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
currently, tree-ssa-ifcombine handles pairs of imbricated ifs that
share
the same then
On Tue, 24 Jul 2012, Richard Guenther wrote:
On Mon, Jul 23, 2012 at 10:58 PM, Marc Glisse marc.gli...@inria.fr wrote:
On Wed, 20 Jun 2012, Richard Guenther wrote:
On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse marc.gli...@inria.fr wrote:
Hello,
currently, tree-ssa-ifcombine handles pairs
Hello,
For scalar fields or skipped fields, gengtype produces empty loops.
This patch checks for that case and breaks out if an empty loop body
is detected:
--- gtype-desc.c.orig 2012-07-24 01:57:42.830007515 -0700
+++ gtype-desc.c2012-07-24 02:10:14.530007515 -0700
@@ -89,6 +89,7 @@
On Tue, Jul 24, 2012 at 10:25 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 23 Jul 2012, Andrew Pinski wrote:
This patch improves ifcombine by doing two things. First tries to see
if the then and else are swapped and handles that case.
Just a few hours after I posted:
On Tue, Jul 24, 2012 at 11:31 AM, Laurynas Biveinis
laurynas.bivei...@gmail.com wrote:
For scalar fields or skipped fields, gengtype produces empty loops.
This patch checks for that case and breaks out if an empty loop body
is detected:
Bootstrappedtested on powerpc64-unknown-linux-gnu. OK?
On Mon, Jul 23, 2012 at 10:00 PM, Igor Zamyatin izamya...@gmail.com wrote:
2012-07-23 Sergey Melnikov sergey.melni...@intel.com
* config/i386/i386.md (stack_protect_set): Disable the pattern
for Android since Android libc (bionic) does not provide random
value for
I think it merely points to a bogus GTY annotation, not sure if we want to
gobble this kind of gengtype hacks which only benefit -O0 ...
The annotation is libcpp/include/line-map.h
In the example above x1 (for token +) is going to be the same
as y1. x0 is the spelling location
On Mon, Jul 23, 2012 at 10:17 PM, Teresa Johnson tejohn...@google.com wrote:
Resending in plain text mode so it goes through.
Teresa
On Mon, Jul 23, 2012 at 12:03 PM, Teresa Johnson tejohn...@google.com wrote:
Any possibility of getting these patches (186979 and 186993), along with
r184891
On Tue, Jul 24, 2012 at 12:11 PM, Laurynas Biveinis
laurynas.bivei...@gmail.com wrote:
I think it merely points to a bogus GTY annotation, not sure if we want to
gobble this kind of gengtype hacks which only benefit -O0 ...
The annotation is libcpp/include/line-map.h
In the example
Hello!
Attached patch implements suggested solution [1] from thread, starting
at [2]. Patch works OK in my tree without ada directory.
gcc/lto/ChangeLog:
2012-07-24 Uros Bizjak ubiz...@gmail.com
* lto-tree.h (lang_decl): Add variable_size GTY option.
gcc/go/ChangeLog:
2012-07-24
gcc/lto/ChangeLog:
2012-07-24 Uros Bizjak ubiz...@gmail.com
* lto-tree.h (lang_decl): Add variable_size GTY option.
gcc/go/ChangeLog:
2012-07-24 Uros Bizjak ubiz...@gmail.com
* go-lang.c (lang_decl): Add variable_size GTY option.
gcc/c/ChangeLog:
2012-07-24 Uros
2012/7/24 Steven Bosscher stevenb@gmail.com:
On Tue, Jul 24, 2012 at 12:11 PM, Laurynas Biveinis
laurynas.bivei...@gmail.com wrote:
I think it merely points to a bogus GTY annotation, not sure if we want to
gobble this kind of gengtype hacks which only benefit -O0 ...
This one indeed
On Mon, 23 Jul 2012 17:41:03 +0100
Ramana Radhakrishnan ramana.radhakrish...@linaro.org wrote:
On 23 July 2012 16:41, Julian Brown jul...@codesourcery.com wrote:
On Mon, 23 Jul 2012 14:40:59 +0100
Richard Earnshaw rearn...@arm.com wrote:
On 23/07/12 13:17, Julian Brown wrote:
Hi,
Hello!
2012-07-24 Uros Bizjak ubiz...@gmail.com
PR target/53961
* config/i386/i386.c (ix86_legitimate_address_p): Move check for
negative constant address for TARET_X32 ...
(ix86_decompose_address): ... here. Reject constant addresses
that don't satisfy
On 17/07/12 21:42, Roland McGrath wrote:
Richard, here is the patch against the current trunk, as I promised
last week in Prague. Please apply.
Done.
I've tweaked the comments slightly, but the functional modification is
unchanged.
R.
Thanks,
Roland
gcc/
2012-07-17 Roland
Hi ,
While testing my neon intrinsics work with some testcases
that I was writing up, I ran into PR54051 . The one change which is
probably a bit long standing is the fact that for register only
addressing modes i.e. something like mem (reg:SI) we were printing out
addresses with an
Hi Sandra,
I've updated the patch
One suggestion - rather than having architecture specific test files,
why not just have a single generic test case with a new
dg-require-naked-attribute qualifier. That way the mcore port would be
tested as well as the ARM port.
I'm not set up to test
On Fri, 20 Jul 2012 11:15:27 +0100
Julian Brown jul...@codesourcery.com wrote:
Anyway: this revised version of the patch removes the strange libgcc
Makefile-fragment changes, the equivalent of which have since been
incorporated into mainline GCC now anyway, so the patch is somewhat
more
On Tue, 2012-07-24 at 10:57 +0200, Richard Guenther wrote:
On Mon, 23 Jul 2012, William J. Schmidt wrote:
This patch completes the conversion of the vectorizer cost model to use
target hooks for recording vectorization information and calculating
costs. Previous work handled the costs
On Tue, Jul 24, 2012 at 2:09 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Uros, Jakub, thanks fr the inputs!
I've refactored the patch appliying (hopefully) all of your inputs.
It is attached.
Can you please split the patch to three independent parts, prefetchw
handling, rdseed and
Hello!
This large patch makes GATHER_STATISTICS always take a value, 0
(disabled) or 1 (enabled), and turns all related #ifdef code into
conditional branches.
This slightly increases the data section of cc1, but only marginally.
There is no impact on compile time, because all tests are
On Tue, Jul 24, 2012 at 3:18 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Mon, Jul 23, 2012 at 10:17 PM, Teresa Johnson tejohn...@google.com wrote:
Resending in plain text mode so it goes through.
Teresa
On Mon, Jul 23, 2012 at 12:03 PM, Teresa Johnson tejohn...@google.com
wrote:
Any
Can you please split the patch to three independent parts, prefetchw
handling, rdseed and adxx. I got lost in the patch the second time.
Sure!
I propose we start with prefetch. You are still changing prefetch
patterns, while it should be enough to only change prefetch expander.
Actually,
Hi,
while looking into the re-opened c++/51213 (doesn't seem trivial:
details soon, I hope) I noticed things which look like old pastos: in
convert_template_argument and tsubst we are computing complain
tf_error for the - tsubst_flags_t, not bool - fourth argument of
make_typename_type.
Hi!
I just committed this patch:
2012-07-24 Jan-Benedict Glaw jbg...@lug-owl.de
* MAINTAINERS (Write After Approval): Add myself.
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -371,6 +371,7 @@ Pompapathi V Gadad
pompapathi.v.ga...@nsc.com
Kaveh Ghazi
On Tue, Jul 24, 2012 at 3:08 PM, Uros Bizjak ubiz...@gmail.com wrote:
This patch (r189803) regressed a bunch of tests on x86_64 [1], [2].
[1] http://gcc.gnu.org/ml/gcc-testresults/2012-07/msg02066.html
[2] http://gcc.gnu.org/ml/gcc-regression/2012-07/msg00177.html
These are all
On Tue, Jul 24, 2012 at 4:06 PM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Can you please split the patch to three independent parts, prefetchw
handling, rdseed and adxx. I got lost in the patch the second time.
Sure!
I propose we start with prefetch. You are still changing prefetch
On Tue, Jul 24, 2012 at 4:37 PM, Steven Bosscher stevenb@gmail.com wrote:
On Tue, Jul 24, 2012 at 3:08 PM, Uros Bizjak ubiz...@gmail.com wrote:
This patch (r189803) regressed a bunch of tests on x86_64 [1], [2].
[1] http://gcc.gnu.org/ml/gcc-testresults/2012-07/msg02066.html
[2]
I think it merely points to a bogus GTY annotation, not sure if we want to
gobble this kind of gengtype hacks which only benefit -O0 ...
../../trunk/gcc/../libcpp/include/line-map.h:168: option `length' may
not be applied to arrays of scalar types
../../trunk/gcc/emit-rtl.c:5913: option
On Tue, Jul 24, 2012 at 5:20 PM, Laurynas Biveinis
laurynas.bivei...@gmail.com wrote:
s/arrays of scalar types/arrays of atomic types, add a ChangeLog entry
and the snippet above is approved ;)
Can you commit it along with your fixes? I think it'd be good to keep
this and your fixes together in
On 07/21/2012 06:10 AM, Oleg Endo wrote:
I think on SH the cost test in lshift_cheap_p with
gen_rtx_ASHIFT (word_mode, const1_rtx, reg), speed_p);
will always 'fail', because of sh.c (shiftcosts):
/* There is no pattern for constant first operand. */
if (CONST_INT_P (XEXP (x, 0)))
On Tue, Jul 24, 2012 at 2:37 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Jul 24, 2012 at 10:25 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 23 Jul 2012, Andrew Pinski wrote:
This patch improves ifcombine by doing two things. First tries to see
if the then and else
On 07/21/2012 03:25 AM, Alexandre Oliva wrote:
PR debug/52983
* valtrack.c (dead_debug_insert_temp): Use cleanup_auto_inc_dec.
Ok.
r~
On 2012/07/23 21:21:28, Cary wrote:
This patch is for the google/gcc-4_7 branch. It backports five
Fission
patches from trunk, with two minor differences:
- The DW_AT_GNU_pubnames and DW_AT_GNU_pubtypes attributes are
both generated, and provide the offset to the corresponding
Hi,
Before tuples was introduced, VN used to lookup the simplified
expression to see if it was available already and use that instead of
the non simplified one. This patch adds the support back to VN to do
exactly that.
OK? Bootstrapped and tested on x86_64-linux-gnu with no regressions.
On 2012.07.23 at 19:01 +0200, Steven Bosscher wrote:
Hello,
This large patch makes GATHER_STATISTICS always take a value, 0
(disabled) or 1 (enabled), and turns all related #ifdef code into
conditional branches.
This slightly increases the data section of cc1, but only marginally.
There
Thanks muchly!
On Tue, Jul 24, 2012 at 6:21 PM, Markus Trippelsdorf
mar...@trippelsdorf.de wrote:
On 2012.07.23 at 19:01 +0200, Steven Bosscher wrote:
Hello,
This large patch makes GATHER_STATISTICS always take a value, 0
(disabled) or 1 (enabled), and turns all related #ifdef code into
conditional
On 07/20/2012 01:39 AM, Richard Guenther wrote:
I've not yet speed tested this, only completed bootstrap + test
runs for both x86_64 and ppc64. I've done sanity cross compiles
to alpha and mips (the only SWITCHABLE_TARGET).
It would be interesting to see startup-time effects of this patch
This patch prevents errors in dg-error, dg-warning, and dg-message from
being silently ignored by GCC's overrides for those procedures. It's
been tested on i686-pc-linux-gnu for the languages that use the overrides
(c,c++,fortran,objc,obj-c++). I scanned through target-specific tests
and didn't
Hello,
On 24.07.2012 18:56, Steven Bosscher wrote:
On Tue, Jul 24, 2012 at 4:37 PM, Steven Bosscherstevenb@gmail.com wrote:
On Tue, Jul 24, 2012 at 3:08 PM, Uros Bizjakubiz...@gmail.com wrote:
This patch (r189803) regressed a bunch of tests on x86_64 [1], [2].
[1]
On Tue, 24 Jul 2012, Andrew Pinski wrote:
On Tue, Jul 24, 2012 at 2:37 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Jul 24, 2012 at 10:25 AM, Marc Glisse marc.gli...@inria.fr wrote:
On Mon, 23 Jul 2012, Andrew Pinski wrote:
This patch improves ifcombine by doing two
AFAIR the qsort is just for getting a stable ordering over two pools to find
the leaked regsets afterwards, the type of ordering doesn't matter.
What matters is that the compare function gives a reliable result. You
can't subtract pointers like that for qsort.
After consulting the experts on
On Tue, 2012-07-24 at 08:34 -0700, Richard Henderson wrote:
On 07/21/2012 06:10 AM, Oleg Endo wrote:
I think on SH the cost test in lshift_cheap_p with
gen_rtx_ASHIFT (word_mode, const1_rtx, reg), speed_p);
will always 'fail', because of sh.c (shiftcosts):
/* There is no pattern for
Hans-Peter Nilsson wrote:
From: Richard Earnshaw rearn...@arm.com
Date: Mon, 11 Jun 2012 15:16:50 +0200
The ARM ABI states that vectors larger than 64 bits in size still have
64-bit alignment; never-the-less, the HW supports alignment hints of up
to 128-bits in some cases and will trap
On 07/23/2012 01:01 PM, Steven Bosscher wrote:
This large patch makes GATHER_STATISTICS always take a value, 0
(disabled) or 1 (enabled), and turns all related #ifdef code into
conditional branches.
Did you test this with --enable-gather-detailed-mem-stats? It didn't
build for me, so I fixed
On Tue, Jul 24, 2012 at 7:53 PM, Jason Merrill ja...@redhat.com wrote:
On 07/23/2012 01:01 PM, Steven Bosscher wrote:
This large patch makes GATHER_STATISTICS always take a value, 0
(disabled) or 1 (enabled), and turns all related #ifdef code into
conditional branches.
Did you test this
On 07/24/2012 10:23 AM, Paolo Carlini wrote:
while looking into the re-opened c++/51213 (doesn't seem trivial:
details soon, I hope) I noticed things which look like old pastos: in
convert_template_argument and tsubst we are computing complain
tf_error for the - tsubst_flags_t, not bool -
This patch allows the use of both target and xfail in the selector
of any test directive that currently takes either target or xfail:
{ target selector1 xfail selector2 }
The test is only used if the target selector is matched, and the test
is expected to fail if the xfail selector is matched.
On Tue, 24 Jul 2012, Uros Bizjak wrote:
Also can I modify doc/invoke.texi now?
OK for mainline SVN.
And, yes, I think you can update invoke.texi.
And add a note to the release notes at http://gcc.gnu.org/4.8/changes.html
;-)
Gerald
On 07/20/2012 05:23 AM, Anna Tikhonova wrote:
Hi all,
I've fixed test which was failing on Android NDK
testsuite/ChangeLog:
2012-07-20 Anna Tikhonova anna.m.tikhon...@gmail.com
* gcc.dg/20020201-1.c: Include stdlib.h.
Patch attached.
Ok for trunk 4.7?
OK if you
This patch is OK for Google branches. We'll need to update top of trunk
pubnames to match.
Thanks. Committed at r189822.
For trunk, I wanted to take a longer look at fixing what's there to
put the enumerators in pubnames. If that's not feasible, I'll just go
with what I did here.
-cary
On 07/24/2012 08:06 PM, Jason Merrill wrote:
On 07/24/2012 10:23 AM, Paolo Carlini wrote:
while looking into the re-opened c++/51213 (doesn't seem trivial:
details soon, I hope) I noticed things which look like old pastos: in
convert_template_argument and tsubst we are computing complain
Laurynas == Laurynas Biveinis laurynas.bivei...@gmail.com writes:
Laurynas 2012-06-24 Laurynas Bivienis laurynas.bivei...@gmail.com
Laurynas * include/line-map.h (line_map_macro): Use the atomic GTY
option
Laurynas for the macro_locations field.
Laurynas gcc/java:
Laurynas
Ping.
On Fri, Jul 13, 2012 at 02:00:54PM +0200, Marek Polacek wrote:
I think the comment at the start of the file is wrong, since
it speaks about varpool_finalize_variable, but there's no such
function (not even mentioned in CLs). I'd say the author meant
varpool_finalize_decl.
2012-07-13
On 07/24/2012 03:05 PM, Paolo Carlini wrote:
Thus, all in all, I propose doing the change anyway, of course with a
different ChangeLog: if what I summarized above is correct the risk of
breaking something is really tiny and the code would become a tad simpler.
OK.
Jason
Hello!
Attached patch backports recent LEA improvements to 4.7 branch. The
patch unifies handling of LEA addresses with generic address handling,
allowing some more zero-extended addresses for both cases.
2012-XX-XX Uros Bizjak ubiz...@gmail.com
Backport from mainline
I was looking to see what needs to be done to un-stick this previously
submitted patch:
http://gcc.gnu.org/ml/gcc-patches/2012-05/msg01419.html
Paolo's suggestion was to re-write this to use a tortoise-and-hare
algorithm to detect the circularity, rather than Andrew's solution of
using a
This patch modifies pmu-profile to allow gooda_feedback access to important
gcov to gcda conversion functions and also modifies the gcov pmu format store
indices in a string table of filenames as opposed to storing the filename
inside of every pmu entry. This reduces the amount of size bloat in
On 24/07/2012, at 10:08 PM, Uros Bizjak wrote:
On Mon, Jul 23, 2012 at 10:00 PM, Igor Zamyatin izamya...@gmail.com wrote:
2012-07-23 Sergey Melnikov sergey.melni...@intel.com
* config/i386/i386.md (stack_protect_set): Disable the pattern
for Android since Android libc
Hi,
Attached is a patch that modifies BE8_LINK_SPEC to pass -be8 option to the
linker by default if the target is big endian, and to not pass -be8 option if
-mlittle-endian is used. It also preserves the existing usage whereby -be8
gets passed with the use of -mbig-endian.
Patch:
bash-3.00$
On Jul 24, 2012, Richard Henderson r...@redhat.com wrote:
On 07/21/2012 03:25 AM, Alexandre Oliva wrote:
PR debug/52983
* valtrack.c (dead_debug_insert_temp): Use cleanup_auto_inc_dec.
Ok.
Thanks. I'm about to go on a trip for the rest of the week and I won't
be online much, so I won't
The libstdc++ prettyprinters test suite isn't called out in Makefile.am.
Backport Andreas's
patch from mainline so that a sequential test run gives the same results as a
parallel test.
Tested with no regressions on x86_64-linux-gnu and arm-linux-gnueabi. The
prettyprinter
tests are enabled
This is OK for Google 4.7
Thanks, committed at r189833.
-cary
On Tue, Jul 24, 2012 at 5:08 PM, Cary Coutant ccout...@google.com wrote:
This patch is for the google/gcc-4_7 branch.
Backport Fission patches from trunk at r186934, r187061, and r188759.
Backport pending Fission patch at http://codereview.appspot.com/6305113.
Patches for include/dwarf2.def
This patch is for the google/gcc-4_7 branch.
Add -gfission as alias for -gsplit-dwarf.
Tested by bootstrap and regression testing.
* common.opt (-gfission): Alias for -gsplit-dwarf.
(-gno-fission): Alias for -gno-split-dwarf.
Index: gcc/common.opt
s/arrays of scalar types/arrays of atomic types, add a ChangeLog entry
and the snippet above is approved ;)
Can you commit it along with your fixes? I think it'd be good to keep
this and your fixes together in a single commit.
OK
--
Laurynas
On 07/24/2012 05:18 AM, nick clifton wrote:
Hi Sandra,
I've updated the patch
One suggestion - rather than having architecture specific test files,
why not just have a single generic test case with a new
dg-require-naked-attribute qualifier. That way the mcore port would be
tested as
71 matches
Mail list logo