[PATCH, Fortran] Fix ICE due to comparison between UNION components

2016-10-02 Thread Fritz Reese
All, The attached fixes an[other] ICE in the comparison between UNIONs. This time the ICE is due to a BT_UNION component comparing itself to a BT_DERIVED component, thus considering their FL_STRUCT and FL_UNION typenodes to be equal. This is very similar to PR fortran/77782, except it is an error

[PATCH] add uClibc target hook (PR bootstrap/77819)

2016-10-02 Thread Martin Sebor
The attached patch adds a uclibc_printf_pointer_format target hook equivalent to the linux glibc_printf_pointer_format hook. I couldn't find a good uclibc-only file where to put the new definition of the hook so I conditionally added it to targethooks.c. I tested the new hook by bootstrapping GCC

Re: PATCH to fix bogus -Wimplicit-fallthrough warning (PR c++/77803)

2016-10-02 Thread Jakub Jelinek
On Sun, Oct 02, 2016 at 02:42:23PM -0400, Jason Merrill wrote: > On Sat, Oct 1, 2016 at 10:17 AM, Marek Polacek wrote: > > + && (last_eval == NULL > > + || !gimple_call_internal_p (last_eval, IFN_FALLTHROUGH)) > > Isn't this still assuming that non-null last_eval must be a

RE: Fix PR tree-optimization/77808, ICE in duplicate_ssa_name_ptr_info, at tree-ssanames.c:630 starting with r240439

2016-10-02 Thread Doug Gilmore
Hi Christophe, > From: Christophe Lyon [christophe.l...@linaro.org] > Sent: Saturday, October 01, 2016 7:57 AM > To: Doug Gilmore > Cc: gcc-patches@gcc.gnu.org > Subject: Re: Fix PR tree-optimization/77808, ICE in > duplicate_ssa_name_ptr_info, at tree-ssanames.c:630 starting with r240439 > > Hi

[PATCH] fix outstanding -Wformat-length failures (pr77735 et al.)

