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 wor
On Mon, Jul 23, 2012 at 7:01 PM, 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.
On Mon, Jul 23, 2012 at 10:58 PM, Marc Glisse wrote:
> On Wed, 20 Jun 2012, Richard Guenther wrote:
>
>> On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse wrote:
>>>
>>> Hello,
>>>
>>> currently, tree-ssa-ifcombine handles pairs of imbricated "if"s that
>>> share
>>> the same then branch, or the same
On Tue, 24 Jul 2012, Richard Guenther wrote:
On Mon, Jul 23, 2012 at 10:58 PM, Marc Glisse wrote:
On Wed, 20 Jun 2012, Richard Guenther wrote:
On Sun, Jun 10, 2012 at 4:16 PM, Marc Glisse wrote:
Hello,
currently, tree-ssa-ifcombine handles pairs of imbricated "if"s that
share
the same th
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 @@
> 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:
> Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK?
>
> Ciao!
> Steven
OK, thanks!
--
Laurynas
On Tue, Jul 24, 2012 at 10:25 AM, Marc Glisse 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:
> http://gcc.gnu.org/ml/gcc
On Tue, Jul 24, 2012 at 11:31 AM, Laurynas Biveinis
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:
>> Bootstrapped&tested on powerpc64-unknown-linux-gnu. OK?
>>
>> Ciao!
>> Stev
On Tue, Jul 24, 2012 at 11:38 AM, Richard Guenther
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 above x1 (for token "+") is
Hello!
> 2012-7-18 Venkataramanan Kumar
>
> Jaguar Enablement
> * config.gcc (i[34567]86-*-linux* | ...): Add btver2.
> (case ${target}): Add btver2.
> * config/i386/driver-i386.c (host_detect_local_cpu): Let
> -march=native recognize btver2 processors.
> * c
On Mon, Jul 23, 2012 at 10:00 PM, Igor Zamyatin wrote:
> 2012-07-23 Sergey Melnikov
>
> * config/i386/i386.md (stack_protect_set): Disable the pattern
> for Android since Android libc (bionic) does not provide random
> value for stack protection guard at gs:0x14. Guard va
> 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 loc
On Mon, Jul 23, 2012 at 10:17 PM, Teresa Johnson wrote:
> Resending in plain text mode so it goes through.
> Teresa
>
> On Mon, Jul 23, 2012 at 12:03 PM, Teresa Johnson wrote:
>> Any possibility of getting these patches (186979 and 186993), along with
>> r184891 (which added the and->zext splitte
On Tue, Jul 24, 2012 at 12:11 PM, Laurynas Biveinis
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 above x1 (for token
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
* lto-tree.h (lang_decl): Add variable_size GTY option.
gcc/go/ChangeLog:
2012-07-24 Uros Bizjak
> gcc/lto/ChangeLog:
>
> 2012-07-24 Uros Bizjak
>
> * lto-tree.h (lang_decl): Add variable_size GTY option.
>
> gcc/go/ChangeLog:
>
> 2012-07-24 Uros Bizjak
>
> * go-lang.c (lang_decl): Add variable_size GTY option.
>
> gcc/c/ChangeLog:
>
> 2012-07-24 Uros Bizjak
>
>
2012/7/24 Steven Bosscher :
> On Tue, Jul 24, 2012 at 12:11 PM, Laurynas Biveinis
> 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 looks redundant, are there others?
>
> Yes:
On Mon, 23 Jul 2012 17:41:03 +0100
Ramana Radhakrishnan wrote:
> On 23 July 2012 16:41, Julian Brown wrote:
> > On Mon, 23 Jul 2012 14:40:59 +0100
> > Richard Earnshaw wrote:
> >
> >> On 23/07/12 13:17, Julian Brown wrote:
> >> > Hi,
> >> >
> >> > The test case gcc.dg/torture/stackalign/builtin
Hello!
2012-07-24 Uros Bizjak
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 x86_64_immediate_
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 Rol
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 imm
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 o
On Fri, 20 Jul 2012 11:15:27 +0100
Julian Brown 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 straightforward than it w
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 c
On Tue, Jul 24, 2012 at 2:09 PM, Kirill Yukhin 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 adxx. I got lost in the
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 wrote:
>
> On Mon, Jul 23, 2012 at 10:17 PM, Teresa Johnson wrote:
> > Resending in plain text mode so it goes through.
> > Teresa
> >
> > On Mon, Jul 23, 2012 at 12:03 PM, Teresa Johnson
> > wrote:
> >> Any possibility of getting these patches (1869
> 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, t
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. Tha
Hi!
I just committed this patch:
2012-07-24 Jan-Benedict Glaw
* 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 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 selective-scheduler test cas
On Tue, Jul 24, 2012 at 4:06 PM, Kirill Yukhin 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
>> patterns, while it
On Tue, Jul 24, 2012 at 4:37 PM, Steven Bosscher wrote:
> On Tue, Jul 24, 2012 at 3:08 PM, Uros Bizjak 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/20
>>> 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: opt
On Tue, Jul 24, 2012 at 5:20 PM, Laurynas Biveinis
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 a single commit.
Thanks
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
wrote:
> On Tue, Jul 24, 2012 at 10:25 AM, Marc Glisse 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.
>>
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.
Thank
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.
>
Thanks muchly!
On Tue, Jul 24, 2012 at 6:21 PM, Markus Trippelsdorf
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 branches.
>>
>> Thi
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 th
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 s
Hello,
On 24.07.2012 18:56, Steven Bosscher wrote:
On Tue, Jul 24, 2012 at 4:37 PM, Steven Bosscher wrote:
On Tue, Jul 24, 2012 at 3:08 PM, Uros Bizjak 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
On Tue, 24 Jul 2012, Andrew Pinski wrote:
On Tue, Jul 24, 2012 at 2:37 AM, Richard Guenther
wrote:
On Tue, Jul 24, 2012 at 10:25 AM, Marc Glisse 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
> 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 pa
Hans-Peter Nilsson wrote:
> > From: Richard Earnshaw
> > 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 in a
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 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 with --
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 - fourt
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
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
>
> * gcc.dg/20020201-1.c: Include .
>
>
> Patch attached.
>
> Ok for trunk & 4.7?
OK if you remove the declarations fo
> 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 &
tf_er
> "Laurynas" == Laurynas Biveinis writes:
Laurynas> 2012-06-24 Laurynas Bivienis
Laurynas> * include/line-map.h (line_map_macro): Use the "atomic" GTY
option
Laurynas> for the macro_locations field.
Laurynas> gcc/java:
Laurynas> 2012-07-24 Laurynas Biveinis
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-
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
Backport from mainline
2012-07-24 Uros Bizjak
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 po
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 wrote:
>
>> 2012-07-23 Sergey Melnikov
>>
>>* config/i386/i386.md (stack_protect_set): Disable the pattern
>>for Android since Android libc (bionic) does not provide random
>>
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$ s
On Jul 24, 2012, Richard Henderson 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 check them in
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 a
> This is OK for Google 4.7
Thanks, committed at r189833.
-cary
On Tue, Jul 24, 2012 at 5:08 PM, Cary Coutant 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 have been back
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
> te
On 24.07.2012 21:13, Steven Bosscher wrote:
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 f
74 matches
Mail list logo