On 05/16/13 15:32, Rainer Orth wrote:
The new gcc.dg/visibility-21.c testcase fails on i386-pc-solaris2.11 and
x86_64-unknown-linux-gnu:
FAIL: gcc.dg/visibility-21.c (test for excess errors)
Excess errors:
/vol/gcc/src/hg/trunk/local/gcc/testsuite/gcc.dg/visibility-21.c:1:0: warning:
this targ
Hi!
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg00282.html
Reject -fsanitize=address -fsanitize=thread linking that won't ever work at
runtime.
Jakub
On Thu, May 16, 2013 at 07:59:00PM +0200, Mikael Pettersson wrote:
> Jakub Jelinek writes:
> > On Thu, Apr 25, 2013 at 11:47:02PM +0200, Jakub Jelinek wrote:
> > > This patch adds folding of constant arguments v>> and v<<, which helps to
> > > optimize the testcase from the PR back into constant
Followup (and depending on) to the C binding patches for
* COMMON: http://gcc.gnu.org/ml/fortran/2013-05/msg00048.html
* Procedures: http://gcc.gnu.org/ml/fortran/2013-05/msg00051.html
which honour Fortran 2008, where the Fortran name is no longer a global
identifier if a binding name has been sp
On Wed, May 15, 2013 at 03:24:37PM +0200, Richard Biener wrote:
> We have the same issue in some other places where we insert invariant
> code into the loop body - one reason there is another LIM pass
> after vectorization.
Well, in this case it causes the shift amount to be loaded into a vector
i
On Thu, May 16, 2013 at 04:00:53PM -0700, Sriraman Tallam wrote:
> On Thu, May 16, 2013 at 3:55 PM, Marc Glisse wrote:
> > I don't really understand why you made the change to x86intrin.h instead of
> > making it inside each *mmintrin.h header. The code would be the same size,
> > it would let us
Thank you Uros for the patch. Could you backport this to the 4.8.0?
-Original Message-
From: Uros Bizjak [mailto:ubiz...@gmail.com]
Sent: Wednesday, May 15, 2013 11:16 PM
To: gcc-patches@gcc.gnu.org
Cc: Gopalasubramanian, Ganesh
Subject: [PATCH, i386]: Update processor_alias_table for mis
On Thu, May 16, 2013 at 10:40 PM, Bill Schmidt
wrote:
> This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
> vectorization opportunities. Previously, GCC treated malloc'd arrays as
> only guaranteeing 4-byte alignment, even though the glibc implementation
> guarantees 8-byte
I've stumbled across this a few times. tree-switch-conversion.c
includes optabs.h and thus should depend on $(OPTABS_H).
Installed as obvious.
commit 7028127e5cdc8c8662a0850dcd3b08df10d229b3
Author: Jeff Law
Date: Thu May 16 21:31:09 2013 -0600
* Makefile.in (tree-switch-conver
Not to much to add beyond the title and the patch.
The test file fails before (eventually, when you run out of stack) and passes
after the patch is applied. No new testsuite failures.
--bud
Index: gcc/gcc/fortran/resolve.c
===
---
This removes two degradations in CPU2006 for 32-bit PowerPC due to lost
vectorization opportunities. Previously, GCC treated malloc'd arrays as
only guaranteeing 4-byte alignment, even though the glibc implementation
guarantees 8-byte alignment. This raises the guarantee to 8 bytes,
which is suff
On May 16, 2013, at 5:26 PM, David Edelsohn wrote:
> This patch is creating new segfaults for 32 bit POWER AIX. Was this
> patch tested with PowerPC?
No, x86_64. I've added a patch to
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57304
If you could let us know if that fixes the problem you've
On Fri, May 17, 2013 at 02:08:27AM +0200, Gerald Pfeifer wrote:
> Ping. This will fix bootstrap on FreeBSD (and it seems NetBSD).
>
> (Paolo provided some comments, though this looks like the simplest
> patch to fix the issue.)
It's ok for me, but I cannot approve it.
-Andi
Mike,
This patch is creating new segfaults for 32 bit POWER AIX. Was this
patch tested with PowerPC?
Program received signal SIGSEGV, Segmentation fault.
0x10a1db88 in _ZL8web_mainv () at /nasfarm/dje/src/src/gcc/web.c:138
138 if (DF_REF_REAL_LOC (*ref) == recog_data.operand_loc[op]
Hi,
in this old issue Joseph pointed out that the C++ parser doesn't accept
the GNU Extension 'sizeof compound-literal' and provided indications
about the way to approach it. Indeed, I found very easy to use in
cp_parser_sizeof_operand the same approach adopted in
cp_parser_cast_expression (m
Ping. This will fix bootstrap on FreeBSD (and it seems NetBSD).
(Paolo provided some comments, though this looks like the simplest
patch to fix the issue.)
Gerald
On Mon, 1 Apr 2013, Gerald Pfeifer wrote:
> Andi's patch broke bootstrap on all FreeBSD platforms, which took me
> a bit to realize
On Fri, May 10, 2013 at 11:43 PM, Chung-Ju Wu wrote:
> 2013/5/10 Sebastian Huber :
> v2: Format changes
>
> gcc/testsuite/ChangeLog
> 2013-05-10 Sebastian Huber
>
> PR target/55033
> * gcc.target/powerpc/pr55033.c: New.
The testcase is okay.
Do you have CVS write access or do
On Thu, May 16, 2013 at 3:55 PM, Marc Glisse wrote:
> On Thu, 16 May 2013, Sriraman Tallam wrote:
>
>> Hi Jakub,
>>
>> I have taken your proposed changes and made patch for this. Please
>> let me know what you think. I have changed only the headers mmintrin.h
>> and x86intrin.h as that includes
On Thu, 16 May 2013, Sriraman Tallam wrote:
Hi Jakub,
I have taken your proposed changes and made patch for this. Please
let me know what you think. I have changed only the headers mmintrin.h
and x86intrin.h as that includes all the other headers.
I don't really understand why you made the
Hi Jakub,
I have taken your proposed changes and made patch for this. Please
let me know what you think. I have changed only the headers mmintrin.h
and x86intrin.h as that includes all the other headers. The builtins
get enabled automatically when the pragma target is specified so need
to do
Hello!
According to cpuid dumps at [1], we can use detect_caches_amd for
several othver vendors.
2013-05-16 Uros Bizjak
* config/i386/driver-i386.c (host_detect_local_cpu): Determine
cache parameters using detect_caches_amd also for CYRIX,
NSC and TM2 signatures.
Tested on x86_64
On Thu, May 16, 2013 at 2:02 PM, Richard Sandiford
wrote:
> Steven Bosscher writes:
>> On Wed, May 15, 2013 at 12:14 AM, Mike Stump wrote:
>>> I don't what to bike shed. So, I'm happy if the next poor soul that
>>> touches it just does so. If people like recog_data_info, I'd be happy
>>> to cha
Steven Bosscher writes:
> On Wed, May 15, 2013 at 12:14 AM, Mike Stump wrote:
>> I don't what to bike shed. So, I'm happy if the next poor soul that
>> touches it just does so. If people like recog_data_info, I'd be happy
>> to change it to that. Let's give then peanut gallery a day to vote on
On Thu, May 16, 2013 at 06:44:03PM +0200, Marek Polacek wrote:
> --- gcc/tree-ssa-strlen.c.mp 2013-05-15 14:11:20.079707492 +0200
> +++ gcc/tree-ssa-strlen.c 2013-05-16 17:57:33.963150006 +0200
> @@ -1693,8 +1693,10 @@ handle_char_store (gimple_stmt_iterator
> }
> els
Hello!
Attached patch introduces handling of VIA/Centaur processors with
-march=native compile directive. Comparing to original patch in
PR45359, attached patch handles detection of "unknown" processors via
default detection procedure. I also choose to use generic tuning for
all detected processo
OK.
Jason
On Thu, 16 May 2013, DJ Delorie wrote:
> SIZE_TYPE is compared - with individual strcmp's! - against a fixed
> set of C types, instead of using some sort of table lookup to support
> non-power-of-two types.
Joern had a patch (as of December 2010, possibly updated since then) to
convert at least
Trunk was merged into c++-concepts branch at revision 198984.
-- Gaby
Jakub Jelinek writes:
> On Thu, Apr 25, 2013 at 11:47:02PM +0200, Jakub Jelinek wrote:
> > This patch adds folding of constant arguments v>> and v<<, which helps to
> > optimize the testcase from the PR back into constant store after vectorized
> > loop is unrolled.
>
> As this fixes a regre
On 05/15/2013 10:55 PM, Teresa Johnson wrote:
This patch refactors rtl_verify_flow_info_1 and rtl_verify_flow_info
by outlining the verification code into several different routines.
rtl_verify_flow_info_1 in particular was getting very large.
For the most part the functionality is exactly the s
On 05/16/2013 11:10 AM, Steve Ellcey wrote:
On Thu, 2013-05-16 at 10:58 +0200, Richard Biener wrote:
Hmm, not terribly happy with that wording, but that gives you an idea of
what I'm after. When would someone set UPDATE_DOMINANCE to true and what
are their responsibilities when they do so.
A
On Thu, May 16, 2013 at 1:55 PM, Rainer Orth wrote:
> Steven Bosscher writes:
>
> Unfortunately, this patch broke SPARC bootstrap since it lost the
> HAVE_cc0 guard around link_cc0_insns:
Oops...
I followed Bernhard's suggestion and made those two moved functions
static in reorg.c just before com
> I thought it was the other way round - that after Bernd's fixes (July
> 2011) towards support for 40-bit integers, FRACTIONAL_INT_MODE worked
> better than PARTIAL_INT_MODE.
Probably most accurate to say that both ways are not well supported.
In general, I always have a hard time with anythin
On Thu, 2013-05-16 at 10:58 +0200, Richard Biener wrote:
> >
> > Hmm, not terribly happy with that wording, but that gives you an idea of
> > what I'm after. When would someone set UPDATE_DOMINANCE to true and what
> > are their responsibilities when they do so.
> >
> > Approved with the name cha
Hi,
testcase committed to mainline, I'm closing the PR as fixed.
Thanks,
Paolo.
2013-05-16 Paolo Carlini
PR c++/17410
* g++.dg/template/pr17410.C: New.
Index: g++.dg/template/pr17410.C
===
-
On Thu, 16 May 2013, DJ Delorie wrote:
> > What's the blocker to convert the existing 5 cases of
> > PARTIAL_INT_MODE use to specify a precision?
>
> In general? For me, PARTIAL_INT_MODE() works, FRACTIONAL_INT_MODE()
> didn't.
I thought it was the other way round - that after Bernd's fixes (Ju
On Thu, May 16, 2013 at 04:28:19PM +0200, Jakub Jelinek wrote:
> On Thu, May 16, 2013 at 04:18:27PM +0200, Jakub Jelinek wrote:
> > As q could point to p, if we didn't do what your patch does on the p[0] =
> > 'X';
> > store, then we'd need to invalidate the recorded length of the q string.
> > Si
> What's the blocker to convert the existing 5 cases of
> PARTIAL_INT_MODE use to specify a precision?
In general? For me, PARTIAL_INT_MODE() works, FRACTIONAL_INT_MODE()
didn't.
> Interestingly we have exactly that for AVR:
>
> config/avr/avr-modes.def:FRACTIONAL_INT_MODE (PSI, 24, 3);
I know. I tried copying them, it didn't work for me.
This one is OK.
> 2013-05-16 Nick Clifton
>
> * config/rl78/rl78.c (rl78_attribute_table): Add naked.
> (rl78_is_naked_func): New function.
> (rl78_expand_prologue): Skip prologue generation for naked
> functions.
> (rl78_expand_epilogue): Skip epilogue g
On Wed, May 15, 2013 at 03:24:37PM +0200, Richard Biener wrote:
> Ok with ...
> > + /* Pattern detected. */
> > + if (dump_enabled_p ())
> > +dump_printf_loc (MSG_OPTIMIZED_LOCATIONS, vect_location,
> > +"vect_recog_rotate_pattern: detected: ");
>
> Please use MSG_NOTE here
Hi!
As discussed in the PR, there seem to be only 3 define_split
patterns that use dead_or_set_p, one in i386.md and two in s390.md,
but unfortunately insn splitting is done in many passes
(combine, split{1,2,3,4,5}, dbr, pro_and_epilogue, final, sometimes mach)
and only in combine the note proble
Hi,
as agreed, I regtested on x86_64-linux the below diagnostic tweak.
Thanks,
Paolo.
//
/cp
2013-05-16 Paolo Carlini
PR c++/17314
* call.c (enforce_access): Improve protected access error message.
/testsuite
2013-05-16 Paolo Carlini
PR c
Hi DJ,
OK, I am abandoning my RL78 interrupt prologue patch for now. There
are just too many complications to make it work. Instead here is a
much simpler patch to add support for a naked function attribute.
OK to apply ?
Cheers
Nick
gcc/ChangeLog
2013-05-16 Nick Clifton
On 05/16/13 06:41, Rainer Orth wrote:
Work is going on to incorporate all applicable fixincludes fixes into
the Solaris headers proper. One fix is currently problematic since it
uses an G++-internal macro (__GXX_EXPERIMENTAL_CXX0X__) where libstdc++
already switched to testing __cplusplus. The
On Thu, May 16, 2013 at 6:55 AM, Teresa Johnson wrote:
>
> * cfgrtl.c (verify_hot_cold_block_grouping): Return err.
> (rtl_verify_edges): New function.
> (rtl_verify_bb_insns): Ditto.
> (rtl_verify_bb_pointers): Ditto.
> (rtl_verify_bb_insn_chain): Ditto.
>
Hello,
this is a minor patch, instead of manually building a NE_EXPR, it seems
better to call the front-end function whose job it is. Incidentally, it
means one expression is folded a little bit more, so I am re-enabling a
test that I commented out this morning.
Bootstrap+testsuite on x86_64
In C++11 we also need to allow function qualifiers in
alias-declarations, since they are equivalent to typedefs. There
doesn't seem to be any good reason for giving an error in grokdeclarator
at all, since we need to complain about uses of qualified function types
via typedef or template param
My patch for PR19599 yesterday had a problem with arm-none-eabi builds.
I had missed a null pointer check of decl in an AAPCS only
configuration. Now applied as obvious.
regards
Ramana
2013-05-16 Ramana Radhakrishnan
PR target/19599
* config/arm/arm.c (arm_function_ok_for_
On Thu, May 16, 2013 at 8:41 AM, Rainer Orth
wrote:
> Work is going on to incorporate all applicable fixincludes fixes into
> the Solaris headers proper. One fix is currently problematic since it
> uses an G++-internal macro (__GXX_EXPERIMENTAL_CXX0X__) where libstdc++
> already switched to test
Nathan Sidwell writes:
> This patch fixes a problem with section anchors. Found on powerpc, but
> also appears on MIPS and ARM targets.
>
> Section anchors can only be used for definitions known to bind in the
> current object file. The default predicate uses the bind_local_p hook to
> determin
On Thu, May 16, 2013 at 04:18:27PM +0200, Jakub Jelinek wrote:
> As q could point to p, if we didn't do what your patch does on the p[0] = 'X';
> store, then we'd need to invalidate the recorded length of the q string.
> Similarly if there is p[0] = '\0' or p[0] = var.
Ah, another thing while we a
On Thu, May 16, 2013 at 04:07:44PM +0200, Marek Polacek wrote:
> --- gcc/tree-ssa-strlen.c.mp 2013-05-15 14:11:20.079707492 +0200
> +++ gcc/tree-ssa-strlen.c 2013-05-16 16:03:50.373504796 +0200
> @@ -1717,6 +1717,27 @@ handle_char_store (gimple_stmt_iterator
> si->endptr = ssaname;
>
On Thu, May 16, 2013 at 03:38:47PM +0200, Jakub Jelinek wrote:
> Please add here a comment what it does and why, that if si->length
> is non-zero constant, we know that the character at that spot is
> not '\0' and when storing non-'\0' to that location, we can't affect
> size of any strings at all.
Work is going on to incorporate all applicable fixincludes fixes into
the Solaris headers proper. One fix is currently problematic since it
uses an G++-internal macro (__GXX_EXPERIMENTAL_CXX0X__) where libstdc++
already switched to testing __cplusplus. The following patch
updates the fix to matc
On Thu, May 16, 2013 at 03:31:40PM +0200, Marek Polacek wrote:
> Cool, I took the gsi_next approach; seems to work nicely. So, updated
> version (it still doesn't handle p[1], p[2], etc.).
>
> Regtested/bootstrapped on x86_64-linux.
>
> 2013-05-16 Marek Polacek
>
> * tree-ssa-strlen.c
On Wed, May 15, 2013 at 07:20:03PM +0200, Jakub Jelinek wrote:
> Well, if si->length is known constant != 0 (note, it is enough to
> test that it is non-zero and probably the code above this has
> tested that already?) and we are storing non-zero, then that
> should mean we are overwriting a non-ze
Tobias Burnus writes:
> Am 16.05.2013 13:49, schrieb Rainer Orth:
>> Corresponding to the Solaris 9 obsoletion patch, I've now installed the
>> following wwwdocs patch to document it.
>> --- htdocs/gcc-4.9/changes.html 9 May 2013 17:08:19 - 1.11
>> +++ htdocs/gcc-4.9/changes.html
Am 16.05.2013 13:49, schrieb Rainer Orth:
Corresponding to the Solaris 9 obsoletion patch, I've now installed the
following wwwdocs patch to document it.
--- htdocs/gcc-4.9/changes.html 9 May 2013 17:08:19 - 1.11
+++ htdocs/gcc-4.9/changes.html 16 May 2013 11:48:37 - 1.13
@@ -1
On May 15, 2013, at 10:41 PM, Shakthi Kannan wrote:
> | I like using ~/contrib/compare_tests gcc-before.sum gcc-after.sum to
> | determine if there are regressions. You can also use that script to
> | check for regressions between two build trees as well.
> \--
>
> I ran the the script betwee
On Thu, May 16, 2013 at 12:05 PM, Marc Glisse wrote:
> On Thu, 16 May 2013, Richard Biener wrote:
>
>> On Thu, May 16, 2013 at 8:42 AM, Marc Glisse wrote:
>>>
>>> Hello,
>>>
>>> we can get into a cycle where:
>>> (x<0)|1 becomes (x<0)?-1:1
>>> and
>>> (y?-1:1) becomes y|1
>>>
>>> Contrary to what
The following is a revision of the original patch in that it
also constrains versioning for aliasing with the cheap cost model
(formerly -ftree-vect-loop-version only constrained versioning for
alignment). It also changes vectorizer related gates to rely
on global_options_set instead of playing m
Steven Bosscher writes:
> This just removes one unused function, and moves two functions from
> emit-rtl.c to reorg.c which is the only place where they're used.
>
> Will commit in a few days, barring objections.
>
> Ciao!
> Steven
>
>
> * rtl.h (next_label, skip_consecutive_labels, link_
Corresponding to the Solaris 9 obsoletion patch, I've now installed the
following wwwdocs patch to document it.
Rainer
* htdocs/gcc-4.9/changes.html: Document Solaris 9 obsoletion.
Index: htdocs/gcc-4.9/changes.html
===
Rainer Orth writes:
> I think the time has come to obsolete Solaris 9 support:
>
> * According to
>
> http://www.oracle.com/us/support/library/lsp-coverage-sun-software-309122.pdf,
> p.17, Premier Support has already ended in October 2011 and even
> Extended Support will end in October 201
On May 15, 2013, Jason Merrill wrote:
> + elif [ $count == 300 ]; then
s/==/=/
Ok with this change.
Thanks,
--
Alexandre Oliva, freedom fighterhttp://FSFLA.org/~lxoliva/
You must be the change you wish to see in the world. -- Gandhi
Be Free! -- http://FSFLA.org/ FSF Latin America
I noticed that when you enable vectorization for the autopar
testcases most of them fail. This is because vectorization
introduces new loops and tests that make the CFG complicated
enough for autopar to give up. But then autopar is a high-level
transform and should be done earlier anyway. Which
Hi!
As the testcase show, if in a schedule(static) (the default schedule)
or schedule(static,N) non-collapsed loop with unsigned integral iterator
the loop condition is false upon entering the loop,
but (n2 + step-1 - n1) / step is not 0, we could run the loop body, possibly
many times, rather tha
On Thu, 16 May 2013, Richard Biener wrote:
On Thu, May 16, 2013 at 8:42 AM, Marc Glisse wrote:
Hello,
we can get into a cycle where:
(x<0)|1 becomes (x<0)?-1:1
and
(y?-1:1) becomes y|1
Contrary to what I posted in the PR, I am disabling the second
transformation here. It can be done later (t
On Wed, May 15, 2013 at 4:35 PM, Marek Polacek wrote:
> On Wed, May 15, 2013 at 02:58:22PM +0200, Richard Biener wrote:
>> On Wed, May 15, 2013 at 2:07 PM, Marek Polacek wrote:
>> > /* Replace the uses of the name. */
>> > if (name != ev)
>> > - replace_uses_by (nam
On Thu, May 16, 2013 at 10:54:53AM +0200, Richard Biener wrote:
> Can we properly distinguish the case of
>
> char *s = "Hello\0World!";
> s[5] = ' ';
>
> (minor the imperfections in that example)? Thus, overwriting the terminating
> 0?
I think so. Because then for &s[5], either si should
On Thu, May 16, 2013 at 8:42 AM, Marc Glisse wrote:
> Hello,
>
> we can get into a cycle where:
> (x<0)|1 becomes (x<0)?-1:1
> and
> (y?-1:1) becomes y|1
>
> Contrary to what I posted in the PR, I am disabling the second
> transformation here. It can be done later (the x86 target partially does it
On Thu, May 16, 2013 at 5:38 AM, Jeff Law wrote:
> On 05/15/2013 12:28 PM, Steve Ellcey wrote:
>>
>> Here is a patch that adds a flag to gimple_duplicate_sese_region to tell
>> it whether or not to update the dominator information. I had to add the
>> same flag to copy_bbs to make it all work. H
On Wed, May 15, 2013 at 7:24 PM, Mike Stump wrote:
> On May 15, 2013, at 1:27 AM, Richard Biener
> wrote:
>> My question is, if you end up with
>>
>> (truncate:PSI (reg:SI 27))
>>
>> and then constant propagate 0x7fff to reg:SI 27, what does simplify-rtx.c
>> do here? Truncate to _what_ pr
On Wed, May 15, 2013 at 7:20 PM, Jakub Jelinek wrote:
> On Wed, May 15, 2013 at 06:59:09PM +0200, Marek Polacek wrote:
>> This is a strlen opt patch that better optimizes attached testcase;
>> there's just no need to call strlen again, as we're not changing
>> the length of the string. Unfortunat
Jason Merrill writes:
> I thought I approved this on IRC; please apply it to trunk and 4.8.
Applied to both branches, thanks.
--
Dodji
76 matches
Mail list logo
