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

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

[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

[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 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

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

[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

[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

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

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: 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] 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

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

[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

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) {

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
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: [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] 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: 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

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: PING: Fwd: Re: [patch] implement Cilk Plus simd loops on trunk

2013-10-02 Thread Jason Merrill
On 08/27/2013 04:03 PM, Aldy Hernandez wrote: + /* First, try to parse as an initialized declaration. See + cp_parser_condition, from whence the bulk of this is copied. */ You didn't do this copy, but I'd appreciate it if you could reintegrate this with cp_parser_condition. I notice th

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

2013-10-02 Thread Xinliang David Li
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, but keep those in mind). 1) All accesses in the same group have co

Re: [PATCH GCC] Tweak gimple-ssa-strength-reduction.c:backtrace_base_for_ref () to cover different cases as seen on AArch64

2013-10-02 Thread Bill Schmidt
On Wed, 2013-10-02 at 07:40 -0500, Bill Schmidt wrote: > On Tue, 2013-10-01 at 20:21 -0500, Bill Schmidt wrote: > > On Tue, 2013-10-01 at 23:57 +0100, Yufeng Zhang wrote: > > > On 10/01/13 20:55, Bill Schmidt wrote: > > > > > > > > > > > > On Tue, 2013-10-01 at 11:56 -0500, Bill Schmidt wrote: > >

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

2013-10-02 Thread Cong Hou
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 each >> pair of data refs with possible data dependence, there wi

Re: [PATCH] alternative hirate for builtin_expert

2013-10-02 Thread Jan Hubicka
> 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 retire the hitrate in PRED_BUILTIN_EXPECT and always use > the one specified in the biniltin-expect-probability parameter.

[PATCH, build]: Update x-linux and t-linux-android for automatic dependencies

2013-10-02 Thread Uros Bizjak
Hello! 2013-10-02 Uros Bizjak * config/x-linux (host-linux.o): Remove header dependencies. Use $(COMPILE) and $(POSTCOMPILE). * config/t-linux-android (linux-android.o): Ditto. Bootstrapped on x86_64-pc-linux-gnu and committed to mainline. Uros. Index: config/t-linux-android

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

2013-10-02 Thread Xinliang David Li
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 each >> pair of data refs with possible data dependence, there wi

Re: [Patch, Fortran, committed] PR58579 - fix allocation of string temporaries: Avoid overallocation

2013-10-02 Thread Tobias Burnus
Tobias Burnus wrote: In gfc_conv_string_tmp, gfortran allocates temporary strings. However, using "TYPE_SIZE (type)" didn't yield one byte as intended but 64 - which means that gfortran allocated 64 times as much memory as needed. Committed (Rev. ) after building and regtesting on x86-64-gnu-l

[v3 patch] fix libstdc++/58569

2013-10-02 Thread Jonathan Wakely
2013-10-02 Jonathan Wakely Daniel Krugler PR libstdc++/58569 * include/std/functional (function::_CheckResult): Move to namespace scope and rename to __check_func_return_type. * testsuite/20_util/function/58569.cc: New. Tested x86_64-linux, committ

[v3 patch] fix libstdc++/58594

2013-10-02 Thread Jonathan Wakely
PR libstdc++/58594 * include/bits/shared_ptr_base.h (_Sp_counted_ptr_inplace::_M_get_deleter()): Cast away cv-quals. * testsuite/20_util/shared_ptr/creation/58594.cc: New. Tested x86_64-linux, committed to trunk commit a58a4bea9475af3e3c44959aeab4b3ac48dc1af0 Author

Re: [PATCH, build]: Remove -Wno-warning from expmed.c compilation

2013-10-02 Thread Uros Bizjak
On Wed, Oct 2, 2013 at 1:55 PM, Uros Bizjak wrote: > Compiling expmed.c has been warning free for some time now. As discussed briefly on IRC: let's try -Werror and see which target will break. Committed to mainline. Uros.

Re: [patch] More tree-flow.h prototypes.

2013-10-02 Thread Andrew MacLeod
On 10/02/2013 03:09 PM, Tobias Burnus wrote: This patch (rev. 203118) seems to break bootstrapping with Graphite: g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-lon

Go patch committed: Use backend interface for numeric constants

2013-10-02 Thread Ian Lance Taylor
This patch from Chris Manghane changes the Go frontend to use the backend interface for numeric constants. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline and 4.8 branch. Ian 2013-10-02 Chris Manghane * go-gcc.cc: Include "real.h" and "realmpfr.

Re: [patch] More tree-flow.h prototypes.

2013-10-02 Thread Tobias Burnus
This patch (rev. 203118) seems to break bootstrapping with Graphite: g++ -c -g -DIN_GCC -fno-exceptions -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wno-narrowing -Wwrite-strings -Wcast-qual -Wmissing-format-attribute -pedantic -Wno-long-long -Wno-variadic-macros -Wno-overlength-string

Re: RFA: GNU make 3.80 compatibility

2013-10-02 Thread Jakub Jelinek
On Wed, Oct 02, 2013 at 12:41:32PM -0600, Tom Tromey wrote: > This patch makes the automatic dependency tracking code compatible with > GNU make 3.80, which is documented as the oldest supported version. > This was noticed by Eric Botcazou: > > http://gcc.gnu.org/ml/gcc/2013-09/msg00243.html >

Re: [PATCH] alternative hirate for builtin_expert

2013-10-02 Thread Rong Xu
On Wed, Oct 2, 2013 at 9:08 AM, Jan Hubicka wrote: >> > Hi, >> > >> > >> > >> > Current default probability for builtin_expect is 0.9996. >> > This makes the freq of unlikely bb very low (4), which >> > suppresses the inlining of any calls within those bb. >> > >> > We used FDO data to measure t

RFA: GNU make 3.80 compatibility

2013-10-02 Thread Tom Tromey
This patch makes the automatic dependency tracking code compatible with GNU make 3.80, which is documented as the oldest supported version. This was noticed by Eric Botcazou: http://gcc.gnu.org/ml/gcc/2013-09/msg00243.html Ok? Tom 2013-10-02 Tom Tromey * Makefile.in (DRIVER_DEFI

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

2013-10-02 Thread Dehao Chen
On Wed, Oct 2, 2013 at 10:50 AM, Cong Hou wrote: > On Tue, Oct 1, 2013 at 11:35 PM, Jakub Jelinek wrote: >> On Tue, Oct 01, 2013 at 07:12:54PM -0700, Cong Hou wrote: >>> --- gcc/tree-vect-loop-manip.c (revision 202662) >>> +++ gcc/tree-vect-loop-manip.c (working copy) >> >> Your mailer ate all th

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

2013-10-02 Thread Jakub Jelinek
On Wed, Oct 02, 2013 at 10:50:21AM -0700, Cong Hou wrote: > >> + if (int_cst_value (p11.offset) != int_cst_value (p21.offset)) > >> +return int_cst_value (p11.offset) < int_cst_value (p21.offset); > > > > This is going to ICE whenever the offsets wouldn't fit into a > > HOST_WIDE_INT. > > > >

Re: [PATCH] Relax the requirement of reduction pattern in GCC vectorizer.

2013-10-02 Thread Cong Hou
Ping.. Any comment on this patch? thanks, Cong On Sat, Sep 28, 2013 at 9:34 AM, Xinliang David Li wrote: > You can also add a test case of this form: > > int foo( int t, int n, int *dst) > { >int j = 0; >int s = 1; >t++; >for (j = 0; j < n; j++) > { > dst[j] = t;

Re: [PATCH] Sanitize block partitioning under -freorder-blocks-and-partition

2013-10-02 Thread Jan Hubicka
> But why do we want to consider blocks as "probably never executed" > when the frequency suggests they are sometimes executed? Well, probably never executed is mean to reffer to one run. If you have something like code handling fatal errors, you probably still want to have it in cold secion even

Re: [PATCH] Sanitize block partitioning under -freorder-blocks-and-partition

2013-10-02 Thread Teresa Johnson
On Wed, Oct 2, 2013 at 9:19 AM, Jan Hubicka wrote: >> 2013-09-29 Teresa Johnson >> >> * bb-reorder.c >> (find_rarely_executed_basic_blocks_and_crossing_edges): >> Treat profile insanities conservatively. >> * predict.c (probably_never_executed): New function. Treat prof

[patch] tree-eh.c prototypes

2013-10-02 Thread Andrew MacLeod
This patch moves the prototypes for tree-eh.c into a new file tree-eh.h. This file is in fact really gimple-eh.. we'll rename that later with the other tree->gimple renaming that is needed. however, using_eh_for_cleanups() is in fact a front end routine which is called when eh regions are us

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

2013-10-02 Thread Cong Hou
On Tue, Oct 1, 2013 at 11:35 PM, Jakub Jelinek wrote: > On Tue, Oct 01, 2013 at 07:12:54PM -0700, Cong Hou wrote: >> --- gcc/tree-vect-loop-manip.c (revision 202662) >> +++ gcc/tree-vect-loop-manip.c (working copy) > > Your mailer ate all the tabs, so the formatting of the whole patch > can't be c

Honnor ix86_accumulate_outgoing_args again

2013-10-02 Thread Jan Hubicka
Hi, currently ix86_accumulate_outgoing_args is ignored on all targets except for Solaris (that sets USE_IX86_FRAME_POINTER to true). It seems like accidental effect of http://gcc.gnu.org/ml/gcc-patches/2010-08/txt00102.txt that enabled omit-frame-pointer for 32bit (I take the 64bit change was pure

Re: libgo patch committed: Implement reflect.MakeFunc for 386

2013-10-02 Thread Ian Lance Taylor
On Wed, Oct 2, 2013 at 7:45 AM, Rainer Orth wrote: > > Here's what I came up with. As I said, it is inspired by the libffi > code, but a bit simplified since e.g. stuff like no .ascii support > aren't relevant on the Solaris versions supported on mainline and 4.8 > branch. > > Bootstrapped on x8

Re: [PATCH] Improve probability/profile distribution in ORIF expansion

2013-10-02 Thread Teresa Johnson
On Wed, Oct 2, 2013 at 9:03 AM, Jan Hubicka wrote: >> 2013-10-01 Teresa Johnson >> >> * dojump.c (do_jump_1): Divide probability between >> both conditions of a TRUTH_ORIF_EXPR. >> >> + { >> +/* Spread the probability evenly between the two conditions. So >> +

Re: [patch] More tree-flow.h prototypes.

2013-10-02 Thread Richard Biener
Andrew MacLeod wrote: >On 10/02/2013 07:58 AM, Andrew MacLeod wrote: >> On 10/02/2013 04:37 AM, Richard Biener wrote: >>> On Tue, Oct 1, 2013 at 11:01 PM, Andrew MacLeod > >>> wrote: This patch moves prototypes into gimple-fold.h (which already >existed). There were a few in tree-flow.h

Re: [patch] move phiopt, ssa-dce and ssa-dom prototypes.

2013-10-02 Thread Richard Biener
Andrew MacLeod wrote: >Handle a few more prototypes in tree-flow.h > >There were only 2 routines exported from tree-ssa-phiopts, and neither >really belonged there. >* I moved nonfreeing_call_p() to gimple.c since it is gimple dependent. >blocks_in_phiopt_order() returns basic blocks in an order

Re: [C++ Patch] PR 58535

2013-10-02 Thread Jason Merrill
OK. Jason

Re: [c++-concepts] constrained friends redux

2013-10-02 Thread Jason Merrill
On 10/02/2013 09:05 AM, Andrew Sutton wrote: + // Do not permit the declaration of constrained friend + // function declarations. They cannot be instantiated since + // the resulting declaration would never match the definition, + // which must be a non-templat

[PATCH, build]: Update x-i386 and x-alpha for automatic dependencies

2013-10-02 Thread Uros Bizjak
Hello! 2013-10-02 Uros Bizjak * config/i386/x-i386 (driver-i386.o): Remove header dependencies. Use $(COMPILE) and $(POSTCOMPILE). * config/alpha/x-alpha (driver-alpha.o): Ditto. Bootstrapped on x86_64-pc-linux-gnu and alphaev68-pc-linux-gnu, committed to mainline SVN. Uros. Ind

Re: [C++ Patch] PR 58535

2013-10-02 Thread Paolo Carlini
... oops attached the patch which I just committed. Sorry. Right attachments below. Thanks, Paolo. // /cp 2013-10-02 Paolo Carlini PR c++/58535 * parser.c (cp_parser_function_specifier_opt): Upon error about virtual templates don't set ds_virtual

Re: [PATCH] Sanitize block partitioning under -freorder-blocks-and-partition

2013-10-02 Thread Jan Hubicka
> 2013-09-29 Teresa Johnson > > * bb-reorder.c (find_rarely_executed_basic_blocks_and_crossing_edges): > Treat profile insanities conservatively. > * predict.c (probably_never_executed): New function. Treat profile > insanities conservatively. > (probably

[C++ Patch] PR 58535

2013-10-02 Thread Paolo Carlini
Hi, in this [4.8/4.9] diagnostic regression the gcc_assert in check_member_templates trips: /* The parser rejects any use of virtual in a function template. */ gcc_assert (!(TREE_CODE (decl) == FUNCTION_DECL && DECL_VIRTUAL_P (decl))); the ultimate reason being that i

Re: [PATCH] fix size_estimation for builtin_expect

2013-10-02 Thread Jan Hubicka
> > 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 size/time. The end result > > is: for w/ and w/o builtin_expect, we have different size/time estimatio

[PATCH][4.8] S/390: Transactional memory fixes

2013-10-02 Thread Andreas Krebbel
Hi, with the attached patch we support more operand types in the tabort and tbegin_retry builtins. The patch also removes the constraint letters in the expanders and fixes a builtin prototype in the documentation. The testcase is adjusted accordingly. Bootstrapped and regtested on s390 and s390

Re: [PATCH] alternative hirate for builtin_expert

2013-10-02 Thread Jan Hubicka
> > Hi, > > > > > > > > Current default probability for builtin_expect is 0.9996. > > This makes the freq of unlikely bb very low (4), which > > suppresses the inlining of any calls within those bb. > > > > We used FDO data to measure the branch probably for > > the branch annotated with builtin_

Re: [PATCH] Improve probability/profile distribution in ORIF expansion

2013-10-02 Thread Jan Hubicka
> 2013-10-01 Teresa Johnson > > * dojump.c (do_jump_1): Divide probability between > both conditions of a TRUTH_ORIF_EXPR. > > + { > +/* Spread the probability evenly between the two conditions. So > + the first condition has half the total probability of

Re: [PATCH][RFC] Detect and use implementations of BLAS routines

2013-10-02 Thread Alexander Monakov
Hello, You probably want to disable this transformation when the number of iterations is predicted to be small, right? Shouldn't dot product transform be predicated on -fassociative-math? Do you have a vision of a generalized pattern matcher to allow adding other routines easily? I'm curious wh

Re: [Patch] Fix incorrect behavior of "[[=a=]]" in regex

2013-10-02 Thread Jonathan Wakely
On 2 October 2013 15:26, Tim Shen wrote: > _BracketMatcher<>::_M_add_equivalence_class is misimplemented so I try > `git blame regex_compiler.h`...that's me! > > Booted and tested under -m32, -m64. This is OK to commit, thanks

Re: [Patch] Fix incorrect behavior of "[[=a=]]" in regex

2013-10-02 Thread Tim Shen
Committed. Thanks! On Wed, Oct 2, 2013 at 11:10 AM, Jonathan Wakely wrote: > On 2 October 2013 15:26, Tim Shen wrote: >> _BracketMatcher<>::_M_add_equivalence_class is misimplemented so I try >> `git blame regex_compiler.h`...that's me! >> >> Booted and tested under -m32, -m64. > > This is OK to

Re: [wwwdocs] Mention libstdc++-v3 in 4.9 changes.html

2013-10-02 Thread Jonathan Wakely
On 2 October 2013 15:52, Tim Shen wrote: > I feel little bit uncomfortable with "new ISO C++ standard, C++11", > since C++14 is already there, so I removed it. Good idea. > Please check the words, since English is not my first language >.< The english is fine, please wait a few hours in case any

[patch] move phiopt, ssa-dce and ssa-dom prototypes.

2013-10-02 Thread Andrew MacLeod
Handle a few more prototypes in tree-flow.h There were only 2 routines exported from tree-ssa-phiopts, and neither really belonged there. * I moved nonfreeing_call_p() to gimple.c since it is gimple dependent. blocks_in_phiopt_order() returns basic blocks in an order that guarantees any single

Re: [PATCH][RFC] Detect and use implementations of BLAS routines

2013-10-02 Thread Tobias Burnus
Richard Biener wrote: > This adds recognition of [sd]axpy and [sd]dot computing partitions > to loop distribution (as an example for a moderately complex kernel > and one kernel involving a reduction). > > To make this a reality we have to control this by an option > (-fblas?) and we have to settl

Re: [wwwdocs] Mention libstdc++-v3 in 4.9 changes.html

2013-10-02 Thread Tim Shen
I feel little bit uncomfortable with "new ISO C++ standard, C++11", since C++14 is already there, so I removed it. Please check the words, since English is not my first language >.< Thanks! Index: htdocs/index.html === RCS file: /c

Re: [PATCH, i386, MPX, 1/X] Support of Intel MPX ISA. 1/2 Bound type and modes

2013-10-02 Thread Ilya Enkovich
Ping 2013/9/17 Ilya Enkovich : > Hi, > > Here is a patch introducing new type and mode for bounds. It is a part of MPX > ISA support patch (http://gcc.gnu.org/ml/gcc-patches/2013-07/msg01094.html). > > Bootstrapped and tested on linux-x86_64. Is it OK for trunk? > > Thanks, > Ilya > -- > > gcc/ >

Re: libgo patch committed: Implement reflect.MakeFunc for 386

2013-10-02 Thread Rainer Orth
Ian Lance Taylor writes: > On Mon, Sep 30, 2013 at 7:07 AM, Rainer Orth > wrote: >> Ian Lance Taylor writes: >> >>> On Mon, Sep 30, 2013 at 6:07 AM, Rainer Orth >>> wrote: Ian Lance Taylor writes: > Following up on my earlier patch, this patch implements the > reflect.MakeFu

Re: [PATCH GCC] Tweak gimple-ssa-strength-reduction.c:backtrace_base_for_ref () to cover different cases as seen on AArch64

2013-10-02 Thread Yufeng Zhang
On 10/02/13 13:40, Bill Schmidt wrote: On Tue, 2013-10-01 at 20:21 -0500, Bill Schmidt wrote: On Tue, 2013-10-01 at 23:57 +0100, Yufeng Zhang wrote: On 10/01/13 20:55, Bill Schmidt wrote: On Tue, 2013-10-01 at 11:56 -0500, Bill Schmidt wrote: OK, thanks. The problem that you've encountered

[Patch] Fix incorrect behavior of "[[=a=]]" in regex

2013-10-02 Thread Tim Shen
_BracketMatcher<>::_M_add_equivalence_class is misimplemented so I try `git blame regex_compiler.h`...that's me! Booted and tested under -m32, -m64. Thanks ;) -- Tim Shen a.patch Description: Binary data

[PATCH][RFC] Detect and use implementations of BLAS routines

2013-10-02 Thread Richard Biener
This adds recognition of [sd]axpy and [sd]dot computing partitions to loop distribution (as an example for a moderately complex kernel and one kernel involving a reduction). To make this a reality we have to control this by an option (-fblas?) and we have to settle to an ABI we rely on (trailing

Re: [patch] More tree-flow.h prototypes.

2013-10-02 Thread Richard Biener
On Wed, Oct 2, 2013 at 1:58 PM, Andrew MacLeod wrote: > On 10/02/2013 04:37 AM, Richard Biener wrote: >> >> On Tue, Oct 1, 2013 at 11:01 PM, Andrew MacLeod >> wrote: >>> >>> This patch moves prototypes into gimple-fold.h (which already existed). >>> There were a few in tree-flow.h and a bunch in

Re: Make the 2 versions of delete more similar

2013-10-02 Thread Jonathan Wakely
On 2 October 2013 13:28, Marc Glisse wrote: > Hello, > > I don't understand why those 2 files differ by more than 1 extra argument, > so I am changing that. > > Bootstrap and testsuite on x86_64. > > 2013-10-03 Marc Glisse > > * libsupc++/del_op.cc (operator delete): Don't test for 0 bef

[PATCH] More loop distribution TLC

2013-10-02 Thread Richard Biener
I split out some TLC to loop distribution from a patch I'll post shortly. Bootstrapped on x86_64-unknown-linux-gnu, testing in progress. Richard. 2013-10-02 Richard Biener * tree-loop-distribution.c: Include tree-vectorizer.h for find_loop_location. (enum partition_k

Re: Make the 2 versions of delete more similar

2013-10-02 Thread Marc Glisse
On Wed, 2 Oct 2013, Christopher Jefferson wrote: On 2 October 2013 13:28, Marc Glisse wrote: Hello, I don't understand why those 2 files differ by more than 1 extra argument, so I am changing that. Bootstrap and testsuite on x86_64. 2013-10-03 Marc Glisse * libsupc++/del_op.cc (

Re: operator new returns nonzero

2013-10-02 Thread Jakub Jelinek
On Wed, Oct 02, 2013 at 04:12:24PM +0300, Marc Glisse wrote: > 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

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. Thanks, that makes sense and I'll do that

[c++-concepts] constrained friends redux

2013-10-02 Thread Andrew Sutton
This patch implements constrained friends and disallows declarations of constrained friend template specialization. There was a previous question about whether I was doing the right thing in determine_specialization. I'm looking at that issue separately. 2013-10-01 Andrew Sutton * gcc/

Re: [PATCH GCC] Tweak gimple-ssa-strength-reduction.c:backtrace_base_for_ref () to cover different cases as seen on AArch64

2013-10-02 Thread Yufeng Zhang
On 10/02/13 02:21, Bill Schmidt wrote: On Tue, 2013-10-01 at 23:57 +0100, Yufeng Zhang wrote: On 10/01/13 20:55, Bill Schmidt wrote: On Tue, 2013-10-01 at 11:56 -0500, Bill Schmidt wrote: OK, thanks. The problem that you've encountered is that you are attempting to do something illegal. ;)

Re: [patch] More tree-flow.h prototypes.

2013-10-02 Thread Andrew MacLeod
On 10/02/2013 07:58 AM, Andrew MacLeod wrote: On 10/02/2013 04:37 AM, Richard Biener wrote: On Tue, Oct 1, 2013 at 11:01 PM, Andrew MacLeod wrote: This patch moves prototypes into gimple-fold.h (which already existed). There were a few in tree-flow.h and a bunch in gimple.h. The routines are

Re: operator new returns nonzero

2013-10-02 Thread Jason Merrill
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. Jason

Re: Make the 2 versions of delete more similar

2013-10-02 Thread Christopher Jefferson
On 2 October 2013 13:28, Marc Glisse wrote: > Hello, > > I don't understand why those 2 files differ by more than 1 extra argument, > so I am changing that. > > Bootstrap and testsuite on x86_64. > > 2013-10-03 Marc Glisse > > * libsupc++/del_op.cc (operator delete): Don't test for 0 be

Re: [PATCH GCC] Tweak gimple-ssa-strength-reduction.c:backtrace_base_for_ref () to cover different cases as seen on AArch64

2013-10-02 Thread Bill Schmidt
On Tue, 2013-10-01 at 20:21 -0500, Bill Schmidt wrote: > On Tue, 2013-10-01 at 23:57 +0100, Yufeng Zhang wrote: > > On 10/01/13 20:55, Bill Schmidt wrote: > > > > > > > > > On Tue, 2013-10-01 at 11:56 -0500, Bill Schmidt wrote: > > >> OK, thanks. The problem that you've encountered is that you are

Re: operator new returns nonzero

2013-10-02 Thread Marc Glisse
New version after Jakub's comment, bootstrap and testsuite on x86_64. 2013-10-03 Marc Glisse PR c++/19476 gcc/ * calls.c (alloca_call_p): Use get_callee_fndecl. * fold-const.c (tree_expr_nonzero_warnv_p): Handle operator new. * tree-vrp.c (gimple_stmt_nonzero_w

Make the 2 versions of delete more similar

2013-10-02 Thread Marc Glisse
Hello, I don't understand why those 2 files differ by more than 1 extra argument, so I am changing that. Bootstrap and testsuite on x86_64. 2013-10-03 Marc Glisse * libsupc++/del_op.cc (operator delete): Don't test for 0 before free. * libsupc++/del_opnt.cc (free): Only de

Re: [patch] More tree-flow.h prototypes.

2013-10-02 Thread Andrew MacLeod
On 10/02/2013 04:37 AM, Richard Biener wrote: On Tue, Oct 1, 2013 at 11:01 PM, Andrew MacLeod wrote: This patch moves prototypes into gimple-fold.h (which already existed). There were a few in tree-flow.h and a bunch in gimple.h. The routines are used frequently enough that it makes sense to in

[PATCH, build]: Remove -Wno-warning from expmed.c compilation

2013-10-02 Thread Uros Bizjak
Hello! Compiling expmed.c has been warning free for some time now. 2013-10-02 Uros Bizjak * Makefile.in (expmed.o-warn): Remove. Bootstrapped on x86_64-pc-linux-gnu. OK for mainline? Uros. Index: Makefile.in === --- Makef

Re: [patch] move htab_iterator

2013-10-02 Thread Andrew MacLeod
On 10/01/2013 05:04 PM, DJ Delorie wrote: I'm typically against adding things to libiberty "because there's no other place for them". The purpose of libiberty is to provide a portability layer, not a trash can. However, htab is already in there, and the argument for putting its accessors there

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

2013-10-02 Thread Richard Biener
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 each > pair of data refs with possible data dependence, there will be two > comparisons generated to make sure there is no al

Re: Copyright years for new old ports (Re: Ping^6: contribute Synopsys Designware ARC port)

2013-10-02 Thread Joern Rennecke
Quoting Gerald Pfeifer : On Wed, 2 Oct 2013, Joern Rennecke wrote: From my understanding, the condition for adding the current Copyright year without a source code change is to have a release in that year. Are we sure 4.9.0 will be released this year? We are sure we don't want 4.9.0 to be rel

Re: [build] Update t-sparc, t-sol2 etc. for automatic dependencies

2013-10-02 Thread Paolo Bonzini
Il 02/10/2013 12:39, Rainer Orth ha scritto: > Inspired by the t-i386 changes, the following patch moves SPARC and > Solaris files over to automatic dependencies. > > Bootstrapped without regression on sparc-sun-solaris2.11, verified that > dependencies were generated for affected files. > > Ok f

Re: [wwwdocs] Mention libstdc++-v3 in 4.9 changes.html

2013-10-02 Thread Paolo Carlini
On 10/02/2013 12:59 PM, Jakub Jelinek wrote: We have announced only core language feature completeness, the library was known to be incomplete. And, I think for 4.9 the library C++11 support is still meant to be experimental because of the ABI issues, where we know we'll need to change std::str

Re: [wwwdocs] Mention libstdc++-v3 in 4.9 changes.html

2013-10-02 Thread Jakub Jelinek
On Wed, Oct 02, 2013 at 12:55:48PM +0200, Paolo Carlini wrote: > On 10/02/2013 12:45 PM, Jonathan Wakely wrote: > >On 2 October 2013 11:41, Paolo Carlini wrote: > >>Minimally, I would talk about "improved support": the evolution from > >>-std=c++0x to -std=c++11 meant that we aren't in experimental

Re: [wwwdocs] Mention libstdc++-v3 in 4.9 changes.html

2013-10-02 Thread Paolo Carlini
On 10/02/2013 12:45 PM, Jonathan Wakely wrote: On 2 October 2013 11:41, Paolo Carlini wrote: Minimally, I would talk about "improved support": the evolution from -std=c++0x to -std=c++11 meant that we aren't in experimental mode anymore. From speaking to Jason he's pretty adamant it's still ex

Re: [wwwdocs] Mention libstdc++-v3 in 4.9 changes.html

2013-10-02 Thread Jonathan Wakely
On 2 October 2013 11:41, Paolo Carlini wrote: > > Minimally, I would talk about "improved support": the evolution from > -std=c++0x to -std=c++11 meant that we aren't in experimental mode anymore. >From speaking to Jason he's pretty adamant it's still experimental for now :-) My understanding was

Re: [wwwdocs] Mention libstdc++-v3 in 4.9 changes.html

2013-10-02 Thread Paolo Carlini
Hi, On 10/02/2013 12:19 PM, Gerald Pfeifer wrote: On Tue, 1 Oct 2013, Tim Shen wrote: Hi, libstdc++-v3 is ready for releasing. Nice! Is it Ok to apply? By the way, do we need a News entry for this improvement? Yes, and yes. :-) Just one question "improved experimental support" sounds a bi

[build] Update t-sparc, t-sol2 etc. for automatic dependencies

2013-10-02 Thread Rainer Orth
Inspired by the t-i386 changes, the following patch moves SPARC and Solaris files over to automatic dependencies. Bootstrapped without regression on sparc-sun-solaris2.11, verified that dependencies were generated for affected files. Ok for mainline? Rainer 2013-10-01 Rainer Orth

Re: Copyright years for new old ports (Re: Ping^6: contribute Synopsys Designware ARC port)

2013-10-02 Thread Gerald Pfeifer
On Wed, 2 Oct 2013, Joern Rennecke wrote: >>> From my understanding, the condition for adding the current Copyright year >>> without a source code change is to have a release in that year. Are we >>> sure 4.9.0 will be released this year? >> We are sure we don't want 4.9.0 to be released this year

Re: [PATCH][AARCH64]Replace gen_rtx_PLUS with plus_constant

2013-10-02 Thread Kyrill Tkachov
On 01/10/13 12:32, Marcus Shawcroft wrote: On 30 September 2013 14:20, Renlin Li wrote: gcc/ChangeLog: 2013-09-30 Renlin Li * config/aarch64/aarch64.c (aarch64_expand_prologue): Use plus_constant. (aarch64_expand_epilogue): Likewise. OK /Marcus I've committed the patch as r203

  1   2   >