Re: operator new returns nonzero

2013-10-02 Thread Marc Glisse
It isn't a front-end patch, but it is still a C++ patch, maybe Jason will have comments? Anyone else? On Mon, 16 Sep 2013, Marc Glisse wrote: Nobody has expressed concern for a week, so it may be worth doing an official review ;-) http://gcc.gnu.org/ml/gcc-patches/2013-09/msg00676.html On M

Re: operator new returns nonzero

2013-10-02 Thread Jakub Jelinek
On Mon, Sep 09, 2013 at 10:49:40PM +0200, Marc Glisse wrote: > --- fold-const.c (revision 202413) > +++ fold-const.c (working copy) > @@ -16171,21 +16171,31 @@ tree_expr_nonzero_warnv_p (tree t, bool > case MODIFY_EXPR: > case BIND_EXPR: >return tree_expr_nonzero_warnv_p

RE: [PING] 3 patches waiting for approval/review

2013-10-02 Thread Paulo Matos
> -Original Message- > From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On > Behalf Of Andreas Krebbel > Sent: 01 October 2013 10:18 > To: gcc-patches@gcc.gnu.org > Subject: [PING] 3 patches waiting for approval/review > > [RFC] Allow functions calling mcount bef

Re: operator new returns nonzero

2013-10-02 Thread Marc Glisse
On Wed, 2 Oct 2013, Jakub Jelinek wrote: On Mon, Sep 09, 2013 at 10:49:40PM +0200, Marc Glisse wrote: --- fold-const.c(revision 202413) +++ fold-const.c(working copy) @@ -16171,21 +16171,31 @@ tree_expr_nonzero_warnv_p (tree t, bool case MODIFY_EXPR: case BIND_EXPR:

Re: [PATCH] Fix PR58554

2013-10-02 Thread Richard Biener
On Tue, 1 Oct 2013, Bernhard Reutner-Fischer wrote: > On 30 September 2013 14:19:01 Richard Biener wrote: > > > > This fixes PR58554, pattern recognition in loop distribution now > > needs to check whether all stmts are unconditionally executed. > > > > Bootstrapped and tested on x86_64-unknown

Re: [PATCH, doc]: Fix "@anchor should not appear in @heading" warning

2013-10-02 Thread Gerald Pfeifer
On Sun, 29 Sep 2013, Uros Bizjak wrote: > Rather trivial fix - put @anchor before @heading, as texi manual suggests. > > 2013-09-29 Uros Bizjak > > * doc/install.texi (Host/target specific installation notes for GCC): > Put @anchor before @heading. > > Tested by "make doc" with texinf

Re: [wwwdocs] Buildstat update for 4.4

2013-10-02 Thread Gerald Pfeifer
On Tue, 1 Oct 2013, Tom G. Christensen wrote: > Testresults for 4.4.7: > i386-pc-solaris2.8 (2) > i386-pc-solaris2.9 > sparc-sun-solaris2.7 (2) > sparc-sun-solaris2.8 > sparc-sun-solaris2.9 Thanks, applied. Gerald

Re: [PATCH, doc]: Fix "@anchor should not appear in @heading" warning

2013-10-02 Thread Uros Bizjak
On Wed, Oct 2, 2013 at 10:12 AM, Gerald Pfeifer wrote: > On Sun, 29 Sep 2013, Uros Bizjak wrote: >> Rather trivial fix - put @anchor before @heading, as texi manual suggests. >> >> 2013-09-29 Uros Bizjak >> >> * doc/install.texi (Host/target specific installation notes for GCC): >> Put

Re: [patch] Move some prototypes out of tree-flow.h

2013-10-02 Thread Richard Biener
On Tue, Oct 1, 2013 at 5:18 PM, Andrew MacLeod wrote: > This patch moves 5 sets of prototypes out of tree-flow.h and creates 4 new > header files. > > I then #include the new header files from tree-flow.h as a temporary > measure until all prototypes have been cleared up. then as previously > dis

Re: [PING] 3 patches waiting for approval/review

2013-10-02 Thread Andreas Krebbel
On 02/10/13 09:10, Paulo Matos wrote: > >> -Original Message- >> From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On >> Behalf Of Andreas Krebbel >> Sent: 01 October 2013 10:18 >> To: gcc-patches@gcc.gnu.org >> Subject: [PING] 3 patches waiting for approval/review

Re: [PATCH]Fix computation of offset in ivopt

2013-10-02 Thread Richard Biener
On Tue, Oct 1, 2013 at 6:13 PM, Bin.Cheng wrote: > 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 extend off if expr is in type which has lower precision >> + than HOST_WIDE_IN

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

2013-10-02 Thread Richard Biener
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 include gimple-fold.h from > gimple.h instead

Re: [PATCH] Improving uniform_vector_p() function.

2013-10-02 Thread Richard Biener
On Tue, Oct 1, 2013 at 7:31 PM, Cong Hou wrote: > The current uniform_vector_p() function only returns non-NULL when the > vector is directly a uniform vector. For example, for the following > gimple code: > > vect_cst_.15_91 = {_9, _9, _9, _9, _9, _9, _9, _9}; > > > The current implementation can

Re: [wwwdocs] Buildstat update for 4.8

2013-10-02 Thread Gerald Pfeifer
On Tue, 1 Oct 2013, Tom G. Christensen wrote: > Latest results for gcc 4.8.x. > > -tgc > > Testresults for 4.8.1 > sparc64-sun-solaris2.9 Thanks! Gerald

[C++ Patch] PR 58565

2013-10-02 Thread Paolo Carlini
Hi, here, with -std=c++11 LABEL_EXPR is unhandled by potential_constant_expression_1 and we can't emit a meaningful diagnostic. It seems safe to just add it to the main switch, together with the existing LABEL_DECL. Tested x86_64-linux. Thanks, Paolo. // /cp 2013-10-02

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

2013-10-02 Thread Joern Rennecke
Quoting Jakub Jelinek : On Tue, Oct 01, 2013 at 04:22:38PM -0600, Jeff Law wrote: >>- The Copyright years should be 2013 in every new file. Or has this >>port been released before? > >The port has been available via git for quite a while: >https://github.com/foss-for-synopsys-dwc-arc-processor

[PATCH][ARM][committed] Add comment in arm_legitimize_reload_address to discourage use of plus_constant

2013-10-02 Thread Kyrill Tkachov
Hi all, I've committed the attached patch to make it mode clear why we use gen_rtx_PLUS instead of plus_constant in that particular instance. Committed as obvious with r203106. Kyrill 2013-10-02 Kyrylo Tkachov * config/arm/arm.c (arm_legitimize_reload_address): Explain why plus_c

Re: [C++ Patch] PR 58565

2013-10-02 Thread Jason Merrill
OK. Jason

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

2013-10-02 Thread Jakub Jelinek
On Wed, Oct 02, 2013 at 06:05:14AM -0400, 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 rele

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

2013-10-02 Thread Gerald Pfeifer
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 bit weak. I think I understand where it is coming

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

2013-10-02 Thread Joern Rennecke
Quoting Jakub Jelinek : On Wed, Oct 02, 2013 at 06:05:14AM -0400, 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'

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

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

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

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

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

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

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: 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: 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: [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: [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. ;)

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

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

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

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

[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

[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

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

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

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

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

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

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

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

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

Re: [C++ Patch] PR 58535

2013-10-02 Thread Jason Merrill
OK. Jason

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

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

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

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

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

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

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.

[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

[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

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

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

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

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

  1   2   >