Re: C++ PATCH for __has_cpp_attribute and fallthrough

2016-09-26 Thread Jason Merrill
OK. On Mon, Sep 26, 2016 at 8:23 AM, Marek Polacek wrote: > This patch updates __has_cpp_attribute for fallthrough, which is now > supported. > > The system.h hunk should aid people building trunk GCC with older GCC 7 not > supporting this attribute yet. > >

Re: [PATCH v2] [RTEMS] Always use atomic builtins for libstdc++

2016-09-26 Thread Bernhard Reutner-Fischer
On 26 September 2016 10:27:13 CEST, Andreas Schwab wrote: >On Sep 22 2016, Bernhard Reutner-Fischer wrote: > >> On 22 September 2016 11:11:42 CEST, Andreas Schwab >wrote: >>>On Sep 22 2016, Sebastian Huber

Re: [PATCH v2] [RTEMS] Always use atomic builtins for libstdc++

2016-09-26 Thread Andreas Schwab
On Sep 26 2016, Bernhard Reutner-Fischer wrote: > On 26 September 2016 10:27:13 CEST, Andreas Schwab wrote: >>On Sep 22 2016, Bernhard Reutner-Fischer wrote: >> >>> On 22 September 2016 11:11:42 CEST, Andreas Schwab

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Thomas Preudhomme
On 26/09/16 11:53, Richard Biener wrote: On Mon, 26 Sep 2016, Thomas Preudhomme wrote: On 26/09/16 10:15, Richard Biener wrote: On Thu, 22 Sep 2016, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for ARMv8-M[1] to GCC. This specific patch moves memory

Re: [PATCH 6/17][ARM] Add data processing intrinsics for float16_t.

2016-09-26 Thread Christophe Lyon
On 26 September 2016 at 15:19, Christophe Lyon wrote: > On 26 September 2016 at 15:03, Ramana Radhakrishnan > wrote: >> On Mon, Sep 26, 2016 at 1:48 PM, Christophe Lyon >> wrote: >>> On 26 September 2016 at

[Committed] PR fortran/77420

2016-09-26 Thread Steve Kargl
This is a follow-up commit to really fix PR fortran/77420. The code has been in my tree for 2 weeks and has passed numerous regression tests. 2016-09-26 Steven G. Kargl PR fortran/77420 * trans-common.c: Handle array elements in equivalence when the

[PATCH RESEND 2/2] Extend -falign-FOO=N to N[,M[,N2[,M2]]]

2016-09-26 Thread Denys Vlasenko
falign-functions=N is too simplistic. Ingo Molnar ran some tests and it seems that on latest x86 CPUs, 64-byte alignment of functions runs fastest (he tried many other possibilites): this way, after a call CPU can fetch a lot of insns in the first cacheline fill. However, developers are less

[PATCH RESEND 1/2] Temporary remove "at least 8 byte alignment" code from x86

2016-09-26 Thread Denys Vlasenko
This change drops forced alignment to 8 if requested alignment is higher than 8: before the patch, -falign-functions=9 was generating .p2align 4,,8 .p2align 3 which means: "align to 16 if the skip is 8 bytes or less; else align to 8". After this change, ".p2align 3" is not

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Thomas Preudhomme
On 26/09/16 18:22, Thomas Preudhomme wrote: I committed as is because emit-rtl.h uses enum memmodel so all files that includes it must be updated. This gets worse because tm.h uses emit-rtl.h so all target needs to be updated as well. The diff was becoming big so I decided to keep it separate.

Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer

