Don't drop attributes on template member functions

2013-10-02 Thread Easwaran Raman
This attempts to address one issue reported in http://gcc.gnu.org/bugzilla/show_bug.cgi?id=33911 . Attributes at the end of a declaration of template member functions are discarded in cp_parser_init_declarator. The suggested workaround is to place the attributes into the declaration-specifiers. Th

Re: Honnor ix86_accumulate_outgoing_args again

2013-10-02 Thread Jan Hubicka
> So I thing we ought to honnor accumulate-outgoing-args again and in fact > consider disabling it for generic - it is disabled for core (that may need > re-benchmarking). For all AMD targets it is currently on. I tested disabling > it on buldozer 32bit and it seems mostly SPEC neutral for specint

Re: [PATCH] fix size_estimation for builtin_expect

2013-10-02 Thread Rong Xu
attached the new patch. OK for check in? On Wed, Oct 2, 2013 at 9:12 AM, Jan Hubicka wrote: >> > Hi, >> > >> > builtin_expect should be a NOP in size_estimation. Indeed, the call >> > stmt itself is 0 weight in size and time. But it may introduce >> > an extra relation expr which has non-zero siz

Re: [PATCH] alternative hirate for builtin_expert

2013-10-02 Thread Rong Xu
Here is the new patch. Honaz: Could you take a look? Thanks, -Rong On Wed, Oct 2, 2013 at 2:31 PM, Jan Hubicka wrote: >> Thanks for the suggestion. This is much cleaner than to use binary parameter. >> >> Just want to make sure I understand it correctly about the orginal hitrate: >> you want to

Re: [PATCH] Reducing number of alias checks in vectorization.

2013-10-02 Thread Cong Hou
On Wed, Oct 2, 2013 at 2:18 PM, Xinliang David Li wrote: > On Wed, Oct 2, 2013 at 4:24 AM, Richard Biener wrote: >> On Tue, 1 Oct 2013, Cong Hou wrote: >> >>> When alias exists between data refs in a loop, to vectorize it GCC >>> does loop versioning and adds runtime alias checks. Basically for e

Re: Honnor ix86_accumulate_outgoing_args again

2013-10-02 Thread H.J. Lu
On Wed, Oct 2, 2013 at 3:45 PM, Jan Hubicka wrote: >> So I thing we ought to honnor accumulate-outgoing-args again and in fact >> consider disabling it for generic - it is disabled for core (that may need >> re-benchmarking). For all AMD targets it is currently on. I tested disabling >> it on bul

Re: [PATCH] Reducing number of alias checks in vectorization.

