Re: [patch, fortran] [5/6 Regression] Line continuation followed by comment character in string fails to compile

2015-05-16 Thread Jerry DeLisle
* ping * On 05/13/2015 06:58 PM, Jerry DeLisle wrote: The attached patch reverts a change I made for pr65456 which caused this regression and adds a check for quotes farther down in the function. This avoids treating a '!' in a string as a comment and wiping the rest of the line. I found

Re: [patch] libstdc++/66055 add missing constructors to unordered containers

2015-05-16 Thread Jonathan Wakely
On 16/05/15 11:39 +0200, François Dumont wrote: On 14/05/2015 15:47, Jonathan Wakely wrote: Reported by Nathan and fixed by his patch. I added the tests. Tested powerpc64le-linux, committed to trunk. This should be backported too. While backporting to debug and profile mode I noticed that

Re: [patch, fortran] [5/6 Regression] Line continuation followed by comment character in string fails to compile

2015-05-16 Thread Steve Kargl
On Sat, May 16, 2015 at 07:52:38AM -0700, Jerry DeLisle wrote: * ping * 2015-05-14 Jerry DeLisle jvdeli...@gcc.gnu.org PR fortran/65903 * io.c (format_lex): Change to NONSTRING when checking for possible doubled quote. * scanner.c (gfc_next_char_literal): Revert

Re: [patch, fortran] [5/6 Regression] Line continuation followed by comment character in string fails to compile

2015-05-16 Thread Jerry DeLisle
On 05/16/2015 08:11 AM, Steve Kargl wrote: On Sat, May 16, 2015 at 07:52:38AM -0700, Jerry DeLisle wrote: * ping * 2015-05-14 Jerry DeLisle jvdeli...@gcc.gnu.org PR fortran/65903 * io.c (format_lex): Change to NONSTRING when checking for possible doubled quote. *

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-16 Thread Iain Buclaw
On 14 May 2015 at 19:47, Joseph Myers jos...@codesourcery.com wrote: On Thu, 14 May 2015, Iain Buclaw wrote: On another note, I've found out why the remaining 20 symbols in my 75k sample failed. They don't fail at all! It's just that they were all greater than 33,000 characters in length,

[Patch, fortran] PR61831 side-effect deallocation of variable components

2015-05-16 Thread Mikael Morin
Hello, this is about PR61831 where in code like: type :: string_t character(LEN=1), dimension(:), allocatable :: chars end type string_t type(string_t) :: prt_in (...) tmp = new_prt_spec ([prt_in]) the deallocation of the

Re: [RFC]: Remove Mem/address type assumption in combiner

2015-05-16 Thread Segher Boessenkool
On Sat, May 16, 2015 at 12:36:38PM -0400, Hans-Peter Nilsson wrote: On Sat, 16 May 2015, Segher Boessenkool wrote: On Fri, May 15, 2015 at 10:40:48PM -0400, Hans-Peter Nilsson wrote: I confess the test-case-guarded addi pattern should have been expressed with a shift in addition to the

Re: Refactor gimple_expr_type

2015-05-16 Thread Trevor Saunders
On Fri, May 15, 2015 at 07:13:35AM +, Aditya K wrote: Hi, I have tried to refactor gimple_expr_type to make it more readable. Removed the switch block and redundant if. Please review this patch. for some reason your mail client seems to be inserting non breaking spaces all over the

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-16 Thread H.J. Lu
On Sat, May 16, 2015 at 12:03 PM, Rich Felker dal...@libc.org wrote: On Sat, May 16, 2015 at 11:59:56AM -0700, H.J. Lu wrote: On Sat, May 16, 2015 at 7:19 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, May 15, 2015 at 4:49 PM, Rich Felker dal...@libc.org wrote: On Fri, May 15, 2015 at

MAINTAINERS (Write After Approval): Add myself.

2015-05-16 Thread Iain Buclaw
Adding myself as Write After Approval. Regards Iain --- ChangeLog | 4 MAINTAINERS | 1 + 2 files changed, 5 insertions(+) --- ChangeLog +++ ChangeLog @@ -1,3 +1,7 @@ +2015-05-16 Iain Buclaw ibuc...@gdcproject.org + + * MAINTAINERS (Write After Approval): Add myself. + 2015-05-11

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-16 Thread H.J. Lu
On Fri, May 15, 2015 at 4:49 PM, Rich Felker dal...@libc.org wrote: On Fri, May 15, 2015 at 04:34:57PM -0700, H.J. Lu wrote: On Fri, May 15, 2015 at 4:30 PM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, May 15, 2015 at 4:14 PM, H.J. Lu hjl.to...@gmail.com wrote: My relax branch proposal works

Re: [patch, fortran] [5/6 Regression] Line continuation followed by comment character in string fails to compile

2015-05-16 Thread Mikael Morin
Le 14/05/2015 03:58, Jerry DeLisle a écrit : The attached patch reverts a change I made for pr65456 which caused this regression and adds a check for quotes farther down in the function. This avoids treating a '!' in a string as a comment and wiping the rest of the line. I found the added

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-16 Thread Rich Felker
On Sat, May 16, 2015 at 11:59:56AM -0700, H.J. Lu wrote: On Sat, May 16, 2015 at 7:19 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, May 15, 2015 at 4:49 PM, Rich Felker dal...@libc.org wrote: On Fri, May 15, 2015 at 04:34:57PM -0700, H.J. Lu wrote: On Fri, May 15, 2015 at 4:30 PM, H.J. Lu

Re: [PATCH] fortran/66045 -- NULL() on RHS of assignment

