Re: [PATCH, C++, CPP] Add C++1z to the preprocessor. Rename C++1y to C++14.

2014-08-24 Thread Jason Merrill
Adding a few more tweaks: commit bf054f32b165fda6e90c52a57a31538e5ae24678 Author: Jason Merrill Date: Mon Aug 25 00:48:17 2014 -0400 * c.opt: Change -std=c++1y and -std=gnu++1y to be aliases for -std=c++14 and -std=gnu++14, rather than the reverse. * c-opts.c (c_common_handle_o

Re: [GOOGLE, AUTOFDO] Assign different discriminators to calls with the same lineno

2014-08-24 Thread Wei Mi
To avoid the unused new discriminator value, I added a map "found_call_this_line" to track whether a call is the first call in a source line seen when assigning discriminators. For the first call in a source line, its discriminator is 0. For the following calls in the same source line, a new discri

Re: C++ PATCH to support non-constexpr variable templates

2014-08-24 Thread Jason Merrill
On 08/23/2014 06:29 PM, Ville Voutilainen wrote: Based on my quick tests with it, it seems to me that we need more tests for static member variable templates. My first attempt at something like that gave an ICE... Yeah, I figured there would be more to do. More tests are welcome. :) Jason

Re: [C++ RFC/Patch] PR 34938

2014-08-24 Thread Jason Merrill
OK. Jason

Re: [PING PATCH] demangler, only access valid fields for DEMANGLE_COMPONENT_FIXED_TYPE.

2014-08-24 Thread Ian Lance Taylor
Jason, can you take a look? Thanks. Ian On Tue, Aug 19, 2014 at 3:46 AM, Gary Benson wrote: > Hi all, > > I just retested this patch. The crash it fixes is still there, > and the patch still fixes it. Is this ok to commit? > > Cheers, > Gary > > Andrew Burgess wrote: >> In two places when a s

[SH] Fix PR target/62111

2014-08-24 Thread Kaz Kojima
I've applied the attached patch to fix PR target/62111 which is a sh64 specific 4.9/5 regression. It seems that *zero_extendhisi2_media makes reload unhappy with permitting (truncate (pseudo_reg)) when pseudo_reg can be substituted by mem. See https://gcc.gnu.org/bugzilla/show_bug.cgi?id=62111 f

Re: [PATCH] Refactor handle_section_attribute to reduce nesting and distinguish error cases

2014-08-24 Thread Josh Triplett
On Sun, Aug 24, 2014 at 05:47:23PM -0400, Trevor Saunders wrote: > On Sun, Aug 24, 2014 at 01:58:52PM -0700, Andrew Pinski wrote: > > On Sun, Aug 24, 2014 at 1:42 PM, Josh Triplett > > wrote: > > > handle_section_attribute contains many levels of nested conditionals and > > > branching code flow

Re: [PATCH] Refactor handle_section_attribute to reduce nesting and distinguish error cases

2014-08-24 Thread Trevor Saunders
On Sun, Aug 24, 2014 at 01:58:52PM -0700, Andrew Pinski wrote: > On Sun, Aug 24, 2014 at 1:42 PM, Josh Triplett wrote: > > handle_section_attribute contains many levels of nested conditionals and > > branching code flow paths, with the error cases sometimes in the else > > case and sometimes in th

Re: [PATCH] Refactor handle_section_attribute to reduce nesting and distinguish error cases

2014-08-24 Thread Josh Triplett
On Sun, Aug 24, 2014 at 01:58:52PM -0700, Andrew Pinski wrote: > On Sun, Aug 24, 2014 at 1:42 PM, Josh Triplett wrote: > > handle_section_attribute contains many levels of nested conditionals and > > branching code flow paths, with the error cases sometimes in the else > > case and sometimes in th

Re: [PATCH] Refactor handle_section_attribute to reduce nesting and distinguish error cases

2014-08-24 Thread Andrew Pinski
On Sun, Aug 24, 2014 at 1:42 PM, Josh Triplett wrote: > handle_section_attribute contains many levels of nested conditionals and > branching code flow paths, with the error cases sometimes in the else > case and sometimes in the if case. Simplify the code flow into a series > of potential failure

[PATCH] Refactor handle_section_attribute to reduce nesting and distinguish error cases

2014-08-24 Thread Josh Triplett
handle_section_attribute contains many levels of nested conditionals and branching code flow paths, with the error cases sometimes in the else case and sometimes in the if case. Simplify the code flow into a series of potential failure cases ending with the successful path, with no nesting and no

Re: [PATCH, DOC]: Fix for Options That Control Optimization section

2014-08-24 Thread Gerald Pfeifer
On Fri, 11 Jul 2014, Martin Liška wrote: 2014-07-11 Martin Liska * doc/invoke.texi: Added missing options to options that control optimization. Missing -foptimize-strlen option introduced. I applied the following patch on top of yours, adding @code in one, and @option in another

RE: [PATCH] PR fortran/62245 fix INT docs.

2014-08-24 Thread VandeVondele Joost
>> but helpful for gfortran users ? >The problem is that a similar sentence would then need to be >added to other intrinsic subroutines/functions where an actual >argument might produce a result that is out-of-range of the >representable entity. So, I could remove the sentence if there is conse

Re: [PATCH] PR fortran/62245 fix INT docs.