2016-09-26 Thread Richard Biener
On September 26, 2016 5:46:28 PM GMT+02:00, Bernd Edlinger wrote: >Hi, > >>@@ -2310,7 +2313,8 @@ create_intersect_range_checks_index >(loop_vec_info loop_vinfo, tree *cond_expr, >> gcc_assert (TREE_CODE (DR_STEP (dr_a.dr)) == INTEGER_CST); >> >> bool neg_step =

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Richard Biener
On September 26, 2016 7:39:13 PM GMT+02:00, Thomas Preudhomme wrote: >On 26/09/16 18:22, Thomas Preudhomme wrote: >> >> I committed as is because emit-rtl.h uses enum memmodel so all files >that >> includes it must be updated. This gets worse because tm.h uses

Re: [PATCH] Fix PR tree-optimization/77550

2016-09-26 Thread Christophe Lyon
On 22 September 2016 at 22:30, Bernd Edlinger wrote: > On 09/22/16 21:30, Christophe Lyon wrote: >> On 21 September 2016 at 22:20, Bernd Edlinger >> wrote: >>> On 09/21/16 21:57, Christophe Lyon wrote: Hi, The new testcase

Re: [PATCH 5/5] rs6000: Separate shrink-wrapping

2016-09-26 Thread David Edelsohn
On Fri, Sep 23, 2016 at 4:21 AM, Segher Boessenkool wrote: > This implements the hooks for separate shrink-wrapping for rs6000. > It handles GPRs and LR. The GPRs get a component number corresponding > to their register number; LR gets component number 0. > > >

Re: [PATCH, Fortran] Extension: COTAN and degree-valued trig intrinsics with -fdec-math

2016-09-26 Thread Toon Moene
On 09/26/2016 04:01 PM, Fritz Reese wrote: All, Attached is a patch extending the GNU Fortran front-end to support some additional math intrinsics, enabled with a new compile flag -fdec-math. The flag adds the COTAN intrinsic (cotangent), as well as degree versions of all trigonometric

Re: [patch, avr, pr71676 and pr71678] Issues with casesi expand

2016-09-26 Thread Georg-Johann Lay
On 26.09.2016 15:19, Pitchumani Sivanupandi wrote: Attached patch for PR71676 and PR71678. PR71676 is for AVR target that generates wrong code when switch case index is more than 16 bits. Switch case index of larger than SImode are checked for out of range before 'casesi' expand. RTL expand of

Re: [PR77719] Fix ICE in pp_string, at pretty-print.c:955

2016-09-26 Thread Jeff Law
On 09/25/2016 08:39 PM, kugan wrote: On 25/09/16 04:50, kugan wrote: Hi, In make_new_ssa_for_def (tree-reassoc.c) we should use gimple_get_lhs to get lhs instead of gimple_assign_lhs as stmt can be builtins too. Attached patch fixes this. Testcase from PR (attached) seems to fail. I dont

Re: [PATCH 3/5] regrename: Don't rename restores

2016-09-26 Thread Jeff Law
On 09/23/2016 02:21 AM, Segher Boessenkool wrote: A restore is supposed to restore some certain register. Restoring it into some other register will not work. Don't. 2016-09-23 Segher Boessenkool * regrename.c (build_def_use): Invalidate chains that

Re: [Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-09-26 Thread Bernd Schmidt
On 09/16/2016 09:02 PM, Senthil Kumar Selvaraj wrote: Does this make sense? I ran a reg test for the avr target with a slightly older version of this patch, it did not show any regressions. If this is the right fix, I'll make sure to run reg tests on x86_64 after backporting to a gcc

Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer

2016-09-26 Thread Bernd Edlinger
Hi, >@@ -2310,7 +2313,8 @@ create_intersect_range_checks_index (loop_vec_info >loop_vinfo, tree *cond_expr, > gcc_assert (TREE_CODE (DR_STEP (dr_a.dr)) == INTEGER_CST); > > bool neg_step = tree_int_cst_compare (DR_STEP (dr_a.dr), size_zero_node) < > 0; >- unsigned HOST_WIDE_INT abs_step =

Re: [PATCH, RFC] gcov: dump in a static dtor instead of in an atexit handler

2016-09-26 Thread Martin Liška
On 08/12/2016 04:08 PM, Martin Liška wrote: > On 08/10/2016 02:53 PM, Nathan Sidwell wrote: >> On 08/10/16 06:43, Martin Liška wrote: >>> Hello. >>> >>> There are multiple PRs (mentioned in ChangeLog) which suffer from missing >>> capability of gcov >>> to save counters for functions with

[wwwdocs] Add some missing C++17 features

2016-09-26 Thread Jakub Jelinek
Hi! The following patch adds some missing C++1z features to the table (wording from https://isocpp.org/std/standing-documents/sd-6-sg10-feature-test-recommendations#recs.cpp17 except for P0386R2 which is too new for that (Jon, can you look up the feature test macro if any from newer SD-6 draft?).

Re: parallel/algo.h cleanup

2016-09-26 Thread Jonathan Wakely
On 25/09/16 17:23 +0200, François Dumont wrote: Hi Trivial patch to cleanup parallel/algo.h by generalizing usage of std::__iterator_category. * include/parallel/algo.h: Generalize usage of std::__iterator_category. Adjust whitespaces. N.B. for a patch of this size it would have

Re: Split c-common.c?

2016-09-26 Thread Jeff Law
On 09/26/2016 07:38 AM, Marek Polacek wrote: On Mon, Sep 26, 2016 at 03:26:28PM +0200, Jakub Jelinek wrote: On Mon, Sep 26, 2016 at 03:22:08PM +0200, Marek Polacek wrote: Before I spend time on this, I wanted to check if you consider this a good idea. Since c-common.c has grown a lot and is

Re: [PATCH] Disable sim build for ARC in top-level configure.ac

2016-09-26 Thread Jeff Law
On 09/22/2016 06:12 AM, Anton Kolesov wrote: Hi, This disables "sim" build for ARC, otherwise it is required to pass --disable-sim to top-level configure. Anton 0001-Disable-sim-builds-for-ARC-in-top-level-configure.ac.patch From 1fb11bf060ef0bccb0dd949fbf7ea63c763f008d Mon Sep 17 00:00:00

Re: [PATCH 1/5] separate shrink-wrap: New command-line flag, status flag, hooks, and doc

2016-09-26 Thread Jeff Law
On 09/23/2016 02:21 AM, Segher Boessenkool wrote: This patch adds a new command-line flag "-fshrink-wrap-separate", a status flag "shrink_wrapped_separate", hooks for abstracting the target components, and documentation for all those. 2016-09-23 Segher Boessenkool

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-26 Thread Arnaud Charlet
> this patch caused quite some breakage: Ada, Solaris/x86 and SPARC don't > bootstrap any longer. > > The following patch allows i386-pc-solaris2.12 and > sparc-sun-solaris2.12 > bootstraps continue. > > Strangely, I needed to use gcc_fallthrough () in i386.c; a mere /* FALLTHRU > */ > had no

Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer

2016-09-26 Thread Robin Dapp
> And also please mention https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77724 > and perhaps > add its testcase, too. > Patch according to Richard's suggestion, included test case. The test case works unpatched on x86-64 but fails on s390. Is gcc.dg/vect the proper place for it? (I didn't manage

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-26 Thread Jeff Law
On 09/26/2016 03:55 AM, Thomas Preudhomme wrote: How about this reworked patch? Best regards, Thomas On 23/09/16 17:17, Martin Sebor wrote: On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas

Re: [PATCH 5/5] rs6000: Separate shrink-wrapping

2016-09-26 Thread Jeff Law
On 09/23/2016 02:21 AM, Segher Boessenkool wrote: This implements the hooks for separate shrink-wrapping for rs6000. It handles GPRs and LR. The GPRs get a component number corresponding to their register number; LR gets component number 0. 2016-09-23 Segher Boessenkool

Re: PATCH to use the new gimple_call_internal_p overload

2016-09-26 Thread Jeff Law
On 09/26/2016 07:53 AM, Marek Polacek wrote: In my recent -Wimplicit-fallthrough patch I introduced a new overload gimple_call_internal_p to check whether a stmt is a specific internal function. So use this predicate in the rest of the codebase, too. Bootstrapped/regtested on x86_64-linux, ok

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-26 Thread Jason Merrill
It seems unfortunate that the warning doesn't accept /* ... fall through ... */ as a fallthrough comment. Jason

Re: [PATCH, Fortran] Extension: COTAN and degree-valued trig intrinsics with -fdec-math

2016-09-26 Thread Tobias Burnus
Fritz Reese wrote: > Attached is a patch extending the GNU Fortran front-end to support > some additional math intrinsics, enabled with a new compile flag > -fdec-math. The flag adds the COTAN intrinsic (cotangent), as well as > degree versions of all trigonometric intrinsics (SIND, TAND, ACOSD, >

Re: [PATCH] Fixed up missing semicolons.

2016-09-26 Thread Jeff Law
On 09/23/2016 09:28 AM, lhmouse wrote: Hi GCC developers, Today I was trying bootstrapping GCC 7.0.0 and stage 1 compilation failed because of two missing semicolons. After this patch was applied, xgcc could be built successfully, although it still failed the self-test.

Re: [PATCH 2/5] dce: Don't dead-code delete separately wrapped restores

2016-09-26 Thread Jeff Law
On 09/23/2016 02:21 AM, Segher Boessenkool wrote: If there is a separately wrapped register restore on some path that is dead (say, control goes into an endless loop after it), then we cannot delete that restore because that would confuse the DWARF CFI (if there is another path joining after

Re: [wwwdocs] Add some missing C++17 features

2016-09-26 Thread Jonathan Wakely
On 26/09/16 18:35 +0200, Jakub Jelinek wrote: Hi! The following patch adds some missing C++1z features to the table (wording from https://isocpp.org/std/standing-documents/sd-6-sg10-feature-test-recommendations#recs.cpp17 except for P0386R2 which is too new for that (Jon, can you look up the

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-26 Thread Rainer Orth
Hi Marek, > On Mon, Sep 26, 2016 at 03:34:40PM +0200, Rainer Orth wrote: >> Hi Marek, >> >> > All right. I'll commit the patch on Monday. >> >> this patch caused quite some breakage: Ada, Solaris/x86 and SPARC don't >> bootstrap any longer. > > Sorry about that. I had tested Ada +

Re: [PATCH, LRA] Fix PR 77714

2016-09-26 Thread Vladimir N Makarov
On 09/26/2016 09:55 AM, Bernd Edlinger wrote: Bootstrapped and reg-tested on x86_64-pc-linux-gnu. Is it OK for trunk? Yes. Thank you for working on the problem, Bernd.

Re: [Patch, avr] Backport fix for PR 65210 to gcc-5-branch

2016-09-26 Thread Denis Chertykov
Approved. Please commit. 2016-09-26 15:30 GMT+03:00 Senthil Kumar Selvaraj : > Ping! > > Regards > Senthil > Senthil Kumar Selvaraj writes: > >> Hi, >> >> Is it ok to backport PR 65210 to gcc-5-branch? The patch is already in >> 6.x and trunk. >> >> Regards

Re: [PATCH] Fix PR tree-optimization/77550

2016-09-26 Thread Bernd Edlinger
On 09/26/16 21:19, Christophe Lyon wrote: > > Sorry for the delay, I've been travelling. > > Compilation is: > .../xg++ > -B/aci-gcc-fsf/builds/gcc-fsf-gccsrc/obj-arm-none-eabi/gcc3/gcc/testsuite/g++/../../ > /aci-gcc-fsf/sources/gcc-fsf/gccsrc/gcc/testsuite/g++.dg/pr77550.C > g++_tg.o

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-26 Thread Rainer Orth
Hi Jakub, > On Mon, Sep 26, 2016 at 03:34:40PM +0200, Rainer Orth wrote: >> Ok for mainline if the bootstraps pass (with appropriate changelog >> entries, of course)? > > Yes. testing completed successfully, so I've installed the patch with this ChangeLog entry: 2016-09-26 Rainer Orth

Re: debug container mutex association

2016-09-26 Thread François Dumont
Fixed with attached patch. François On 26/09/2016 13:56, Andreas Schwab wrote: FAIL: 23_containers/list/debug/invalidation/4.cc (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20160926/Build/m68k-linux/libstdc++-v3/include/debug/safe_sequence.tcc:89: error: 'void __gnu_debug

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Richard Biener
On Thu, 22 Sep 2016, Thomas Preudhomme wrote: > Hi, > > This patch is part of a patch series to add support for ARMv8-M[1] to GCC. > This specific patch moves memory model declarations in memmodel.h. > > Currently, is_mm_* memory model related functions are declared in tree.h which > prevents

Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer

2016-09-26 Thread Robin Dapp
> Comments added. Bootstrap and test, is it reasonable? This introduces an ICE on s390x with a Fortran testcase, because of an assertion failure in tree_to_uhwi (DR_STEP (dr_a.dr)) for DR_STEP (dr_a.dr) = -8(OVF). The attached patch fixes this by introducing an additional tree_fits_uhwi_p().

Re: [patch] Fix ICE on ACATS test for Aarch64 at -O

2016-09-26 Thread Eric Botcazou
> So while emit_store_flag_force knows what to do upon NULL return, I'm > not sure the other users of expand_shift/expand_shift_1 do. Revised patch attached, tested on x86-64/Linux, OK for the mainline? 2016-09-26 Eric Botcazou * expmed.c (expand_shift_1):

Re: [PATCH] Fix missed debug PR77692

2016-09-26 Thread Richard Biener
On Fri, 23 Sep 2016, Richard Biener wrote: > > I am testing the following patch which re-instantiates behavior > to emit DW_AT_const_value attributes for optimized out non-readonly > global statics. Those are readonly by means of the ability to > remove them early (before any optimization such

RE: [PATCH] MIPS/doc: Fix `d' constraint description

2016-09-26 Thread Matthew Fortune
Maciej Rozycki writes: > Revert a part of the `d' constraint documentation update made with > commit 111868 ("Use define_constraint for MIPS"), > , > ,

Re: [PATCH v2] [RTEMS] Always use atomic builtins for libstdc++

2016-09-26 Thread Andreas Schwab
On Sep 22 2016, Bernhard Reutner-Fischer wrote: > On 22 September 2016 11:11:42 CEST, Andreas Schwab wrote: >>On Sep 22 2016, Sebastian Huber >>wrote: >> >>> diff --git a/libstdc++-v3/acinclude.m4

Re: [PATCH] Increase lto-min-partition

2016-09-26 Thread Richard Biener
On Sat, Sep 24, 2016 at 10:52 AM, Markus Trippelsdorf wrote: > On 2016.09.23 at 15:29 +0200, Richard Biener wrote: >> > >> > So 5 looks too big to me. >> >> I think the issue is that the default number of partitions is too high >> (32) which pessimizes 4-core machines

[wwwdocs] [[fallthrough]] attribute is supported in GCC 7

2016-09-26 Thread Marek Polacek
... so update C++1z Language Features. Applying. Index: projects/cxx-status.html === RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cxx-status.html,v retrieving revision 1.20 diff -u -r1.20 cxx-status.html --- projects/cxx-status.html

Re: Ping Re: Make max_align_t respect _Float128 [version 3]

2016-09-26 Thread Jeff Law
On 09/19/2016 09:58 AM, Joseph Myers wrote: Ping. This patch is pending review. OK. I never like adding more conditionals to stddef.h, but it's likely unavoidable in this case. jeff

Re: [v3 PATCH] Implement LWG 2729 for pair.

2016-09-26 Thread Ville Voutilainen
On 27 September 2016 at 03:21, Ville Voutilainen wrote: > On 27 September 2016 at 03:16, Christophe Lyon > wrote: >> 20_util/declval/requirements/1_neg.cc (test for errors, line 2263) >> 20_util/declval/requirements/1_neg.cc (test for

Re: [PATCH] Fix PR tree-optimization/77550

2016-09-26 Thread Christophe Lyon
On 26 September 2016 at 22:09, Bernd Edlinger wrote: > On 09/26/16 21:19, Christophe Lyon wrote: >> >> Sorry for the delay, I've been travelling. >> >> Compilation is: >> .../xg++ >> -B/aci-gcc-fsf/builds/gcc-fsf-gccsrc/obj-arm-none-eabi/gcc3/gcc/testsuite/g++/../../

Re: ICE at -O1 and above in both 32-bit and 64-bit modes on x86_64-linux-gnu

2016-09-26 Thread Richard Biener
On Sat, Sep 24, 2016 at 5:36 AM, kugan wrote: > Hi Richard, > > There is also one more issue as reported by Pat Haugen. That is, in > converting value_range of ssa (which we get from get_range_info) to > argument type, my implementation is too simplistic and

Re: [PATCH] Fix various minor gimple-ssa-sprintf.c issues

2016-09-26 Thread Gerald Pfeifer
Hi Martin, I'm afraid there may be further fallout from your patch. GCC now bootstraps fine on my tester, but building Wine I started to see internal compiler error: in format_floating, at gimple-ssa-sprintf.c:1165 four or so days ago. https://gcc.gnu.org/bugzilla/show_bug.cgi?id=77740 has

Re: [PATCH] Increase lto-min-partition

2016-09-26 Thread Markus Trippelsdorf
On 2016.09.26 at 09:42 +0200, Richard Biener wrote: > On Sat, Sep 24, 2016 at 10:52 AM, Markus Trippelsdorf > wrote: > > On 2016.09.23 at 15:29 +0200, Richard Biener wrote: > >> > > >> > So 5 looks too big to me. > >> > >> I think the issue is that the default number

Re: [v3 PATCH] PR libstdc++/77717

2016-09-26 Thread Jonathan Wakely
On 25/09/16 17:44 +0300, Ville Voutilainen wrote: PR libstdc++/77717 * testsuite/21_strings/basic_string_view/operations/compare/char/1.cc: Fix an out-of-bounds access. OK, thanks.

Re: [PATCH, GCC] Deal with singular sentences in builtin-sprintf-warn-1.c regex

2016-09-26 Thread Thomas Preudhomme
How about this reworked patch? Best regards, Thomas On 23/09/16 17:17, Martin Sebor wrote: On 09/23/2016 09:42 AM, Thomas Preudhomme wrote: Sorry, forgot the patch. Please find it attached. Best regards, Thomas On 23/09/16 16:40, Thomas Preudhomme wrote: Hi, New builtin-sprintf-warn-1.c

Re: [PATCH] Implement C++17 node extraction and insertion (P0083R5)

2016-09-26 Thread Jonathan Wakely
On 21/09/16 14:48 +0100, Jonathan Wakely wrote: This implements container node extraction/insertion, and merging. The patch includes Debug Mode support and pretty printers for the node handles. This adds some assertions to check for the mistake of trying to extract the end iterator. Tested

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Thomas Preudhomme
On 26/09/16 10:15, Richard Biener wrote: On Thu, 22 Sep 2016, Thomas Preudhomme wrote: Hi, This patch is part of a patch series to add support for ARMv8-M[1] to GCC. This specific patch moves memory model declarations in memmodel.h. Currently, is_mm_* memory model related functions are

[C++] fix some nits

2016-09-26 Thread Nathan Sidwell
In working on PR66443, I encountered a few nits. Fixed thusly: 1) a misaligned } in init.c 2) process_subob_fn used goto...label, when regular if {return} works fine. (I guess the control flow got simplified at some point) 3) implicitly_declare_fn repeats a lambda deleted fn check that

Re: [v3 PATCH] PR libstdc++/77727

2016-09-26 Thread Jonathan Wakely
On 26/09/16 02:06 +0300, Ville Voutilainen wrote: PR libstdc++/77727 * include/std/optional (optional(const optional<_Up>&)): Default-initialize the base and use emplace. (optional(optional<_Up>&&)): Likewise. * testsuite/20_util/optional/cons/77727.cc: New. OK, thanks.

Re: [PATCH, 1/7] Move memory model declarations in memmodel.h

2016-09-26 Thread Richard Biener
On Mon, 26 Sep 2016, Thomas Preudhomme wrote: > > > On 26/09/16 10:15, Richard Biener wrote: > > On Thu, 22 Sep 2016, Thomas Preudhomme wrote: > > > > > Hi, > > > > > > This patch is part of a patch series to add support for ARMv8-M[1] to GCC. > > > This specific patch moves memory model

[PATCH] Remove HIST_TYPE_CONST_DELTA counter type (PR, gcov-profile/23332).

2016-09-26 Thread Martin Liška
Following patch removes old type of GCOV counter that was used in times when RTL instrumentation was done. Patch is preapproved by Honza. Patch can bootstrap on x86_64-linux-gnu and survives regression tests. Martin >From 75d56f7882d1bc40f7c638462bb6fec3b84c3732 Mon Sep 17 00:00:00 2001 From:

Re: [Patch, fortran] Clean up of error recovery in DTIO procedures

2016-09-26 Thread Paul Richard Thomas
Dear Rainer and Dominique, Committed as obvious in revision 240493. Thanks for doing the testing. Cheers Paul 2016-09-26 Paul Thomas PR fortran/48298 * interface.c (gfc_find_specific_dtio_proc) : Return NULL if the derived type is broken, as indicated by a

Re: debug container mutex association

2016-09-26 Thread Jonathan Wakely
On 20/09/16 09:57 +0100, Jonathan Wakely wrote: On 19/09/16 21:56 +0200, François Dumont wrote: Hi Following our conversation here is a much simpler patch. I just consider that all debug containers will have the same alignment. Even if I submit this patch as a whole I will commit into

Re: [PATCH 6/17][ARM] Add data processing intrinsics for float16_t.

2016-09-26 Thread Matthew Wahab
Hello, On 25/09/16 14:00, Christophe Lyon wrote: This patch adds the new intrinsics: vbsl_f16, vbslq_f16, vdup_n_f16, vdupq_n_f16, vdup_lane_f16, vdupq_lane_f16, vext_f16, vextq_f16, vmov_n_f16, vmovq_n_f16, vrev64_f16, vrev64q_f16, vtrn_f16, vtrnq_f16, vuzp_f16, vuzpq_f16, vzip_f16,

[PATCH] Remove redefinition of __cpp_lib_array_constexpr

2016-09-26 Thread Jonathan Wakely
I mistakenly defined this macro before I committed the feature, and then we ended up with two (slightly different) definitions. * include/bits/range_access.h (__cpp_lib_array_constexpr): Do not redefine macro defined in . Tested powerpc64le-linux, committed to trunk. commit

Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer

2016-09-26 Thread Richard Biener
On Mon, Sep 26, 2016 at 11:21 AM, Robin Dapp wrote: >> Comments added. Bootstrap and test, is it reasonable? > > This introduces an ICE on s390x with a Fortran testcase, because > of an assertion failure in tree_to_uhwi (DR_STEP (dr_a.dr)) for > DR_STEP (dr_a.dr) =

Re: debug container mutex association

2016-09-26 Thread Andreas Schwab
FAIL: 23_containers/list/debug/invalidation/4.cc (test for excess errors) Excess errors: /daten/aranym/gcc/gcc-20160926/Build/m68k-linux/libstdc++-v3/include/debug/safe_sequence.tcc:89: error: 'void __gnu_debug::_Safe_iterator_base::_M_detach_single()' is protected within this context /daten

Re: [PATCH 6/17][ARM] Add data processing intrinsics for float16_t.

2016-09-26 Thread Christophe Lyon
On 26 September 2016 at 11:43, Matthew Wahab wrote: > Hello, > > On 25/09/16 14:00, Christophe Lyon wrote: This patch adds the new intrinsics: vbsl_f16, vbslq_f16, vdup_n_f16, vdupq_n_f16, vdup_lane_f16, vdupq_lane_f16, vext_f16, vextq_f16,

Re: [v3 PATCH] Implement LWG 2729 for pair.

2016-09-26 Thread Jonathan Wakely
On 24/09/16 21:13 +0300, Ville Voutilainen wrote: So, this has been applied to trunk, and also applies cleanly to the gcc-6 branch, and we need to backport it to fix PR libstdc++/77537. Ok for the gcc-6 branch? Yes, OK for gcc-6-branch too, as it fixes a regression since 5.x

Re: [PATCH 6/17][ARM] Add data processing intrinsics for float16_t.

2016-09-26 Thread Ramana Radhakrishnan
On Mon, Sep 26, 2016 at 1:48 PM, Christophe Lyon wrote: > On 26 September 2016 at 11:43, Matthew Wahab > wrote: >> Hello, >> >> On 25/09/16 14:00, Christophe Lyon wrote: > > > This patch adds the new intrinsics: >

Re: [PATCH 5/17][ARM] Enable HI mode moves for floating point values.

2016-09-26 Thread Christophe Lyon
Hi, Sorry for the delay, I've been travelling. On 27 July 2016 at 15:57, Ramana Radhakrishnan wrote: > On Tue, May 17, 2016 at 3:29 PM, Matthew Wahab > wrote: >> The handling of 16-bit integer data-movement in the ARM backend doesn't >>

Re: [PATCH 5/17][ARM] Enable HI mode moves for floating point values.

2016-09-26 Thread Matthew Wahab
On 26/09/16 14:15, Christophe Lyon wrote: Hi, Sorry for the delay, I've been travelling. On 27 July 2016 at 15:57, Ramana Radhakrishnan wrote: On Tue, May 17, 2016 at 3:29 PM, Matthew Wahab wrote: The handling of 16-bit integer

Split c-common.c?

2016-09-26 Thread Marek Polacek
Before I spend time on this, I wanted to check if you consider this a good idea. Since c-common.c has grown a lot and is quite large now, I think we might split it into c-warn.c, where various warning routines would go. What do you think? Marek

[patch, avr, pr71676 and pr71678] Issues with casesi expand

2016-09-26 Thread Pitchumani Sivanupandi
Attached patch for PR71676 and PR71678. PR71676 is for AVR target that generates wrong code when switch case index is more than 16 bits. Switch case index of larger than SImode are checked for out of range before 'casesi' expand. RTL expand of casesi gets index as SImode, but index is

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-26 Thread Rainer Orth
Hi Marek, > All right. I'll commit the patch on Monday. this patch caused quite some breakage: Ada, Solaris/x86 and SPARC don't bootstrap any longer. The following patch allows i386-pc-solaris2.12 and sparc-sun-solaris2.12 bootstraps continue. Strangely, I needed to use gcc_fallthrough () in

Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer

2016-09-26 Thread Bin.Cheng
On Mon, Sep 26, 2016 at 10:21 AM, Robin Dapp wrote: >> Comments added. Bootstrap and test, is it reasonable? > > This introduces an ICE on s390x with a Fortran testcase, because > of an assertion failure in tree_to_uhwi (DR_STEP (dr_a.dr)) for > DR_STEP (dr_a.dr) =

Re: [PATCH GCC][v2]Simplify alias check code generation in vectorizer

2016-09-26 Thread Markus Trippelsdorf
On 2016.09.26 at 13:14 +0200, Richard Biener wrote: > On Mon, Sep 26, 2016 at 11:21 AM, Robin Dapp wrote: > >> Comments added. Bootstrap and test, is it reasonable? > > > > This introduces an ICE on s390x with a Fortran testcase, because > > of an assertion failure in

Re: [PATCH] Increase lto-min-partition

2016-09-26 Thread Wilco Dijkstra
Prathamesh Kulkarni wrote: > Hi Wilco, > I am working on LTO varpool partitioning to improve performance for > section anchors. > I posted a preliminary patch posted at: > https://gcc.gnu.org/ml/gcc/2016-07/msg00033.html > Unfortunately I haven't yet been able to

Re: [PATCH 6/17][ARM] Add data processing intrinsics for float16_t.

2016-09-26 Thread Matthew Wahab
On 26/09/16 14:03, Ramana Radhakrishnan wrote: On Mon, Sep 26, 2016 at 1:48 PM, Christophe Lyon wrote: On 26 September 2016 at 11:43, Matthew Wahab wrote: Hello, On 25/09/16 14:00, Christophe Lyon wrote: This patch adds the new

Re: [PATCH 6/17][ARM] Add data processing intrinsics for float16_t.

2016-09-26 Thread Christophe Lyon
On 26 September 2016 at 15:03, Ramana Radhakrishnan wrote: > On Mon, Sep 26, 2016 at 1:48 PM, Christophe Lyon > wrote: >> On 26 September 2016 at 11:43, Matthew Wahab >> wrote: >>> Hello, >>> >>> On 25/09/16

Re: Split c-common.c?

2016-09-26 Thread Jakub Jelinek
On Mon, Sep 26, 2016 at 03:22:08PM +0200, Marek Polacek wrote: > Before I spend time on this, I wanted to check if you consider this a good > idea. Since c-common.c has grown a lot and is quite large now, I think we > might split it into c-warn.c, where various warning routines would go. What >

[PATCH] MIPS/doc: Fix `d' constraint description

2016-09-26 Thread Maciej W. Rozycki
Revert a part of the `d' constraint documentation update made with commit 111868 ("Use define_constraint for MIPS"), , , which inadvertently made the registers covered to be

Re: [Patch, reload, tentative, PR 71627] Tweak conditions in find_valid_class_1

2016-09-26 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > The below patch fixes what I think are a couple of problems with > reload.c:find_valid_class_1. > > First, even if no regno is in_hard_reg_set_p, it goes ahead and > considers rclass as valid. bad is set only if a regno is

Re: [Patch, avr] Backport fix for PR 65210 to gcc-5-branch

2016-09-26 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil Senthil Kumar Selvaraj writes: > Hi, > > Is it ok to backport PR 65210 to gcc-5-branch? The patch is already in > 6.x and trunk. > > Regards > Senthil > > gcc/ChangeLog > > 2016-09-22 Senthil Kumar Selvaraj > > Backport from

[PATCH, OBVIOUS] Fix documentation of gcov tool

2016-09-26 Thread Martin Liška
gcov program produces a different output compared to what's mentioned in documentation. Attached patch changes that. Installed as obvious. Martin >From 76d40ef712b42b60f74087a6ecbd6ee163554d5b Mon Sep 17 00:00:00 2001 From: marxin Date: Mon, 26 Sep 2016 13:51:55 +0200 Subject:

Re: [PATCH] Increase lto-min-partition

2016-09-26 Thread Wilco Dijkstra
Markus Trippelsdorf wrote: > On 2016.09.26 at 09:42 +0200, Richard Biener wrote: > > On Sat, Sep 24, 2016 at 10:52 AM, Markus Trippelsdorf > > wrote: > > > On 2016.09.23 at 15:29 +0200, Richard Biener wrote: > > > If for example you limit partitions to 4 on a 4-core

C++ PATCH for __has_cpp_attribute and fallthrough

2016-09-26 Thread Marek Polacek
This patch updates __has_cpp_attribute for fallthrough, which is now supported. The system.h hunk should aid people building trunk GCC with older GCC 7 not supporting this attribute yet. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2016-09-26 Marek Polacek

Re: Split c-common.c?

2016-09-26 Thread Marek Polacek
On Mon, Sep 26, 2016 at 03:26:28PM +0200, Jakub Jelinek wrote: > On Mon, Sep 26, 2016 at 03:22:08PM +0200, Marek Polacek wrote: > > Before I spend time on this, I wanted to check if you consider this a good > > idea. Since c-common.c has grown a lot and is quite large now, I think we > > might

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-26 Thread Jakub Jelinek
On Mon, Sep 26, 2016 at 03:34:40PM +0200, Rainer Orth wrote: > Ok for mainline if the bootstraps pass (with appropriate changelog > entries, of course)? Yes. > diff --git a/gcc/config/i386/i386.c b/gcc/config/i386/i386.c > --- a/gcc/config/i386/i386.c > +++ b/gcc/config/i386/i386.c > @@ -17917,6

Re: Implement -Wimplicit-fallthrough (version 9)

2016-09-26 Thread Marek Polacek
On Mon, Sep 26, 2016 at 03:34:40PM +0200, Rainer Orth wrote: > Hi Marek, > > > All right. I'll commit the patch on Monday. > > this patch caused quite some breakage: Ada, Solaris/x86 and SPARC don't > bootstrap any longer. Sorry about that. I had tested Ada + x86_64/ppc64/aarch64, but

[PATCH, LRA] Fix PR 77714

2016-09-26 Thread Bernd Edlinger
Hi! This fixes another fallout of the recent change in the pattern matching, which makes LRA choose a different alternative for this insn: (insn 48 21 23 3 (set (reg/f:SI 102 sfp) (reg/f:SI 7 r7)) 808 {*thumb1_movsi_insn} (nil)) This is replaced as a special case to

Re: [PATCH] Bits from Early LTO debug merge -- move stuff from late to early finish

2016-09-26 Thread Richard Biener
On Fri, 23 Sep 2016, Richard Biener wrote: > On Fri, 23 Sep 2016, Richard Biener wrote: > > > On Thu, 22 Sep 2016, Richard Biener wrote: > > > > > > > > This merges moving of unused type pruning from late to early finish as > > > well as handling of debug types and dwarf2 dups elimination.

PATCH to use the new gimple_call_internal_p overload

2016-09-26 Thread Marek Polacek
In my recent -Wimplicit-fallthrough patch I introduced a new overload gimple_call_internal_p to check whether a stmt is a specific internal function. So use this predicate in the rest of the codebase, too. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2016-09-26 Marek Polacek

[PATCH, Fortran] Extension: COTAN and degree-valued trig intrinsics with -fdec-math

2016-09-26 Thread Fritz Reese
All, Attached is a patch extending the GNU Fortran front-end to support some additional math intrinsics, enabled with a new compile flag -fdec-math. The flag adds the COTAN intrinsic (cotangent), as well as degree versions of all trigonometric intrinsics (SIND, TAND, ACOSD, etc...). This

Re: [PATCH] Fix handling of -fsanitize-recover* options.

2016-09-26 Thread Martin Liška
On 09/23/2016 02:48 PM, Jakub Jelinek wrote: > On Fri, Sep 23, 2016 at 01:53:48PM +0200, Martin Liška wrote: >> 3) -fsanitize=leak is combinable with -fsanitize=address or >>-fsanitize=thread > > Is it really combinable with -fsanitize=thread? I thought only libasan > or liblsan provides the

Re: [PATCH] Fix handling of -fsanitize-recover* options.

2016-09-26 Thread Jakub Jelinek
On Mon, Sep 26, 2016 at 11:56:01AM +0200, Martin Liška wrote: > @item -fsanitize=kernel-address > @opindex fsanitize=kernel-address > @@ -10169,17 +10170,19 @@ details. The run-time behavior can be influenced > using the @env{TSAN_OPTIONS} > environment variable; see >

[wwwdocs] Replace outdated C++ FAQ URL and fix grammar.

2016-09-26 Thread Jonathan Wakely
Marshall Cline's FAQ at parashift.com has been superseded by the isocpp.org FAQ. I've also replaced "resp.", as https://en.wiktionary.org/wiki/resp.#Usage_notes agrees that this usage isn't correct. Maybe it would be even better at the end of the sentence, but this is OK too. Index:

Re: Deprecating Libstdc++ Profile Mode

2016-09-26 Thread Jonathan Wakely
As discussed on the libstdc++ list (see below) I'm formally deprecating the Profile Mode, and will commit this patch to the wwwdocs. I don't plan to actually remove the code until SystemTap trace points have been added to the rest of the library to make the Profile Mode instrumentation

  1   2   >