Re: RFA (libstdc++): PATCH to implement C++17 over-aligned new

FAIL: g++.dg/cpp1z/aligned-new1.C (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20160910/gcc/testsuite/g++.dg/cpp1z/aligned-new1.C:10:20: warning: requested alignment 64 is larger than 16 [-Wattributes] FAIL: g++.dg/cpp1z/aligned-new1.C execution test FAIL: g++.dg/cpp1z

Re: [PATCH] PR fortran/77507

FAIL: gfortran.dg/pr77507.f90 -O (test for excess errors) Excess errors: /opt/gcc/gcc-20160910/gcc/testsuite/gfortran.dg/pr77507.f90:3:6: Fatal Error: Can't open module file 'ieee_arithmetic.mod' for reading at (1): No such file or directory Andreas. -- Andreas Schwa

[C PATCH] Tidy build_unary_op

A minor cleanup. First, FLAG is a really bad name for a parameter, and second, using two vars for the same thing is redundant (we never modify any of them). Also some bad formatting. Bootstrap/regtest on x86_64-linux running. Ok for trunk? 2016-09-10 Marek Polacek * c-typeck.c (buil

[PING] Unreviewed C/C++ warning patches

Hi, the following C/C++ FE warning patches are still waiting for review: 20 Aug 2016: https://gcc.gnu.org/ml/gcc-patches/2016-08/msg01463.html [PATCH, C++] Warn on redefinition of builtin functions (PR c++/71973) 4 Sep 2016: https://gcc.gnu.org/ml/gcc-patches/2016-09/msg00155.html [PATCHv2] Ad

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

On Sat, Sep 10, 2016 at 05:07:51PM +0200, Jakub Jelinek wrote: > Hi! > > Just a minor nit: > > On Sat, Sep 10, 2016 at 04:58:43PM +0200, Marek Polacek wrote: > > Spurred by the recent findings, I > > decided to > > implement a warning that warns when a pointer

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

Hi! Just a minor nit: On Sat, Sep 10, 2016 at 04:58:43PM +0200, Marek Polacek wrote: > Spurred by the recent findings, I decided > to > implement a warning that warns when a pointer is compared with a zero > character > literal (constant), because this isn't l

[PATCH 1/1] [netbsd] when building shared, link against libc

--- gcc/config/netbsd.h | 2 ++ 1 file changed, 2 insertions(+) diff --git a/gcc/config/netbsd.h b/gcc/config/netbsd.h index f2d6cc6..65ce943 100644 --- a/gcc/config/netbsd.h +++ b/gcc/config/netbsd.h @@ -96,6 +96,7 @@ along with GCC; see the file COPYING3. If not see %{!pg:-lposix}}

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

Spurred by the recent findings, I decided to implement a warning that warns when a pointer is compared with a zero character literal (constant), because this isn't likely to be intended. So e.g. ptr == L'\0' is probably wrong and should've been written as

Re: [PATCH] PR fortran/77507

On Sat, Sep 10, 2016 at 09:03:16AM +0200, Andreas Schwab wrote: > FAIL: gfortran.dg/c_assoc_2.f03 -O (test for errors, line 23) > Sorry about that. Seems I forgot to check-in the updated testcases with r240050. Should be fixed now. -- Steve

Re: [PATCH] Fix -Wshadow warning in libiberty/cp-demangle.c

On 9/10/16, Ian Lance Taylor wrote: > On Fri, Sep 9, 2016 at 5:06 PM, Mark Wielaard wrote: >> valgrind contains a copy of the libiberty demangler which gets compiled >> with -Wshadow. That shows the following warning: >> >> cp-demangle.c: In function ‘d_substitution’: >> cp-demangle.c:3772:35: wa

libgo patch committed: update to Go 1.7.1 release

This patch updates libgo to the Go 1.7.1 release. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE === --- gcc/go/gofrontend/MERGE (revision 240070) +++ gcc/go/g

libgo patch committed: add runtime/internal/atomic package

This patch to libgo adds the runtime/internal/atomic package. This copies the API that is in the Go 1.7 library, but for gccgo the actual implementation is in C, written using the __atomic intrinsics. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index: gcc

Re: [PATCH] Delete GCJ

On Mon, Sep 5, 2016 at 11:13 AM, Andrew Haley wrote: > As discussed. I think I should ask a Global reviewer to approve this > one. For obvious reasons I haven't included the diffs to the deleted > gcc/java and libjava directories. The whole tree, post GCJ-deletion, > is at svn+ssh://gcc.gnu.org

Re: [DOC] Boolean -> boolean changes

On Tue, Sep 06, 2016 at 02:59:09PM +, Joseph Myers wrote: > On Tue, 6 Sep 2016, Marek Polacek wrote: > > > * doc/tm.texi: Likewise. > > You can't modify tm.texi directly; you have to modify whichever is > appropriate of target.def and tm.texi.in, then regenerate tm.texi. I see. So like

Re: [PATCH] Fix -Wshadow warning in libiberty/cp-demangle.c

On Sat, Sep 10, 2016 at 01:57:54AM -0700, Ian Lance Taylor wrote: > On Fri, Sep 9, 2016 at 5:06 PM, Mark Wielaard wrote: > > Fix that by renaming the struct demangle_component variable to dc > > and add -Wshadow to ac_libiberty_warn_cflags (the only warning is this one). > > > > libiberty/ChangeLo

Go patch committed: remove unused function

This patch to the GCC interface to the Go frontend removes the unused function go_trampoline_info. That function was replaced by libffi closures. Bootstrapped on x86_64-pc-linux-gnu. Committed to mainline. Ian 2016-09-10 Ian Lance Taylor * go-backend.c (go_trampoline_info): Remove. * go-c.

Go patch committed: remove useless #include

This patch from Than McIntosh removes a useless #include of "simple-object.h" from the Go frontend, and adjust an out of date comment. Bootstrapped and ran Go testsuite on x86_64-pc-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE ==

Re: RFA (libstdc++): PATCH to implement C++17 over-aligned new

On 10/09/16 11:25 +0200, Marc Glisse wrote: On Sat, 10 Sep 2016, Christophe Lyon wrote: On aarch64*-elf and arm-eabi (using newlib), I'm seeing: /gccsrc/libstdc++-v3/libsupc++/new_opa.cc:66: undefined reference to `aligned_alloc' https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;a=com

Re: [PATCH 9/9] cse.c selftests

On 09/10/16 01:11, Jeff Law wrote: >> First the good news. The unit test does actually work, but not only >> on aarch even without -mabi=ilp32, but also on i386. All that's >> missing is a check that we don't get a reg/f here: > Sweet. This starts to touch on some of those unanswered questions

Re: RFA (libstdc++): PATCH to implement C++17 over-aligned new

On 09/09/16 17:20 -0400, Jason Merrill wrote: OK, this is what I'm checking in; could someone test it on MinGW? MSDN suggests we need for _aligned_malloc, but I'm not sure. Last time I tried to build and test MinGW it made me cry, and I no longer have any access to Windows machines anyway. Bu

Re: RFA (libstdc++): PATCH to implement C++17 over-aligned new

On Sat, 10 Sep 2016, Christophe Lyon wrote: On aarch64*-elf and arm-eabi (using newlib), I'm seeing: /gccsrc/libstdc++-v3/libsupc++/new_opa.cc:66: undefined reference to `aligned_alloc' https://sourceware.org/git/gitweb.cgi?p=newlib-cygwin.git;a=commit;h=f86afe5a3ac62a92e821c764a011e1625abdd32

Re: [PATCH] Fix -Wshadow warning in libiberty/cp-demangle.c

On Fri, Sep 9, 2016 at 5:06 PM, Mark Wielaard wrote: > valgrind contains a copy of the libiberty demangler which gets compiled > with -Wshadow. That shows the following warning: > > cp-demangle.c: In function ‘d_substitution’: > cp-demangle.c:3772:35: warning: declaration of ‘c’ shadows a previous

Re: [PATCH, c++, PR77427 ] Set TYPE_STRUCTURAL_EQUALITY for sysv_abi va_list

On 07-09-16 11:22, Richard Biener wrote: On Mon, Sep 5, 2016 at 6:11 PM, Tom de Vries wrote: On 05/09/16 09:49, Richard Biener wrote: On Sun, Sep 4, 2016 at 11:30 PM, Tom de Vries wrote: On 04/09/16 16:08, Richard Biener wrote: On September 4, 2016 12:33:02 PM GMT+02:00, Tom de Vries

Re: [PATCH, LRA] Fix PR rtl-optimization 77289, LRA matching constraint problem

On Fri, Sep 9, 2016 at 6:37 PM, Peter Bergner wrote: > On 9/9/16 5:51 PM, Jeff Law wrote: >> >> On 08/30/2016 10:23 PM, Peter Bergner wrote: >>> >>> gcc/ >>> PR rtl-optimization/77289 >>> * lra-constraints.c (get_final_hard_regno): Add support for non hard >>> register numbers. Remove

Re: [PATCH] df: Keep return address register undefined until epilogue_completed

On Fri, Sep 09, 2016 at 04:40:12PM -0600, Jeff Law wrote: > Right now the dataflow is conservatively correct WRT the return register. And conservatively incorrect wrt all other callee-saved regs! > If we made the change you want to make than the dataflow becomes overly > optimistic about the ran

Re: [PATCH] PR fortran/77507

FAIL: gfortran.dg/c_assoc_2.f03 -O (test for errors, line 23) 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."

Re: RFA (libstdc++): PATCH to implement C++17 over-aligned new

On 9 September 2016 at 23:20, Jason Merrill wrote: > On Thu, Sep 8, 2016 at 7:06 AM, Jonathan Wakely wrote: >> On 08/09/16 09:10 +0200, Marc Glisse wrote: >>> >>> Do we want a generic fallback implementation (similar to >>> gcc/config/i386/gmm_malloc.h)? A windows version with _aligned_malloc / >