2014-08-24 Thread Steve Kargl
On Sun, Aug 24, 2014 at 03:58:42PM +, VandeVondele Joost wrote: > >> +of @var{A} and whose sign is the same as the sign of @var{A}. The result > >> +is undefined if it can not be represented as an @code{INTEGER} of the > >> +given @code{KIND}. > > > >The last sentence above is not needed. >

RE: [PATCH] PR fortran/62245 fix INT docs.

2014-08-24 Thread VandeVondele Joost
>> +of @var{A} and whose sign is the same as the sign of @var{A}. The result >> +is undefined if it can not be represented as an @code{INTEGER} of the >> +given @code{KIND}. > >The last sentence above is not needed. but helpful for gfortran users ? > There is a general > prohibition in 13.7.1 a

Re: [PATCH] PR fortran/62245 fix INT docs.

2014-08-24 Thread Steve Kargl
On Sun, Aug 24, 2014 at 12:47:18PM +, VandeVondele Joost wrote: > > One small ask: these lines are too long (already before your patch); > > can you please reformat those lines your patch touches to <80 columns? > > sure: > > +of @var{A} and whose sign is the same as the sign of @var{A}. Th

Re: [patch, fortran] use vec<> in frontend-passes.c

2014-08-24 Thread Thomas Koenig
Hi, Trevor Saunders wrote: >> -static gfc_expr ***expr_array; >> -static int expr_size, expr_count; >> +static vec expr_array = vec(); > > that's usually written as just static vec foo; vec doesn't actually > have ctors, and 0 initialization works fine. Fixed (also below). >> + // doloop_li

RE: [PATCH] PR fortran/62245 fix INT docs.

2014-08-24 Thread VandeVondele Joost
> One small ask: these lines are too long (already before your patch); > can you please reformat those lines your patch touches to <80 columns? sure: Index: intrinsic.texi === --- intrinsic.texi (revision 214408) +++ intrinsic.

Re: [PATCH] PR fortran/62245 fix INT docs.

2014-08-24 Thread Gerald Pfeifer
On Sun, 24 Aug 2014, VandeVondele Joost wrote: > A doc change to refine wording for result value of int, avoiding the > word range and using magnitude as does the standard. Mentions undefined > behavior. > > 2014-08-24 Joost VandeVondele > > PR fortran/62245 > * intrinsic.texi (

[PATCH] PR fortran/62245 fix INT docs.

2014-08-24 Thread VandeVondele Joost
A doc change to refine wording for result value of int, avoiding the word range and using magnitude as does the standard. Mentions undefined behavior. 2014-08-24 Joost VandeVondele PR fortran/62245 * intrinsic.texi (INT): clarify result and undefined behavior. Index: intrins

[SH][committed] Fix PR 61996

2014-08-24 Thread Oleg Endo
Hi, The attached patch fixes PR 61996 and does some minor documentation fixes. Committed to trunk (r214406), 4.9 (r214407) and 4.8 (r214408). Tested with 'make all', verifying that the new test case passes for m1* and m2* targets and with 'make info dvi pdf'. Cheers, Oleg gcc/ChangeLog:

Re: [wwwdocs] Buildstat update for 4.9

2014-08-24 Thread Tom G. Christensen
On 24/08/14 10:45, Gerald Pfeifer wrote: I am wondering, should we consider merging i686-pc-linux-gnu and i686-unknown-linux-gnu? It's practically the same, isn't it? I did consider it, but I don't think it's really a problem to have them both listed and since I don't really know what the diff

Re: [PATCH] make excessive template instantiation depth a fatal error

2014-08-24 Thread Manuel López-Ibáñez
PING: https://gcc.gnu.org/ml/gcc-patches/2014-08/msg01709.html On 18 August 2014 00:03, Manuel López-Ibáñez wrote: > Following a suggestion in PR16564, this patch makes excessive template > instantiation depth a fatal error. In fact, this allows simplifying > the code a lot, just by printing firs

Re: [wwwdocs] Buildstat update for 4.9

2014-08-24 Thread Gerald Pfeifer
On Sun, 17 Aug 2014, Tom G. Christensen wrote: > Latest results for 4.9.x > > -tgc > > Testresults for 4.9.1: > arm-unknown-linux-gnueabihf > hppa2.0w-hp-hpux11.11 > hppa64-hp-hpux11.11 > i386-pc-solaris2.9 > i686-pc-linux-gnu > powerpc-apple-darwin8.11.0 > sparc-sun-solaris2.9 >

Re: [patch, fortran] use vec<> in frontend-passes.c

2014-08-24 Thread Steve Kargl
On Sun, Aug 24, 2014 at 12:47:17AM +0200, Thomas Koenig wrote: > Hello world, > > the attached patch uses the vec<> templates instead of the > home-grown memory management. > > Did I get the style right? I was a bit confused because, with > the declarations used in the patch, using the vec_safe_

Re: [GSoC] Elimination of CLooG library installation dependency

2014-08-24 Thread Roman Gareev
Thank you for the report! > (1) FAIL: gcc.dg/graphite/vect-pr43423.c scan-tree-dump-times vect > "vectorized 2 loops" 1 > > before I saw > > [Book15] f90/bug% grep vectorized vect-pr43423.c.114t.vect > /opt/gcc/p_work/gcc/testsuite/gcc.dg/graphite/vect-pr43423.c:14:17: note: === > vect_mark_stmt