2013-10-02 Thread Cong Hou
Forget to mention that the alias check merger can reduce the number of checks from 7 to 2 for this example: struct A { int *base; int offset; int offset2; int offset3; int offset4; int offset5; int offset6; int offset7; int offset8; }; void foo (struct A * ar1, struct A* ar2) {

[PATCH] fix target 58587, disable -mvsx-timode for now

2013-10-02 Thread Michael Meissner
This patch disables -mvsx-timode from being set by default because of yet another problem with it (PR target 58587, cannot bootstrap with ada and --with-cpu=power7). I will need to address this in my later stage of secondary reload issues in order to support the ISA 2.07 128-bit integer arithmetic

libgo patch committed: Fix append of slice with zero-sized elements

2013-10-02 Thread Ian Lance Taylor
This patch to libgo fixes using append with a slice of elements of zero size. Previously this would get a division by zero error. I also fixed the code to use the correct type--intgo rather than int--and to handle overflow in the initial allocation calculation. Bootstrapped and ran Go testsuite

Re: [PATCH] fix target 58587, disable -mvsx-timode for now

2013-10-02 Thread David Edelsohn
On Wed, Oct 2, 2013 at 7:15 PM, Michael Meissner wrote: > This patch disables -mvsx-timode from being set by default because of yet > another problem with it (PR target 58587, cannot bootstrap with ada and > --with-cpu=power7). I will need to address this in my later stage of > secondary > reloa

Re: Generic tuning in x86-tune.def 1/2

2013-10-02 Thread H.J. Lu
On Fri, Sep 27, 2013 at 8:46 AM, H.J. Lu wrote: >> So I would incline to be apply extra care on this flag and keep it for extra >> release or two. Most of gcc.opensuse.org testing runs on these and adding >> random branch mispredictions will trash them. >> >> At the related note, would would you

Re: [PATCH] Reducing number of alias checks in vectorization.

2013-10-02 Thread Cong Hou
On Wed, Oct 2, 2013 at 2:47 PM, Xinliang David Li wrote: > I think you need to augment (using a wrapper class) the DDR to capture > more information about aliased memory pairs. It should be flexible > enough to handle the following cases (you don't have to handle all > cases in your first patch, b

RE: [PATCH, PR 57748] Check for out of bounds access, Part 2

2013-10-02 Thread Bernd Edlinger
Hi Eric, On Fri, 27 Sep 2013 10:36:57, Eric Botcazou wrote: > >> Sure, but the modifier is not meant to force something into memory, >> especially when it is already in an register. Remember, we are only >> talking of structures here, and we only want to access one member. >> >> It is more the ot

[committed] Don't define MULTIARCH_DIRNAME on hppa*64*-*-linux*

2013-10-02 Thread John David Anglin
Currently, there is no glibc port for the hppa64-linux target and there are no headers. It is not built with --enable-multiarch, so we don't need to define MULTIARCH_DIRNAME. Using the 32-bit directory is also wrong. This patch removes adding the pa/t-linux file from the tmake_file variabl

[C++ Patch] PR 58584

2013-10-02 Thread Paolo Carlini
Hi, to avoid these ICE on invalid with C++11 alignas, we can simply avoid calling save_template_attributes and cp_check_const_attributes when something went wrong and attributes is error_mark_node. The parser bit just tidies the diagnostic, ie, avoids redundant messages about semicolons, etc

Re: Honnor ix86_accumulate_outgoing_args again

2013-10-02 Thread Vladimir Makarov
On 13-10-02 6:45 PM, Jan Hubicka wrote: So I thing we ought to honnor accumulate-outgoing-args again and in fact consider disabling it for generic - it is disabled for core (that may need re-benchmarking). For all AMD targets it is currently on. I tested disabling it on buldozer 32bit and it see

patch to fix building Ada on ppc64 with LRA

2013-10-02 Thread Vladimir Makarov
The following patch fixes building Ada on ppc64 with LRA (elimination part changes which makes it behave as reload pass code). The patch also fixes a failure on new test added for ppc recently. This change is in very sensitive part of LRA and there is possibility that other targets can be aff

[committed] Define extra_parts on hppa*64*-*-linux* and hppa*-*-linux*

2013-10-02 Thread John David Anglin
The hppa*-*-linux* targets don't support libvtv. The default linux define for extra_parts now includes vtv_start.o vtv_end.o vtv_start_preinit.o and vtv_end_preinit.o. These are not built and cause extraneous install warnings. Tested on both targets and committed to trunk. Dave -- John Dav

[patch] Add tree-ssa-loop.h and friends.

2013-10-02 Thread Andrew MacLeod
this patch consolidates tree-ssa-loop*.c files with new .h files as required (8 in total) A number of the prototypes were in tree-flow.h, but there were also a few in cfgloop.h. tree-ssa-loop.h was created to contain a couple of common structs and act as the gathering file for any generally

Re: [PATCH]Fix computation of offset in ivopt

2013-10-02 Thread Bin.Cheng
Sorry that I don't understand tree type system well, so here are two more questions, could you please explain little bit more? Thanks very much. On Tue, Oct 1, 2013 at 6:50 PM, Richard Biener wrote: > On Mon, Sep 30, 2013 at 7:39 AM, bin.cheng wrote: > > I don't think you need > > + /* Sign ex

Re: operator new returns nonzero

2013-10-02 Thread Marc Glisse
On Wed, 2 Oct 2013, Jason Merrill wrote: On 10/02/2013 08:33 AM, Marc Glisse wrote: + if (flag_delete_null_pointer_checks && !flag_check_new You can't use flag_check_new in language-independent code without moving it from c.opt to common.opt. New version, tested with bootstrap+testsu

<    1   2