2015-05-16 Thread FX
PR fortran/66045 * expr.c (gfc_check_assign): Check for assignment of NULL() instead of the (intended) pointer assignment. OK

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-16 Thread H.J. Lu
On Sat, May 16, 2015 at 7:19 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, May 15, 2015 at 4:49 PM, Rich Felker dal...@libc.org wrote: On Fri, May 15, 2015 at 04:34:57PM -0700, H.J. Lu wrote: On Fri, May 15, 2015 at 4:30 PM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, May 15, 2015 at 4:14 PM,

Re: [PATCH 6/7] [D] libiberty: Improve support for demangling D2 templates

2015-05-16 Thread Iain Buclaw
On 14 May 2015 at 17:30, Iain Buclaw ibuc...@gdcproject.org wrote: On 14 May 2015 at 15:24, Jeff Law l...@redhat.com wrote: On 05/13/2015 02:51 AM, Iain Buclaw wrote: In my tests, this gives the demangler near-complete support. Of a sample of about 75k symbols pulled from the standard

Re: [patch, fortran] [5/6 Regression] Line continuation followed by comment character in string fails to compile

2015-05-16 Thread Jerry DeLisle
On 05/16/2015 10:45 AM, Jerry DeLisle wrote: --- snip --- Thanks Steve, Committed revision 223248. I had some time to play with this a little more this afternoon. I am going to commit the following little patchlet that gives us the nice warning we should have. (After full regression

Re: [RFC]: Remove Mem/address type assumption in combiner

2015-05-16 Thread Segher Boessenkool
On Fri, May 15, 2015 at 10:40:48PM -0400, Hans-Peter Nilsson wrote: I confess the test-case-guarded addi pattern should have been expressed with a shift in addition to the multiplication. But they wouldn't ever match so they might very well have bitrotted by now :-( (In addition to as the

Re: [RFC]: Remove Mem/address type assumption in combiner

2015-05-16 Thread Hans-Peter Nilsson
On Sat, 16 May 2015, Segher Boessenkool wrote: On Fri, May 15, 2015 at 10:40:48PM -0400, Hans-Peter Nilsson wrote: I confess the test-case-guarded addi pattern should have been expressed with a shift in addition to the multiplication. But they wouldn't ever match so they might very well

Re: [patch, fortran] [5/6 Regression] Line continuation followed by comment character in string fails to compile

2015-05-16 Thread Jerry DeLisle
On 05/16/2015 08:17 AM, Mikael Morin wrote: snip Index: gcc/testsuite/gfortran.dg/continuation_13.f90 === --- gcc/testsuite/gfortran.dg/continuation_13.f90(revision 223105) +++

Re: [PATCH] fortran/66052 -- Prevent dereference of NULL pointer

2015-05-16 Thread Mikael Morin
Le 15/05/2015 17:19, Steve Kargl a écrit : Regression tested on trunk. OK to commit? Hello, Index: gcc/fortran/decl.c === --- gcc/fortran/decl.c(revision 223094) +++ gcc/fortran/decl.c(working copy) @@

Re: Mostly rewrite genrecog

2015-05-16 Thread Andreas Krebbel
Hi Richard, I see regressions with the current IBM z13 vector patchset which appear to be related to the new genrecog. The following two insn definitions only differ in the mode and predicate of the shift count operand. (define_insn lshrmode3 [(set (match_operand:VI 0

Re: *ping* Re: [Patch, fortran] PR65792 - allocation of scalar elemental function with structure constructor fails

2015-05-16 Thread Paul Richard Thomas
Dear Mikael, With your tweak it bootstrapped and regtested OK. Also changed the comment after Steve's pointing out that the last sentence was incomprehensible:-) Committed to trunk as revision 223234. Cheers Paul 2015-05-16 Mikael Morin mik...@gcc.gnu.org Paul Thomas

Re: [patch] libstdc++/66055 add missing constructors to unordered containers

2015-05-16 Thread François Dumont
On 14/05/2015 15:47, Jonathan Wakely wrote: Reported by Nathan and fixed by his patch. I added the tests. Tested powerpc64le-linux, committed to trunk. This should be backported too. While backporting to debug and profile mode I noticed that those constructors were not the only missing ones.

[patch, fortran] Fix PR 66113 error with deeply nested blocks

2015-05-16 Thread Thomas Koenig
Hello world, this (rather obvious) patch fixes array declarations in deeply nested BLOCKs. Regression-tested. OK for trunk? Thomas 2015-05-16 Thomas Koenig tkoe...@gcc.gnu.org PR fortran/66113 * expr.c (is_parent_of_current_ns): New function.

Re: [patch, fortran] Fix PR 66113 error with deeply nested blocks

2015-05-16 Thread Mikael Morin
Le 16/05/2015 12:35, Thomas Koenig a écrit : Hello world, this (rather obvious) patch fixes array declarations in deeply nested BLOCKs. Regression-tested. OK for trunk? OK, thanks. Mikael

Re: [RFC]: Remove Mem/address type assumption in combiner

2015-05-16 Thread Hans-Peter Nilsson
On Sat, 16 May 2015, Segher Boessenkool wrote: On Sat, May 16, 2015 at 12:36:38PM -0400, Hans-Peter Nilsson wrote: On Sat, 16 May 2015, Segher Boessenkool wrote: On Fri, May 15, 2015 at 10:40:48PM -0400, Hans-Peter Nilsson wrote: I confess the test-case-guarded addi pattern should have

Do not output debug info into slim LTO files

2015-05-16 Thread Jan Hubicka
Hi, while debugging verify_type ICE I noticed that we output debug info to slim-lto files. This debug info is never used for anything and should be omitted. I wonder if this can go also to rleease branches since it will likely make the slim files smaller. Of course things will change with