2016-10-02 Thread Martin Sebor
The attached patch fixes a number of outstanding test failures and ILP32-related bugs in the gimple-ssa-sprintf pattch pointed out in bug 77676 and 77735). The patch also fixes c_strlen to correctly handle wide strings (previously it accepted them but treated them as nul-terminated byte sequences

[C++ PATCH] Delete GCJ - C++ part

2016-10-02 Thread Jakub Jelinek
On Sun, Oct 02, 2016 at 03:27:09PM +0200, Andreas Schwab wrote: > Things we may want to remove: > > - references to java in contrib (download_ecj, gcc_update, > patch_tester.sh, update-copyright.py) > - GCJ, GCJ_FOR_BUILD, GCJ_FOR_TARGET in Makefiles.tpl and configure.ac > - LIBGCJ_SONAME in con

Re: [PATCH][RTL ifcvt] Transform (X == CST) ? -CST : Y into (X == CST) ? -X : Y when conditional negation is available

2016-10-02 Thread Andrew Pinski
On Sun, Oct 2, 2016 at 7:50 AM, Jeff Law wrote: > On 10/02/2016 04:48 AM, Andreas Schwab wrote: >> >> This miscompiles the stage2 ada compiler. > > No target identified. He reported it in a bug report, aarch64-linux-gnu. Thanks, Andrew > jeff

Re: [BUILDROBOT] dwarf2out_do_cfi_startproc(bool)’: may write a terminating nul past the end of the destination

2016-10-02 Thread Jason Merrill
OK. On Sat, Oct 1, 2016 at 4:30 PM, Jakub Jelinek wrote: > On Fri, Sep 30, 2016 at 08:59:52PM +0200, Jakub Jelinek wrote: >> Ok if it passes bootstrap/regtest? > > Passed bootstrap/regtest on x86_64-linux and i686-linux. >> >> 2016-09-30 Jakub Jelinek >> >> * dwarf2out.c (output_fde, out

Re: C/C++ PATCH to implement -Wpointer-compare warning (PR c++/64767)

2016-10-02 Thread Jason Merrill
OK, thanks. On Sat, Oct 1, 2016 at 10:16 AM, Marek Polacek wrote: > On Fri, Sep 30, 2016 at 05:48:03PM -0400, Jason Merrill wrote: >> On Fri, Sep 30, 2016 at 12:43 PM, Marek Polacek wrote: >> > On Fri, Sep 23, 2016 at 10:31:33AM -0400, Jason Merrill wrote: >> >> On Fri, Sep 23, 2016 at 9:15 AM,

Re: PATCH to fix bogus -Wimplicit-fallthrough warning (PR c++/77803)

2016-10-02 Thread Jason Merrill
On Sat, Oct 1, 2016 at 10:17 AM, Marek Polacek wrote: > + && (last_eval == NULL > + || !gimple_call_internal_p (last_eval, IFN_FALLTHROUGH)) Isn't this still assuming that non-null last_eval must be a gcall, so we'll get a checking ICE if it's something else? Jason

Re: Implement P0001R1 - C++17 removal of register storage class specifier

2016-10-02 Thread Jason Merrill
OK. On Fri, Sep 30, 2016 at 7:28 PM, Jakub Jelinek wrote: > On Fri, Sep 30, 2016 at 05:32:31PM -0400, Jason Merrill wrote: >> On Fri, Sep 30, 2016 at 4:10 AM, Jakub Jelinek wrote: >> > On Thu, Sep 29, 2016 at 10:57:07PM +, Joseph Myers wrote: >> >> This is missing documentation of the new -W

Re: [PATCH] Make -Wint-in-bool-context warn on suspicious shift ops

2016-10-02 Thread Jason Merrill
OK. On Fri, Sep 30, 2016 at 1:07 AM, Bernd Edlinger wrote: > On 09/29/16 22:38, Jason Merrill wrote: >> On Thu, Sep 29, 2016 at 3:58 PM, Bernd Edlinger >> wrote: >>> Unfortunately, without that exception there is a false positive: >>> >>> In file included from ../../gcc-trunk/gcc/ada/gcc-interfa

Re: [PATCH] Delete GCJ

2016-10-02 Thread Andrew Haley
On 02/10/16 14:27, Andreas Schwab wrote: > Things we may want to remove: > > - references to java in contrib (download_ecj, gcc_update, > patch_tester.sh, update-copyright.py) > - GCJ, GCJ_FOR_BUILD, GCJ_FOR_TARGET in Makefiles.tpl and configure.ac > - LIBGCJ_SONAME in config/i386/{cygwin.h,ming

Re: [PATCH][RTL ifcvt] Transform (X == CST) ? -CST : Y into (X == CST) ? -X : Y when conditional negation is available

2016-10-02 Thread Jeff Law
On 10/02/2016 04:48 AM, Andreas Schwab wrote: This miscompiles the stage2 ada compiler. No target identified. jeff

Re: [PATCH] Delete GCJ

2016-10-02 Thread Andreas Schwab
Things we may want to remove: - references to java in contrib (download_ecj, gcc_update, patch_tester.sh, update-copyright.py) - GCJ, GCJ_FOR_BUILD, GCJ_FOR_TARGET in Makefiles.tpl and configure.ac - LIBGCJ_SONAME in config/i386/{cygwin.h,mingw32.h} - references to java in install.texi Andreas.

[PATCHv2] Cleanup of input.c

2016-10-02 Thread Bernd Edlinger
Hi Dave, here is the new version of the input.c patch: I have updated the comments, and revised the test case as requested. I have additionally done a bootstrap with build config=bootstrap-asan. Bootstrap and reg-testing on x86_64-pc-linux-gnu. Is it OK for trunk? Thanks Bernd. 2016-09-26 Be

Re: [PATCH] Machine-readable RTL dumps: print_rtx_function

2016-10-02 Thread Segher Boessenkool
On Thu, Sep 29, 2016 at 11:36:29AM -0600, Jeff Law wrote: > On 09/29/2016 11:25 AM, Bernd Schmidt wrote: > >On 09/29/2016 07:47 PM, David Malcolm wrote: > >>This patch adds a new function, print_rtx_function, intended for use > >>for generating function dumps suitable for parsing by the RTL fronten

Re: [PATCH][RTL ifcvt] Transform (X == CST) ? -CST : Y into (X == CST) ? -X : Y when conditional negation is available

2016-10-02 Thread Andreas Schwab
This miscompiles the stage2 ada compiler. Andreas. -- Andreas Schwab, sch...@linux-m68k.org GPG Key fingerprint = 58CA 54C7 6D53 942B 1756 01D3 44D5 214B 8276 4ED5 "And now for something completely different."

[PATCH, OpenACC, Fortran] Fix PR77371, ICE on allocatable

2016-10-02 Thread Chung-Lin Tang
Hi Jakub, This patch fixes the two ICEs listed on PR77371. One is due to the Fortran omp_privatize_by_reference hook returning true for types like 'character(kind=1)[1:XX] *', causing them to be processed by the path intended for C++ reference types. The other one is simply not setting 'remove = t