Re: [wwwdocs] GCC 6 Release Notes for RTEMS

I checked this in. On 15/09/15 09:49, Sebastian Huber wrote: I think there are some head lines missing: Index: htdocs/gcc-6/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v retrieving revision 1.29 diff -u

[patch] Fix wrong code on dynamic array slices

Hi, the attached Ada testcase fails on x86/Linux (32-bit only) because the FRE pass wrongly deletes stores as redundant: Deleted redundant store VIEW_CONVERT_EXPR(*a.18_41)[0] {lb: 4294967292 sz: 4} = 100; Deleted redundant store

[PATCH] Refactor dwarf2out_late_global_decl

This makes dwarf2out_late_global_decl only do late work, namely add location/value attributes. For current LTO we need to make it do early_global_decl as well (with LTO early debug this will be conditionalized on !flag_lto_early_debug). Bootstrapped and tested on x86_64-unknown-linux-gnu and

Re: [PATCH] shrink-wrap: Fix up partitions (PR67587)

On 09/16/2015 02:17 AM, Segher Boessenkool wrote: With the new shrink-wrap algorithm, blocks reachable both with and without prologue are duplicated, and their incoming edges are then distributed accordingly. So we need to call fixup_partitions. Is this okay for trunk? I think so, given that

Re: [PATCH, i386]: Fix PR 67484, asan detects heap-use-after-free with target options

On Wed, Sep 16, 2015 at 10:45 AM, Richard Biener wrote: >> As mentioned in the PR, ix86_valid_target_attribute_tree creates >> temporary copies of current options strings and saves *pointers* to >> these copies with build_target_option_node. A couple of lines below,

Re: [PATCH, i386]: Fix PR 67484, asan detects heap-use-after-free with target options

