Re: [PATCH v3] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-12-03 Thread Segher Boessenkool
[ I did not see this patch before, sorry. ] This causes the second half of PR78638. On Thu, Dec 01, 2016 at 04:30:08PM +0100, Dominik Vogt wrote: > --- a/gcc/combine.c > +++ b/gcc/combine.c > @@ -5600,6 +5600,18 @@ combine_simplify_rtx (rtx x, machine_mode op0_mode, > int in_dest, >

Re: [PATCH, Fortran, v2] Fix deallocation of nested derived typed components

2016-12-03 Thread Dominique d'Humières
Hi Andre, I fear the patch is causing another set of failures with -fopenmp: FAIL: libgomp.fortran/allocatable11.f90 -O0 (internal compiler error) … FAIL: libgomp.fortran/allocatable8.f90 -g -flto (test for excess errors) of the kind collect2: error: ld returned 1 exit status [Book15]

[PATCH] libiberty: Don't update and remove did_subs field from demangler structs.

2016-12-03 Thread Mark Wielaard
The d_info field did_subs was used for estimating the string output size. It was no longer used when the malloc-less callback interface was introduced in 2007 (svn r121305). But the field was still updated. When backtracking was introduced in 2013 (svn r205292) did_subs was also added to the

Re: [PATCH, Fortran, v2] Fix deallocation of nested derived typed components

2016-12-03 Thread Andre Vehreschild
Hi all, @Dominique: Thanks for checking. And also for pointing out that the initial version of the patch ICEd on some already closed PRs. The objective of those PRs does not seem to be covered by the current testsuite. I therefore additionally propose to add attached testcase. Ok for trunk? Of

Re: [Patch, Fortran, OOP] PR 43207: [OOP] invalid (pointer) assignment to and from abstract non-polymorphic expressions

2016-12-03 Thread Janus Weil
2016-12-03 18:03 GMT+01:00 Steve Kargl : > On Fri, Dec 02, 2016 at 02:37:24PM +0100, Janus Weil wrote: >> >> the attached patch fixes the PR in the subject line by introducing a >> new check to reject invalid code. It's a slight update of an old patch >> that I

Re: [Patch, Fortran, OOP] PR 42188: F03:C612. The leftmost part-name shall be the name of a data object

2016-12-03 Thread Janus Weil
2016-12-03 18:01 GMT+01:00 Steve Kargl : > On Fri, Dec 02, 2016 at 04:24:20PM +0100, Janus Weil wrote: >> >> another simple fix for a rather old PR. This one adds a new check, in >> order to provide better error messages than just "Unclassifiable >> statement".

Re: [PATCH, Fortran, cosmetics] Use convenience functions and constants

2016-12-03 Thread Steve Kargl
On Sat, Dec 03, 2016 at 06:25:19PM +0100, Andre Vehreschild wrote: > > Er, thanks for the review and sorry, but I fear I already committed > the changes as part of r243021. These improvables occurred to me > while hacking at: > > [PATCH, Fortran, accaf, v1] Add caf-API-calls to asynchronously

Use gen_lowpart_SUBREG consistently in emit_spill_move

2016-12-03 Thread Eric Botcazou
The second branch of the alternative in emit_spill_move correctly uses gen_lowpart_SUBREG to build a lowpart SUBREG, whereas the first branch uses gen_rtx_SUBREG (..., ..., 0), which of course badly breaks on big-endian. Tested on x86_64/Linux and SPARC/Solaris, applied on the mainline as

Re: [PATCH, Fortran, v2] Fix deallocation of nested derived typed components

2016-12-03 Thread Dominique d'Humières
> I also have addressed the ICEs with proc_pointer components. Confirmed, the patch LGTM. Thanks, Dominique

[SPARC] Disable U constraint with LRA

2016-12-03 Thread Eric Botcazou
As diagnosed by DaveM, there is a bad interaction between the U constraint used by the SPARC port to represent "aligned" double-word registers in 32-bit mode and LRA; more precisely, LRA doesn't play as nice as reload with it because it's not declared with define_register_constraint. Fixed by

Re: [PATCH, Fortran, cosmetics] Use convenience functions and constants

2016-12-03 Thread Andre Vehreschild
Hi Steve, Er, thanks for the review and sorry, but I fear I already committed the changes as part of r243021. These improvables occurred to me while hacking at: [PATCH, Fortran, accaf, v1] Add caf-API-calls to asynchronously handle allocatable components in derived type coarrays.

Re: [PATCH] PR fortran/78618 -- RANK() should not ICE

2016-12-03 Thread Steve Kargl
On Sat, Dec 03, 2016 at 05:58:02PM +0100, Andreas Schwab wrote: > On Dez 01 2016, Steve Kargl wrote: > > > PR fortran/78618 > > * gfortran.dg/char_conversion.f90: New test. > > That test still crashes the compiler on m68k: > > 0xb40a7f crash_signal >

Re: [PATCH, Fortran, cosmetics] Use convenience functions and constants

2016-12-03 Thread Steve Kargl
On Tue, Nov 22, 2016 at 06:56:16PM +0100, Andre Vehreschild wrote: > > during more hacking on the allocatable components in derived type coarrays, I > encountered the improvable code fragments in the patch attached. > > Bootstraps and regtests ok on x86_64-linux/F23. Ok for trunk? > OK with a

Re: [Patch, Fortran, OOP] PR 43207: [OOP] invalid (pointer) assignment to and from abstract non-polymorphic expressions

2016-12-03 Thread Steve Kargl
On Fri, Dec 02, 2016 at 02:37:24PM +0100, Janus Weil wrote: > > the attached patch fixes the PR in the subject line by introducing a > new check to reject invalid code. It's a slight update of an old patch > that I posted in the PR quite some time ago, using somewhat tighter > checking to avoid

Re: [Patch, Fortran, OOP] PR 42188: F03:C612. The leftmost part-name shall be the name of a data object

2016-12-03 Thread Steve Kargl
On Fri, Dec 02, 2016 at 04:24:20PM +0100, Janus Weil wrote: > > another simple fix for a rather old PR. This one adds a new check, in > order to provide better error messages than just "Unclassifiable > statement". > > Regtests cleanly on x86_64-linux-gnu. Ok for trunk? > OK. -- Steve

Re: [PATCH] PR fortran/78618 -- RANK() should not ICE

2016-12-03 Thread Andreas Schwab
On Dez 01 2016, Steve Kargl <s...@troutmask.apl.washington.edu> wrote: > PR fortran/78618 > * gfortran.dg/char_conversion.f90: New test. That test still crashes the compiler on m68k: /daten/aranym/gcc/gcc-20161203/gcc/testsuite/gfortran.dg/char_conversion.f90:8:30: Erro

Re: [PATCH v3] Do not simplify "(and (reg) (const bit))" to if_then_else.

2016-12-03 Thread Andreas Schwab
On Dez 01 2016, Dominik Vogt wrote: > gcc/ChangeLog > > * combine.c (combine_simplify_rtx): Suppress replacement of > "(and (reg) (const_int bit))" with "if_then_else". That causes a regression on ia64. FAIL: gcc.target/ia64/builtin-popcount-2.c

[committed] Update baseline symbols on hppa-linux

2016-12-03 Thread John David Anglin
The attached change update the baseline symbols for libstdc++ on hppa-linux-gnu. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2016-12-03 John David Anglin * config/abi/post/hppa-linux-gnu/baseline_symbols.txt: Regenerate. Index:

New French PO file for 'gcc' (version 6.2.0)

2016-12-03 Thread Translation Project Robot
Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the French team of translators. The file is available at: http://translationproject.org/latest/gcc/fr.po (This file, 'gcc-6.2.0.fr.po', has just

Re: [PATCH] Added noexcept on constructors

2016-12-03 Thread Ville Voutilainen
On 3 December 2016 at 14:20, Ville Voutilainen wrote: > On 3 December 2016 at 13:31, Aditya K wrote: >> --- >> libstdc++-v3/ChangeLog | 3 +++ >> libstdc++-v3/src/c++11/shared_ptr.cc | 4 ++-- >> 2 files changed, 5 insertions(+), 2

Re: [PATCH] Added noexcept on constructors

2016-12-03 Thread Ville Voutilainen
On 3 December 2016 at 13:31, Aditya K wrote: > --- > libstdc++-v3/ChangeLog | 3 +++ > libstdc++-v3/src/c++11/shared_ptr.cc | 4 ++-- > 2 files changed, 5 insertions(+), 2 deletions(-) > > diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog > index

Re: [PATCH, Fortran, v2] Fix deallocation of nested derived typed components

2016-12-03 Thread Andre Vehreschild
Hi Dominique, thanks for immediately finding the issues. Please find an updated and extended patch. This time trunk w/o this patch ICEs on compiling the _4-testcase and accesses freed memory for the _3-testcase. Furthermore are now components of derived-typed coarrays deregistered correctly

[PATCH] Added noexcept on constructors

2016-12-03 Thread Aditya K
--- libstdc++-v3/ChangeLog | 3 +++ libstdc++-v3/src/c++11/shared_ptr.cc | 4 ++-- 2 files changed, 5 insertions(+), 2 deletions(-) diff --git a/libstdc++-v3/ChangeLog b/libstdc++-v3/ChangeLog index 08d9229..18924c4 100644 --- a/libstdc++-v3/ChangeLog +++ b/libstdc++-v3/ChangeLog

Re: PR78634: ifcvt/i386 cost updates

2016-12-03 Thread Uros Bizjak
On Fri, Dec 2, 2016 at 8:58 PM, James Greenhalgh wrote: > On Fri, Dec 02, 2016 at 05:00:05PM +0100, Bernd Schmidt wrote: >> With the i386 backend no longer double-counting the cost of a SET, >> the default implementation default_max_noce_ifcvt_seq_cost now >> provides

Re: [Patch, Fortran, OOP] PR 58175: Incorrect warning message on scalar finalizer

2016-12-03 Thread Janus Weil
I have just committed as obvious a minor addition here: https://gcc.gnu.org/viewcvs?rev=243218=gcc=rev It deals with an extended test case that was only reported after the initial fix. Cheers, Janus 2016-11-29 15:16 GMT+01:00 Janus Weil : > Committed as r242960. > > > >