I see in gtype-desc.c: void gt_ggc_mx_cl_target_option (void *x_p) { struct cl_target_option * const x = (struct cl_target_option *)x_p; if (ggc_test_and_set_mark (x)) { gt_ggc_m_S ((*x).x_ix86_arch_string); gt_ggc_m_S ((*x).x_ix86_recip_name); gt_ggc_m_S

[AArch64] Implement copysign[ds]f3

Hi, This patch adds expanders for copysigndf3 and copysignsf3 to the AArch64 backend. These use the BSL/BIT/BIF insn to save us from the default expansion pattern. Bootstrapped on aarch64-none-linux-gnu with no issues, and checked the performance to show a slight improvement to FP routines

Re: [PATCH, i386]: Fix PR 67484, asan detects heap-use-after-free with target options

And it is initialized via void cl_target_option_save (struct cl_target_option *ptr, struct gcc_options *opts) { if (targetm.target_option.save) targetm.target_option.save (ptr, opts); ptr->x_recip_mask = opts->x_recip_mask; ptr->x_ix86_isa_flags = opts->x_ix86_isa_flags;

[PATCH] Fix PR67442

This fixes PR67442, wide_int_to_tree used for type extension from a different signed value (which doesn't work). Bootstrapped and tested on x86_64-unknown-linux-gnu, applied on trunk. Richard. 2015-09-16 Richard Biener PR middle-end/67442 * fold-const.c

Re: [PATCH][C++] Avoid PCH dependent mangling

On Tue, 15 Sep 2015, Richard Biener wrote: > On September 15, 2015 4:25:37 PM GMT+02:00, Jason Merrill > wrote: > >On 08/27/2015 09:36 AM, Richard Biener wrote: > >> > >> With the passes.c hunk in the patch below we FAIL assembly comparison > >> of g++.dg/pch/system-[12].C

Re: [patch] Fix wrong code on dynamic array slices

Ok. Thanks, Richard. On Wed, Sep 16, 2015 at 9:07 AM, Eric Botcazou wrote: > Hi, > > the attached Ada testcase fails on x86/Linux (32-bit only) because the FRE > pass wrongly deletes stores as redundant: > > Deleted redundant store

Re: dejagnu version update?

Mike Stump writes: > The software presently works with 1.4.4 and there aren’t any changes > that require anything newer. SLES 12 has 1.4.4. Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA B9D7

Re: [PATCH WIP] Use Levenshtein distance for various misspellings in C frontend v2

On Tue, Sep 15, 2015 at 5:38 PM, David Malcolm wrote: > Updated patch attached, which is now independent of the rest of the > patch kit; see below. Various other comments inline. > > On Fri, 2015-09-11 at 17:30 +0200, Manuel López-Ibáñez wrote: > On 10/09/15 22:28, David

Re: [PATCH][ARM] Fix arm bootstrap failure due to -Werror=shift-negative-value

On 15/09/15 15:59, Kyrill Tkachov wrote: > Hi all, > > After Marek's patch I see the bootstrap failure on arm: > gcc/config/arm/arm.c:4601:46: error: left shift of negative value > [-Werror=shift-negative-value] > && (val & (-1 << (32 - set_sign_bit_copies))) == val) > > I believe this is

Re: [PATCH v5][aarch64] Implemented reciprocal square root (rsqrt) estimation in -ffast-math

ping patch is in: https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00417.html Thank you > On 07 Sep 2015, at 12:40, Benedikt Huber > wrote: > > This fifth revision of the patch: > * Moves a function declaration to a header. > * Adds comments to functions. >

Re: [patch match.pd c c++]: Ignore results of 'shorten_compare' and move missing patterns in match.pd

On Tue, Sep 15, 2015 at 2:54 PM, Kai Tietz wrote: > Hi Richi, > > thanks for the review. > > 2015-09-15 12:06 GMT+02:00 Richard Biener : >> On Tue, Sep 8, 2015 at 1:17 PM, Kai Tietz wrote: >>> Hi, >>> >>> This patch is

Re: [PATCH, i386]: Fix PR 67484, asan detects heap-use-after-free with target options

On Tue, Sep 15, 2015 at 8:13 PM, Uros Bizjak wrote: > Hello! > > As mentioned in the PR, ix86_valid_target_attribute_tree creates > temporary copies of current options strings and saves *pointers* to > these copies with build_target_option_node. A couple of lines below, > these

Re: [C++ Patch] PR 53184 ("Unnecessary anonymous namespace warnings")

Hi, On 09/15/2015 03:11 AM, Jason Merrill wrote: On 09/14/2015 06:17 PM, Paolo Carlini wrote: Hi Florian, On 09/14/2015 09:41 PM, Florian Weimer wrote: This warning is +enabled by default. Maybe add a sentence why this is bad? I can only guess, but I suspect the reason is this: Such types

Re: [RFC] Masking vectorized loops with bound not aligned to VF.

On Mon, 14 Sep 2015, Kirill Yukhin wrote: > Hello, > I'd like to initiate discussion on vectorization of loops which > boundaries are not aligned to VF. Main target for this optimization > right now is x86's AVX-512, which features per-element embedded masking > for all instructions. The main

Re: Fix 61441

Hi, >> I'll leave the correctness part of the patch to Joseph who knows FP >> arithmetic better than me, >> implementation-wise this is ok if you fix the REAL_CST sharing issue. Ok, will change this. > Changing fold_abs_const is wrong - abs of sNaN is sNaN, no exceptions > raised. Changing

[PATCH] Fix Cygwin bootstrap failing to find win32 libraries

libgcc is failing to find kerne32 etc during the 2nd stage when bootstraping, explicitly add w32api directory to search path. Patch OK? diff --git a/gcc/config/i386/cygwin.h b/gcc/config/i386/cygwin.h index 2a2a0bf..fd3bc0a 100644 --- a/gcc/config/i386/cygwin.h +++ b/gcc/config/i386/cygwin.h @@

Re: [C++] Coding rule enforcement

On 09/15/15 09:33, Jason Merrill wrote: On 09/15/2015 09:26 AM, Richard Biener wrote: Wouldn't warning flags be better so you can decide whether it's an error or a warning via -Werror=virtual-inheritance vs. -Wvirtual-inheritance? Yep. That also handles the system header exemption (unless

Re: [PATCH] Move language dependend DIE annotations early

On Wed, 16 Sep 2015, Richard Biener wrote: > > This moves gen_scheduled_generic_parms_dies and > gen_remaining_tmpl_value_param_die_attribute to early finish as we > are never populating them after that (the patch adds asserts to that > fact). I left handling "leftovers" of >

Re: [PATCH GCC][rework]Improve loop bound info by simplifying conversions in iv base

On Tue, Sep 15, 2015 at 7:57 AM, Bin.Cheng wrote: > Ping. Ok. Thanks, Richard. > On Thu, Aug 27, 2015 at 5:41 PM, Bin Cheng wrote: >> Hi, >> This is a rework for >> https://gcc.gnu.org/ml/gcc-patches/2015-07/msg02335.html, with review >> comments

PR67588: module.c heap use after free and some submodule tidying up

Dear All, The attached patch fixes a memory leak, prevents writing of smod files, when there are no submodules and adds a paragraph to the gfortran documentation on submodules. After this, PR66762, which is the -flto problem, is the last one to sort out in order to complete the implementation of

[gomp4] Break out dimension checking

I've applied this patch, which breaks out the offload function dimension checking to a separate function. I'll be making use of that shortly, and execute_oacc_transform was getting unnecessarily large. nathan 2015-09-16 Nathan Sidwell * omp-low.c

[gomp4] reduction cleanup

I noticed some unreachable code. Committed this. 2015-09-16 Nathan Sidwell * omp-low.c (lower_oacc_reductions): Remove dead initialization and unreachable code. Index: gcc/omp-low.c === ---

Re: [PATCH] New attribute to create target clones

PING 2. On Tue, Sep 8, 2015 at 2:27 PM, Evgeny Stupachenko wrote: > Ping. > > On Thu, Aug 27, 2015 at 2:18 PM, Evgeny Stupachenko > wrote: >> Hi All, >> >> Based on RFC: >> https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01322.html >> >> The patch

Re: [PATCH PR66388]Add sizetype cand for BIV of smaller type if it's used as index of memory ref

Ok. Thanks, Richard. On Tue, Sep 15, 2015 at 7:56 AM, Bin.Cheng wrote: > Just realized that I missed the updated patch before. Here it is... > > Thanks, > bin > > On Tue, Sep 8, 2015 at 6:07 PM, Bin.Cheng wrote: >> On Tue, Sep 8, 2015 at 6:06 PM,

[PATCH, RFC] Implement N4230, Nested namespace definition

This is the first stab, I haven't written the tests yet. Feedback would be most welcome; should I put this code into a separate function? Is the minor code duplication with the regular namespace definition ok? diff --git a/gcc/cp/parser.c b/gcc/cp/parser.c index 3a68dd7..00f18fb 100644 ---

[PATCH] Move language dependend DIE annotations early

This moves gen_scheduled_generic_parms_dies and gen_remaining_tmpl_value_param_die_attribute to early finish as we are never populating them after that (the patch adds asserts to that fact). I left handling "leftovers" of gen_remaining_tmpl_value_param_die_attribute in late finish as well as

Re: dejagnu version update?

On 09/15/2015 09:23 PM, Bernhard Reutner-Fischer wrote: > On September 15, 2015 7:39:39 PM GMT+02:00, Mike Stump > wrote: >> On Sep 14, 2015, at 3:37 PM, Jeff Law wrote: Maybe GCC-6 can bump the required dejagnu version to allow for getting rid

Re: [PATCH WIP] Use Levenshtein distance for various misspellings in C frontend v2

On Wed, Sep 16, 2015 at 3:22 PM, Michael Matz wrote: > Hi, > > On Wed, 16 Sep 2015, Richard Biener wrote: > >> Btw, this looks quite expensive - I'm sure we want to limit the effort >> here a bit. > > I'm not so sure. It's only used for printing an error, so walking all > available

Re: [PATCH WIP] Use Levenshtein distance for various misspellings in C frontend v2

Hi, On Wed, 16 Sep 2015, Richard Biener wrote: > Btw, this looks quite expensive - I'm sure we want to limit the effort > here a bit. I'm not so sure. It's only used for printing an error, so walking all available decls is expensive but IMHO not too much so. > I don't want us to suggest

Re: (patch,rfc) s/gimple/gimple */

On Wed, Sep 16, 2015 at 3:16 PM, Trevor Saunders wrote: > Hi, > > I gave changing from gimple to gimple * a shot last week. It turned out > to be not too hard. As you might expect the patch is huge so its > attached compressed. A quick skim over it makes it look good. >

[PATCH] Fix PR67271

Bootstrapped / tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-09-16 Richard Biener PR middle-end/67271 * fold-const.c (native_encode_expr): Bail out on bogus offsets. * gcc.dg/pr67271.c: New testcase. Index: gcc/fold-const.c

Re: [C++] Coding rule enforcement

On 09/16/2015 08:02 AM, Nathan Sidwell wrote: + else if (warn_multiple_inheritance) + warning (OPT_Wmultiple_inheritance, +"%qT defined with multiple direct bases", ref); You don't need to guard the warning with a check of the warning flag; warning will only give

Re: [C++] Coding rule enforcement

On 09/16/15 10:23, Jason Merrill wrote: On 09/16/2015 08:02 AM, Nathan Sidwell wrote: + else if (warn_multiple_inheritance) +warning (OPT_Wmultiple_inheritance, + "%qT defined with multiple direct bases", ref); You don't need to guard the warning with a check of the warning

[PATCH] Fix PR67253

Bootstrapped / tested on x86_64-unknown-linux-gnu, applied. Richard. 2015-09-16 Richard Biener PR middle-end/67253 * cfgexpand.c (expand_gimple_stmt_1): Do not clobber location of possibly shared trees. * gcc.dg/torture/pr67253.c: New

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, Sep 16, 2015 at 10:28 AM, Bill Schmidt wrote: > Hi, > > A recent patch proposal from Alan Hayward > (https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00690.html) uncovered > that the PowerPC back end doesn't have expansions for > reduc_{smax,smin,umax,umin}_ and

Re: [PATCH v2][GCC] Algorithmic optimization in match and simplify

On 03/09/15 12:11, Andre Vieira wrote: On 01/09/15 15:01, Richard Biener wrote: On Tue, Sep 1, 2015 at 3:40 PM, Andre Vieira wrote: Hi Marc, On 28/08/15 19:07, Marc Glisse wrote: (not a review, I haven't even read the whole patch) On Fri, 28 Aug 2015, Andre

[PATCH] Fix 67064 - Register asm variable broken

Hi, this patch fix the following bug: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=67064 for gcc 5.2 It passes all the gcc tests except for this: FAIL: g++.dg/cpp1y/auto-fn15.C -std=gnu++14 (test for excess errors) but this also happens without the patch. This patch was implemented for gcc 5.2

[PATCH, rs6000] Add expansions for min/max vector reductions

Hi, A recent patch proposal from Alan Hayward (https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00690.html) uncovered that the PowerPC back end doesn't have expansions for reduc_{smax,smin,umax,umin}_ and reduc_{smax,smin,umax,umin}_scal_ for the integer modes. This prevents vectorization of

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, Sep 16, 2015 at 3:34 PM, David Edelsohn wrote: > On Wed, Sep 16, 2015 at 10:28 AM, Bill Schmidt > > This is okay. > > I don't think that I have seen iterators for UNSPECs, but maybe > someone else is aware of the right idiom.

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 12/09/15 12:07 -0600, Martin Sebor wrote: On 09/12/2015 04:09 AM, Jonathan Wakely wrote: On 11 September 2015 at 18:39, Martin Sebor wrote: On 09/11/2015 08:21 AM, Jonathan Wakely wrote: Solaris 10 doesn't follow POSIX in accepting a null pointer as the second argument to realpath(), so

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, 2015-09-16 at 15:37 +0100, Ramana Radhakrishnan wrote: > On Wed, Sep 16, 2015 at 3:34 PM, David Edelsohn wrote: > > On Wed, Sep 16, 2015 at 10:28 AM, Bill Schmidt > > > > > This is okay. > > > > I don't think that I have seen iterators for UNSPECs, but maybe > >

Re: [AArch64] Implement copysign[ds]f3

On 16 September 2015 at 08:40, James Greenhalgh wrote: > > Hi, > > This patch adds expanders for copysigndf3 and copysignsf3 to the AArch64 > backend. These use the BSL/BIT/BIF insn to save us from the default > expansion pattern. > > Bootstrapped on

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, Sep 16, 2015 at 09:28:09AM -0500, Bill Schmidt wrote: > I was able to use iterators for the sub-doubleword ...scal_ > expansions, but that's all. I experimented with trying to use > code_iterators to generate the {smax,smin,umax,umin} expansions, but > couldn't find a way to make that

[PATCH] start of rewrite of unordered_{set,map}

This change is the start of a rewrite of unordered_set and unordered_map. (By the way, I am new to GCC and do not have SVN write access.) SUMMARY OF MOTIVATION FOR THE CHANGE 1. Speed. Linked lists are slow. Without them, we save memory, allowing us to default to a lower load factor; or, the

[C/C++ PATCH] RFC: Implement -Wduplicated-cond (PR c/64249)

This patch implements a new warning, -Wduplicated-cond. It warns for code such as if (x) // ... else if (x) // ... This happened in the GCC codebase as well: . The approach I took for this was to create vectors of conditions

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, 2015-09-16 at 10:14 -0500, Segher Boessenkool wrote: > On Wed, Sep 16, 2015 at 09:28:09AM -0500, Bill Schmidt wrote: > > I was able to use iterators for the sub-doubleword ...scal_ > > expansions, but that's all. I experimented with trying to use > > code_iterators to generate the

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On 16/09/15 15:28, Bill Schmidt wrote: 2015-09-16 Bill Schmidt * config/rs6000/altivec.md (UNSPEC_REDUC_SMAX, UNSPEC_REDUC_SMIN, UNSPEC_REDUC_UMAX, UNSPEC_REDUC_UMIN, UNSPEC_REDUC_SMAX_SCAL, UNSPEC_REDUC_SMIN_SCAL,

Re: dejagnu version update?

On September 16, 2015 3:01:47 PM GMT+02:00, Matthias Klose wrote: >On 09/15/2015 09:23 PM, Bernhard Reutner-Fischer wrote: >> On September 15, 2015 7:39:39 PM GMT+02:00, Mike Stump > wrote: >>> On Sep 14, 2015, at 3:37 PM, Jeff Law wrote:

Re: [PATCH WIP] Use Levenshtein distance for various misspellings in C frontend v2

On 16 September 2015 at 15:33, Richard Biener wrote: > On Wed, Sep 16, 2015 at 3:22 PM, Michael Matz wrote: >>> if we suggest 'foo' instead of foz then we'll get a more confusing followup >>> error if we actually use it. >> >> This particular case could

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On 16/09/15 17:10, Bill Schmidt wrote: On Wed, 2015-09-16 at 16:29 +0100, Alan Lawrence wrote: On 16/09/15 15:28, Bill Schmidt wrote: 2015-09-16 Bill Schmidt * config/rs6000/altivec.md (UNSPEC_REDUC_SMAX, UNSPEC_REDUC_SMIN,

Re: [PATCH] start of rewrite of unordered_{set,map}

Silly me, pasting the patch at the bottom of my original message didn't work. Patch is attached. Also, to clarify, I am primarily seeking high-level comments; I am new here and don't want to waste anybody's time. Index: libstdc++-v3/include/Makefile.am

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 16/09/15 15:42 +0100, Jonathan Wakely wrote: @@ -22,6 +22,8 @@ // see the files COPYING3 and COPYING.RUNTIME respectively. If not, see // . +#define _XOPEN_SOURCE 700 + #include #include #include Unfortunately this completely breaks NetBSD, because

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 16/09/15 17:02 +0100, Jonathan Wakely wrote: I don't know how to use _XOPEN_VERSION or _POSIX_VERSION to check for a suitable realpath without defining one of those feature-test macros, which then breaks other things. I suppose we could also define _NETBSD_SOURCE manually, which is

Re: [PATCH] Fix endianness assumption in LRA

On 09/15/2015 02:39 AM, David Miller wrote: This was the most glaring case, and would result in LRA crashing if this code snippet was actually hit on big-endian, since simplify_gen_subreg() will return NULL in such a case and then we try to blindly emit a move to 'subreg'. Vlad, is this OK to

Re: dejagnu version update?

On Sep 16, 2015, at 12:29 AM, Andreas Schwab wrote: > Mike Stump writes: > >> The software presently works with 1.4.4 and there aren’t any changes >> that require anything newer. > > SLES 12 has 1.4.4. Would be nice to cover them as well, but their

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, 2015-09-16 at 16:29 +0100, Alan Lawrence wrote: > > I proposed a patch to migrate PPC off the old patterns, but have forgotten to > ping it recently - last at > https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01024.html ... (ping?!) > Hi Alan, Thanks for this patch. I see that you

Re: dejagnu version update?

On 16/09/15 17:14, Mike Stump wrote: > On Sep 16, 2015, at 12:29 AM, Andreas Schwab wrote: >> Mike Stump writes: >> >>> The software presently works with 1.4.4 and there aren’t any changes >>> that require anything newer. >> >> SLES 12 has 1.4.4. > >

Re: Fix 61441

On Wed, 16 Sep 2015, Sujoy Saraswati wrote: > > If -fsignaling-nans, then folding of expressions involving sNaNs should be > > disabled, outside of static initializers - such expressions should not get > > folded to return an sNaN (it's incorrect to fold sNaN + 1 to sNaN, for > > example). I

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, 2015-09-16 at 16:29 +0100, Alan Lawrence wrote: > On 16/09/15 15:28, Bill Schmidt wrote: > > 2015-09-16 Bill Schmidt > > > > * config/rs6000/altivec.md (UNSPEC_REDUC_SMAX, UNSPEC_REDUC_SMIN, > > UNSPEC_REDUC_UMAX, UNSPEC_REDUC_UMIN,

Re: [C++] Coding rule enforcement

On 16/09/15 16:36, Nathan Sidwell wrote: On 09/16/15 10:23, Jason Merrill wrote: On 09/16/2015 08:02 AM, Nathan Sidwell wrote: + else if (warn_multiple_inheritance) +warning (OPT_Wmultiple_inheritance, + "%qT defined with multiple direct bases", ref); You don't need to guard

Re: dejagnu version update?

On 09/16/2015 10:25 AM, Ramana Radhakrishnan wrote: On 16/09/15 17:14, Mike Stump wrote: On Sep 16, 2015, at 12:29 AM, Andreas Schwab wrote: Mike Stump writes: The software presently works with 1.4.4 and there aren’t any changes that require

Go patch committed: Don't crash on erroneous channel reads

This patch by Chris Manghane fixes the Go frontend to not crash on erroneous channel reads. This fixes https://golang.org/issue/12320 . Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian Index: gcc/go/gofrontend/MERGE

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, 2015-09-16 at 19:16 +0100, Alan Lawrence wrote: > On 16/09/15 17:19, Bill Schmidt wrote: > > On Wed, 2015-09-16 at 16:29 +0100, Alan Lawrence wrote: > >> > >> I proposed a patch to migrate PPC off the old patterns, but have forgotten > >> to > >> ping it recently - last at > >>

Re: [C++] Coding rule enforcement

On September 16, 2015 6:32:46 PM GMT+02:00, "Manuel López-Ibáñez" wrote: >On 16/09/15 16:36, Nathan Sidwell wrote: >> On 09/16/15 10:23, Jason Merrill wrote: >>> On 09/16/2015 08:02 AM, Nathan Sidwell wrote: + else if (warn_multiple_inheritance) +warning

Re: [PATCH] Update ENABLE_CHECKING to make it usable in "if" conditions

On 09/10/2015 12:07 AM, Jeff Law wrote: >> On Mon, Aug 31, 2015 at 7:49 AM, Mikhail Maltsev >>> In lra.c we have: >>> >>> #ifdef ENABLE_CHECKING >>> >>> /* Function checks RTL for correctness. If FINAL_P is true, it is >>> done at the end of LRA and the check is more

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 16/09/15 11:36 -0600, Martin Sebor wrote: It turns out I was wrong about BSD traditionally supporting realpath(path, NULL), it first appeared in these relatively recent versions: FreeBSD 9.0 OpenBSD 5.0 NetBSD 6.1 Like Solaris 11, some of them still define _POSIX_VERSION=200112L even though

Re: dejagnu version update?

On Sep 16, 2015, at 9:25 AM, Ramana Radhakrishnan wrote: > > Sorry about the obvious (possibly dumb) question. > Can't we just import a copy of dejagnu each year and install it as part of > the source tree? TL;DR: No. We could, and indeed, some people do

[PATCH] Move check_global_declaration from toplev.c to cgraphunit.c

Unfortunately, toplev.c is a kitchen sink of things that do not belong anywhere in particular. For example, check_global_declarations is only used in cgraphunit.c. Moving it there allows us to make it static and remove one call to symtab_node::get. Bootstrapped & regtested on x86_64-linux-gnu.

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On 16/09/15 17:19, Bill Schmidt wrote: On Wed, 2015-09-16 at 16:29 +0100, Alan Lawrence wrote: I proposed a patch to migrate PPC off the old patterns, but have forgotten to ping it recently - last at https://gcc.gnu.org/ml/gcc-patches/2014-12/msg01024.html ... (ping?!) Hi Alan, Thanks for

Re: dejagnu version update?

On September 16, 2015 7:39:42 PM GMT+02:00, David Malcolm wrote: >On Wed, 2015-09-16 at 10:36 -0600, Jeff Law wrote: >> On 09/16/2015 10:25 AM, Ramana Radhakrishnan wrote: >> > >> > >> > On 16/09/15 17:14, Mike Stump wrote: >> >> On Sep 16, 2015, at 12:29 AM, Andreas Schwab

Re: [patch match.pd c c++]: Ignore results of 'shorten_compare' and move missing patterns in match.pd

On 09/15/2015 03:42 AM, Kai Tietz wrote: 2015-09-08 Kai Tietz * gcc.dg/tree-ssa/vrp23.c: Adjust testcase to reflect that pattern is matching now already within forward-propagation pass. * gcc.dg/tree-ssa/vrp24.c: Likewise. So for the new patterns, I

PR pretty-print/67567 do not pass NULL as a string

Fortran passes NULL where a non-null string is expected by the pretty-printer, which causes a sanitizer warning. This could have been found earlier by using gcc_checking_assert. Even if the assertion is false, the result is just an incomplete diagnostic, thus it seems more user-friendly to assert

Re: dejagnu version update?

On September 16, 2015 7:57:03 PM GMT+02:00, Mike Stump wrote: >On Sep 16, 2015, at 9:25 AM, Ramana Radhakrishnan > wrote: >> >> Sorry about the obvious (possibly dumb) question. > >> Can't we just import a copy of dejagnu each year and

Re: (patch,rfc) s/gimple/gimple */

On Wed, 2015-09-16 at 09:16 -0400, Trevor Saunders wrote: > Hi, > > I gave changing from gimple to gimple * a shot last week. It turned out > to be not too hard. As you might expect the patch is huge so its > attached compressed. > > patch was bootstrapped + regtested on x86_64-linux-gnu, and

Re: [PATCH] Update ENABLE_CHECKING to make it usable in "if" conditions

On 09/15/2015 04:11 AM, Richard Biener wrote: I wouldn't expect to see any in the .c files. There'll likely be stragglers (like tree/rtl checking). But even flushing all the stuff out of the .c files is a major win. Do you have a preference on whether or not to make flag_checking a pre,

Re: [PATCH] Move check_global_declaration from toplev.c to cgraphunit.c

On 09/16/2015 11:54 AM, Manuel López-Ibáñez wrote: Unfortunately, toplev.c is a kitchen sink of things that do not belong anywhere in particular. For example, check_global_declarations is only used in cgraphunit.c. Moving it there allows us to make it static and remove one call to

Re: vector lightweight debug mode

On 14/09/2015 21:50, Jonathan Wakely wrote: > On 14/09/15 20:27 +0200, François Dumont wrote: >> Hi >> >>Here is what I had in mind when talking about moving debug checks to >> the lightweight debug checks. > > Ah yes, I hadn't thought about removing reundant checks from the > __gnu_debug

Re: [PATCH, rs6000] Add expansions for min/max vector reductions

On Wed, 2015-09-16 at 18:03 +0100, Alan Lawrence wrote: > On 16/09/15 17:10, Bill Schmidt wrote: > > > > On Wed, 2015-09-16 at 16:29 +0100, Alan Lawrence wrote: > >> On 16/09/15 15:28, Bill Schmidt wrote: > >>> 2015-09-16 Bill Schmidt > >>> > >>> *

Re: vector lightweight debug mode

On 16/09/15 21:37 +0200, François Dumont wrote: On 14/09/2015 21:50, Jonathan Wakely wrote: On 14/09/15 20:27 +0200, François Dumont wrote: diff --git a/libstdc++-v3/include/bits/stl_vector.h b/libstdc++-v3/include/bits/stl_vector.h index 305d446..89a9aec 100644 ---

Re: Openacc launch API

Ping? On 09/11/15 11:50, Nathan Sidwell wrote: Ping? https://gcc.gnu.org/ml/gcc-patches/2015-08/msg01498.html On 09/07/15 08:48, Nathan Sidwell wrote: On 08/25/15 09:29, Nathan Sidwell wrote: Jakub, This patch changes the launch API for openacc parallels. The current scheme passes the

Re: [C/C++ PATCH] RFC: Implement -Wduplicated-cond (PR c/64249)

On 09/16/2015 09:59 AM, Marek Polacek wrote: This patch implements a new warning, -Wduplicated-cond. It warns for code such as if (x) // ... else if (x) // ... As usual, I like this improvement. I think it's worth extending to conditional expressions (e.g., x ? f() : x ? g()

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 16/09/15 16:04 -0600, Martin Sebor wrote: Tested powerpc64le-linux, x86_64-dragonfly4.1 and x86_64-netbsd5.1, do you see any reason not to commit this for now? I see only a couple of potential problems: a missing test for PATH_MAX in the unlikely event it's not defined (or is obscenely In

Re: [PATCH 07/22] Implement token range tracking within libcpp and C/C++ FEs

On Tue, 2015-09-15 at 12:48 +0200, Jakub Jelinek wrote: > On Tue, Sep 15, 2015 at 12:33:58PM +0200, Richard Biener wrote: > > On Tue, Sep 15, 2015 at 12:20 PM, Jakub Jelinek wrote: > > > On Tue, Sep 15, 2015 at 12:14:22PM +0200, Richard Biener wrote: > > >> > diff --git

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

Tested powerpc64le-linux, x86_64-dragonfly4.1 and x86_64-netbsd5.1, do you see any reason not to commit this for now? I see only a couple of potential problems: a missing test for PATH_MAX in the unlikely event it's not defined (or is obscenely large), and a missing check to avoid infinite

[PATCH], PowerPC IEEE 128-bit patch #7 (revised)

This is a revised patch that adds the parts to enable generation of IEEE 128-bit floating point support in the PowerPC backend. The major user visible change of this patch is to change the switch from -mfloat128-software and -mfloat128-none to just -mfloat128 and -mno-float128. I also revised

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 09/16/2015 04:17 PM, Jonathan Wakely wrote: On 16/09/15 16:04 -0600, Martin Sebor wrote: Tested powerpc64le-linux, x86_64-dragonfly4.1 and x86_64-netbsd5.1, do you see any reason not to commit this for now? I see only a couple of potential problems: a missing test for PATH_MAX in the

Re: dejagnu version update?

On Wed, Sep 16, 2015 at 01:39:42PM -0400, David Malcolm wrote: > AIUI, we specifically need >= 1.5.3 (or a version with a backport) to > get support for multiple load_lib paths mentioned by Bernhard, which is > what motivated this thread (on gcc-patches, before it spread to the gcc > list): We

Re: [patch] libstdc++/67173 Fix filesystem::canonical for Solaris 10.

On 16/09/15 19:58 +0100, Jonathan Wakely wrote: commit ef25038796485298ff8f040bc79e0d9a371171fa Author: Jonathan Wakely Date: Wed Sep 16 18:07:32 2015 +0100 Implement filesystem::canonical() without realpath PR libstdc++/67173 * acinclude.m4

Re: [PATCH, PR 57195] Allow mode iterators inside angle brackets

On 09/14/2015 02:34 AM, Richard Sandiford wrote: Michael Collison writes: Here is a modified patch that takes your comments into account. Breaking on depth == 0 with '>' does not work due to the code looking for whitespace. What goes wrong? Just to make sure

[PING] [Aarch64] Use vector wide add for mixed-mode adds

Ping. Originally posted here: https://gcc.gnu.org/ml/gcc-patches/2015-09/msg00408.html Regards, Michael Collison -- Michael Collison Linaro Toolchain Working Group michael.colli...@linaro.org

[patch committed SH] Fix PR target/67573

I've committed the attached patch to fix PR67573. LRA reveals that scratch registers in *call_*pcrel should be defined as early clobbered. The patch is tested on sh4-unkown-linux-gnu. Regards, kaz -- 2015-09-16 Kaz Kojima PR target/67573 *

[PATCH] Pass in avail_expr_stack and setup edge_infos earlier

Next set of changes on the road to fixing 47679. First, we want to pass in the avail_expr_stack to thread_across_edge. That's going to eventually allow the threader to record temporary expression equivalences (after a bit more refactoring). Second, we need to set up the edge_info structures

[patch committed FT32] Fix address space predicate

The attached patch fixes the FT32's memory address space predicate. [gcc] 2015-09-16 James Bowman * config/ft32/ft32.c Fix the memory address space predicate. Index: gcc/config/ft32/ft32.c ===

Re: [PATCH][C++] Avoid PCH dependent mangling

OK. Jason

Re: dejagnu version update?

On Wed, Sep 16, 2015 at 11:27 PM, Mike Stump wrote: > On Sep 16, 2015, at 9:25 AM, Ramana Radhakrishnan > wrote: >> >> Sorry about the obvious (possibly dumb) question. > >> Can't we just import a copy of dejagnu each year and install it

  1   2   >