Re: C++ PATCH to fix missing warning (PR c++/70194)

On Tue, Mar 15, 2016 at 03:41:52PM -0400, Jason Merrill wrote: > Let's factor out that duplicated code into a separate function. Sure. It also allowed me to hoist the cheap tests for both warnings, and while at it, I used 'location' for the first warning. Bootstrapped/regtested on x86_64-linux,

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

On Wed, Mar 16, 2016 at 01:38:21PM -0600, Jeff Law wrote: > On 03/14/2016 04:13 PM, Jakub Jelinek wrote: > >On Mon, Mar 14, 2016 at 03:25:07PM -0600, Martin Sebor wrote: > >>PR c++/67376 - [5/6 regression] Comparison with pointer to past-the-end > >>of array fails inside constant expression >

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

On Thu, Mar 17, 2016 at 8:23 AM, Bernd Schmidt wrote: > On 03/17/2016 04:13 PM, H.J. Lu wrote: >> >> On Thu, Mar 17, 2016 at 8:09 AM, Bernd Schmidt >> wrote: >>> >>> On 03/17/2016 04:06 PM, H.J. Lu wrote: This is the patch I am going to

[PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

Hi! On Wed, Mar 16, 2016 at 05:48:33PM -0500, Segher Boessenkool wrote: > On Wed, Mar 16, 2016 at 01:59:29PM +0100, Bernd Schmidt wrote: > > On 03/16/2016 01:22 PM, Jakub Jelinek wrote: > > >So, this is what we've converged to on IRC and passed bootstrap/regtest > > >on x86_64-linux and

Re: [PATCH] Fix PR64764

On Mon, Feb 9, 2015 at 2:30 AM, Tom de Vries wrote: > On 09-02-15 09:59, Richard Biener wrote: >> >> On Thu, 5 Feb 2015, Tom de Vries wrote: >> >>> On 26-01-15 15:47, Richard Biener wrote: Index: gcc/testsuite/gcc.dg/uninit-19.c

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245)

On Wed, Mar 16, 2016 at 01:59:29PM +0100, Bernd Schmidt wrote: > On 03/16/2016 01:22 PM, Jakub Jelinek wrote: > >So, this is what we've converged to on IRC and passed bootstrap/regtest > >on x86_64-linux and i686-linux. Is this ok for trunk? > > The explanation was a bit confusing at first, but

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. I guess so, but it's one allocation per compiled

Re: [Patch, testsuite] Skip testcase for avr

Rainer Orth writes: > Hi Senthil, > >> diff --git a/gcc/testsuite/gcc.c-torture/compile/20151204.c >> b/gcc/testsuite/gcc.c-torture/compile/20151204.c >> index 036316c..0a60871 100644 >> --- a/gcc/testsuite/gcc.c-torture/compile/20151204.c >> +++ b/gcc/testsuite/gcc.c-torture/compile/20151204.c

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

On Thu, Mar 17, 2016 at 12:16:58PM +0100, Jakub Jelinek wrote: > the rs6000 backend for whatever strange reason I haven't understood > really wants pointer equality instead of REGNO comparison (even when the > modes match), one (reg:DI 12) should be replaced, another (reg:DI 12) > should not. By

Re: [05/05] Fix PR 69102

On 03/15/2016 09:55 AM, Andrey Belevantsev wrote: Hello, On 14.03.2016 12:52, Andrey Belevantsev wrote: Hello, The problem here is readonly dependence contexts in selective scheduler. We're trying to cache the effect of initializing a dependence context with remembering that context and

Re: C++ PATCH to fix missing warning (PR c++/70194)

On Wed, Mar 16, 2016 at 06:43:39PM -0600, Martin Sebor wrote: > >@@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, > >return build3 (VEC_COND_EXPR, type, cmp, minus_one_vec, zero_vec); > > } > > > >+/* Possibly warn about an address never being NULL. */ > >+ > >+static void >

[PATCH] PR testsuite/70150: Check non-pic/ia32 in stackprotectexplicit2.C

For ia32, __stack_chk_fail isn't called in PIC. We need to check non-pic or non-ia32 before scanning for __stack_chk_fail. OK for trunk? H.J. --- PR testsuite/70150 * g++.dg/stackprotectexplicit2.C: Scanning for __stack_chk_fail only for non-pic or non-ia32 . ---

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

On Thu, Mar 17, 2016 at 7:55 AM, H.J. Lu wrote: > On Thu, Mar 17, 2016 at 7:18 AM, Bernd Schmidt wrote: >> On 03/17/2016 02:59 PM, H.J. Lu wrote: >>> >>> On Fri, Mar 11, 2016 at 9:09 AM, H.J. Lu wrote: We can't set

Re: [PATCH] Fix PR c++/70205 (ICE on valid call to qualified static member function)

OK. Jason

Re: [PATCH][ARM][5 Backport] PR target/69875 Fix atomic_loaddi expansion

On Wed, Feb 24, 2016 at 11:23 AM, Kyrill Tkachov wrote: > Hi all, > > This is the GCC 5 backport of > https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01338.html. > The differences are that TARGET_HAVE_LPAE has to be defined in arm.h in a > different way because > the

Re: [AArch64] Add precision choices for the reciprocal square root approximation

Hi Evandro, > For example, though this approximation is improves the performance > noticeably for DF on A57, for SF, not so much, if at all. I'm still skeptical that you ever can get any gain on scalars. I bet the only gain is on 4x vectorized floats. So what I would like to see is this

Re: [AArch64] Emit square root using the Newton series

On 03/08/16 16:08, Evandro Menezes wrote: On 02/16/16 14:56, Evandro Menezes wrote: On 12/08/15 15:35, Evandro Menezes wrote: Emit square root using the Newton series 2015-12-03 Evandro Menezes gcc/ * config/aarch64/aarch64-protos.h

Re: Patch ping

OK. Jason

[AArch64] Add precision choices for the reciprocal square root approximation

Add precision choices for the reciprocal square root approximation Allow a target to prefer such operation depending on the FP precision. gcc/ * config/aarch64/aarch64-protos.h (AARCH64_EXTRA_TUNE_APPROX_RSQRT): New macro. *

Re: [C PATCH] Prevent -Wunused-value warning with __atomic_fetch_* (PR c/69407)

On Mon, Mar 7, 2016 at 2:34 PM, Marek Polacek wrote: > On Fri, Mar 04, 2016 at 07:17:46PM +0100, Uros Bizjak wrote: >> Hello! >> >> > This is not a regression but I thought I'd post this anyway. Martin >> > reported >> > that we generate -Wunused-value warnings on the

[PATCH] Change replace_rtx if from is a REG (PR target/70245)

Hi! The following testcase is miscompiled on ia32, because a peephole2 calls replace_rtx trying to replace SImode %ecx with SImode %edx, but replace_rtx (unlike e.g. simplify_replace_rtx or validate_replace_rtx), in addition to modifying the rtxes in place (fine in this case) only does pointer

Re: Wonly-top-basic-asm

On 3/14/2016 8:28 AM, Bernd Schmidt wrote: The example is not good, as discussed previously, and IMO the best option is to remove it. Otherwise I have no objections to the latest variant. Despite the problems I have with the existing sample, adding the information/warnings is more important

[PATCH V2]PR other/70268: map one directory name (old) to another (new) in __FILE__

Changed in V2: - Rebase to latest master (f958b9e9d4dfed811f36217c800194dd82becf01) - Fix incorrect PR in libcpp/ChangeLog and gcc/ChangeLog //Hongxu >From b8761da98466242e1d18b4e39eb0ea3c6cee4d73 Mon Sep 17 00:00:00 2001 From: Hongxu Jia Date: Wed, 16 Mar 2016

[committed] PR c/70264: fix crash in compatible_locations_p with BUILTINS_LOCATION

In r234088 my fix for PR c++/70105 didn't allow for the possibility that when comparing a pair of macro expansion histories that one of the macros in the history might not be located within a line-map, and PR c/70264 reports a crash due to encountering BUILTINS_LOCATION within the traversal.

Re: C++ PATCH to fix missing warning (PR c++/70194)

On 03/16/2016 08:43 PM, Martin Sebor wrote: @@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, return build3 (VEC_COND_EXPR, type, cmp, minus_one_vec, zero_vec); } +/* Possibly warn about an address never being NULL. */ + +static void +warn_for_null_address (location_t

[oacc, testsuite, comitted] Add goacc/kernels-alias-{3,4}.f95

Hi, I've translated the goacc/kernels-alias-{3,4}.c testcases to fortran. Committed to trunk. Thanks, - Tom Add goacc/kernels-alias-{3,4}.f95 2016-03-17 Tom de Vries * gfortran.dg/goacc/kernels-alias-3.f95: New test. * gfortran.dg/goacc/kernels-alias-4.f95: New

Re: [PATCH][ARM][4.9 Backport] PR target/69875 Fix atomic_loaddi expansion

On Wed, Feb 24, 2016 at 11:23 AM, Kyrill Tkachov wrote: > Hi all, > > This is the GCC 4.9 backport of > https://gcc.gnu.org/ml/gcc-patches/2016-02/msg01338.html. > The differences are that TARGET_HAVE_LPAE has to be defined in arm.h in a > different way because > the

Re: [PATCH][SPARC] sparc: switch -fasynchronous-unwind-tables on by default.

> The backtrace(3) implementation for sparc contains a simple unwinder > that works well in most cases, but that unwinder is not used if > libgcc_s.so can be dlopened and it provides _Unwind_Backtrace. There's no reason that simple unwinder can't be put into

Re: C++ PATCH to fix missing warning (PR c++/70194)

OK. Jason

[PATCH] Add debug_varinfo and debug_varmap

[ was: Re: [RFC] dump_varmap in tree-ssa-structalias.c ] On 10/03/16 10:07, Richard Biener wrote: On Thu, 10 Mar 2016, Tom de Vries wrote: Hi, I wrote attached patch to print the actual contents of the varmap variable in tree-ssa-structalias.c. Does it make sense to rewrite this into a

Re: [i386] Support .lbss etc. sections with Solaris as (PR target/59407)

On Thu, Mar 17, 2016 at 11:40 PM, Rainer Orth wrote: > gcc.target/i386/pr58218.c currently FAILs on 64-bit Solaris/x86 with the > native assembler: > > FAIL: gcc.target/i386/pr58218.c (test for excess errors) > > Excess errors: > Assembler: pr58218.c >

[C++ PATCH] Fix -flifetime-dse bug in dtors too (PR c++/70272)

Hi! This is a follow-up to the PR70259 that got fixed yesterday. As the testcase shows, clobbers for empty classes are harmful not just in the ctors, but also in the dtors, where they can result in stores being removed before the corresponding dtor has acted upon them. The ehcleanup1.C part of

Contents of PO file 'cpplib-6.1-b20160131.fr.po'

cpplib-6.1-b20160131.fr.po.gz Description: Binary data The Translation Project robot, in the name of your translation coordinator.

Re: [PATCH] Fix PR c++/70121 (premature folding of const var that was implicitly captured)

On 03/10/2016 05:58 PM, Patrick Palka wrote: This patch reverses the behavior of process_outer_var_ref, so that we always implicitly capture a const variable if it's capturable, instead of always trying to first fold it to a constant. This behavior however is wrong too, and introduces a

Re: C++ PATCH to fix missing warning (PR c++/70194)

On 03/17/2016 02:51 PM, Martin Sebor wrote: On 03/17/2016 10:48 AM, Patrick Palka wrote: On Thu, Mar 17, 2016 at 12:27 PM, Jeff Law wrote: On 03/16/2016 06:43 PM, Martin Sebor wrote: @@ -3974,6 +3974,38 @@ build_vec_cmp (tree_code code, tree type, return build3

Re: [Patch, testsuite] Skip testcase for avr

On 03/16/2016 11:33 AM, Senthil Kumar Selvaraj wrote: 2016-03-16 Senthil Kumar Selvaraj * gcc.c-torture/compile/20151204.c: Skip for avr. THanks. Applied to the trunk. jeff

Re: [PATCH][ARM][testsuite][committed] Do not override -mcpu in no-volatile-in-it.c

On 16/07/15 16:31, Kyrill Tkachov wrote: > Hi all, > > This scan-assembler test was failing for me when testing with an > explicit /-march=armv7-a variant because > it clashed with the -mcpu=cortex-m7 and overrode it. > > This patch skips the test if the user forces an incompatible -march or >

[omp] Create openmp -fopt-info optimization group

Hi, the following patch does two things. First, it creates a new optinfo group for OpenMP and moves OpenMP lowering and expansion to this group. Second, it changes all gridification MSG_NOTE dumps to MSG_MISSED_OPTIMIZATION, which is more appropriate. (Apparently, I remembered to change the

Re: Please include ada-hurd.diff upstream (try2)

> Yes, you are right, somehow I submitted the old patch. Attached is the > updated one, with a proposed ChangeLog entry included. The copyright notices are wrong (or at least incomplete). Arno > > Thanks! > 2016-03-16 Svante Signell > > *

[PING] [PATCH] c++/65579 - set readonly bit on static constexpr members of templates

I'm looking for a review of the patch below. The problem isn't new (it has existed since at least 4.9) but the fix seems trivial and safe. If it isn't appropriate at this stage please let me know and I'll resubmit it when stage 1 opens.

[gomp-nvptx 3/7] libgomp plugin: make cuMemFreeHost error non-fatal

Unlike cuMemFree and other resource-releasing functions called on exit, cuMemFreeHost appears to re-report errors encountered in kernel launch. This leads to a deadlock after GOMP_PLUGIN_fatal is reentered. While the behavior on libgomp side is suboptimal (there's no need to call

[COMMITTED] Fix 70240

As discussed in the PR, the fix for 68215 was a bit too aggressive and caused this one. There's a simple alternate fix, first suggested by Richi in 68714, that cures both. Thus I apply one patch and revert another, in order, so that nothing breaks in between yet keeps the two commits

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. + bitmap_set_bit (seen_insns, INSN_UID (insn)); + if (! INSN_P (insn))

[C PATCH] Fix up composite_types (PR c/70280)

Hi! Zdenek reported a compare debug issue, where it is dumping used function prototypes and there is a difference between -g0 and -g in -2: static int BIO_vsnprintf (char *, size_t, const char *, struct *, void, ...); +2: static int BIO_vsnprintf (char *, size_t, const char *, struct *);

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

On 03/17/2016 03:16 PM, Martin Sebor wrote: static tree cxx_eval_constant_expression (const constexpr_ctx *, tree, - bool, bool *, bool *, tree * = NULL); + bool, bool *, bool *, bool * = NULL, + tree * = NULL);

Re: [PATCH] c++/67376 Comparison with pointer to past-the-end, of array fails inside constant expression

On 03/17/2016 03:16 PM, Martin Sebor wrote: gcc-67376.patch PR c++/67376 - [5/6 regression] Comparison with pointer to past-the-end of array fails inside constant expression PR c++/70170 - [6 regression] bogus not a constant expression error comparing pointer to array to null

[Patch, fortran] PR68566 ICE on using unusable array in reshape (double free or corruption)

Hi, the above ICE is fixed by the following simple/trivial fix: Index: gcc/fortran/simplify.c === --- gcc/fortran/simplify.c (revision 234170) +++ gcc/fortran/simplify.c (working copy) @@ -5163,6 +5163,9 @@ ||

Re: [PATCH V3]PR other/70268: map one directory name (old) to another (new) in __FILE__

On 03/18/2016 04:05 PM, Hongxu Jia wrote: Please explain why you first alloca() and then strdup the result instead of XNEWVEC BTW, I just refer the implement of -fdebug-prefix-map: In gcc/final.c const char * remap_debug_filename (const char *filename)

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On 03/16/2016 08:38 AM, H.J. Lu wrote: FAIL: g++.dg/abi/pr60336-1.C scan-assembler jmp[\t ]+[^$]*?_Z3xxx9true_type FAIL: g++.dg/abi/pr60336-5.C scan-assembler jmp[\t ]+[^$]*?_Z3xxx9true_type FAIL: g++.dg/abi/pr60336-6.C scan-assembler jmp[\t ]+[^$]*?_Z3xxx9true_type FAIL:

Re: [RFC] Getting LTO incremental linking work

On Wed, Nov 25, 2015 at 12:59 AM, Jan Hubicka wrote: > Hi, > PR 67548 is about LTO not supporting incremental linking. I never really > considered our current incremental linking very useful, because it triggers > code generation at the incremental link time basically nullifying

Re: [PATCH] c++/65579 - set readonly bit on static constexpr members of templates

On 03/09/2016 05:09 PM, Martin Sebor wrote: While going through constexpr bugs looking for background on one I'm currently working on I came across bug 65579 - [C++11] gcc requires definition of a static constexpr member even though it is not odr-used. The bug points out that GCC (sometimes)

[committed patch] Sync top-level configure.ac with binutils-gdb

I'm committing this patch to sync the top-level configure with binutils-gdb. -cary 2016-03-17 Cary Coutant * configure.ac: Add mips and s390 to the gold target check. * configure: Regenerate. Index: configure

Re: [PATCH, rs6000] Add support for xxpermr and vpermr instructions

On Thu, Mar 17, 2016 at 2:58 PM, Kelvin Nilsen wrote: > > This patch adds support for two new Power9 instructions, xxpermr and vpermr, > providing more efficient vector permutation operations on > little-endian configurations. These new instructions are described in >

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On 03/17/2016 02:02 PM, Jeff Law wrote: On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. +

Emit square root using the Newton series

2016-03-16 Evandro Menezes Wilco Dijkstra gcc/ * config/aarch64/aarch64-tuning-flags.def (AARCH64_EXTRA_TUNE_APPROX_SQRT_{SF,DF}): New tuning macros. * config/aarch64/aarch64-protos.h

Re: PING: [PATCH] PR driver/70192: Properly set flag_pie and flag_pic

On Thu, Mar 17, 2016 at 8:09 AM, Bernd Schmidt wrote: > On 03/17/2016 04:06 PM, H.J. Lu wrote: >> >> This is the patch I am going to check in. > > > That still mentions darwin which I imagine might not be an exhaustive test. > We can add an effective target, something like

Re: [RFA 1/2]: Don't ignore target_header_dir when deciding inhibit_libc

On 23/10/15 12:31, Bernd Schmidt wrote: > On 10/12/2015 11:58 AM, Ulrich Weigand wrote: >> >> Index: gcc/configure.ac >> === >> --- gcc/configure.ac(revision 228530) >> +++ gcc/configure.ac(working copy) >> @@ -1993,7 +1993,7

[PATCH, PR tree-optimization/70252] Fix boolean vectors conversion

Hi, Current widening and narrowing vectorization functions may work incorrectly for scalar masks because we may have different boolean vector types having the same mode. E.g. vec(4) and vec(8) both have QImode. That means if we need to convert vec(4) into vec(16) we may actually find

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

On 03/17/2016 12:16 PM, Jakub Jelinek wrote: Thus, I've reverted the patch (kept the testcase), and after some discussions on IRC bootstrapped/regtested on x86_64-linux and i686-linux following version, which right now should change behavior just for the i?86 case and nothing else, so shouldn't

Re: [gomp4.1] map clause parsing improvements

Hi! On Mon, 19 Oct 2015 12:34:08 +0200, Jakub Jelinek wrote: > On Mon, Oct 19, 2015 at 12:20:23PM +0200, Thomas Schwinge wrote: > > > @@ -77,7 +79,21 @@ enum gomp_map_kind > > > +/* OpenMP 4.1 alias for forced deallocation. */ > > > +GOMP_MAP_DELETE =

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On 03/16/2016 07:55 AM, H.J. Lu wrote: On Tue, Mar 15, 2016 at 7:51 PM, Jason Merrill wrote: On 03/15/2016 08:25 PM, Joseph Myers wrote: On Tue, 15 Mar 2016, H.J. Lu wrote: On Tue, Mar 15, 2016 at 3:34 PM, Joseph Myers wrote: On Tue, 15 Mar

Re: [patch] libstdc++/69945 Add __gnu_cxx::__freeres hook

On Thu, 2016-03-03 at 16:34 +0100, Mark Wielaard wrote: > On Wed, 2016-02-24 at 18:35 +, Jonathan Wakely wrote: > > This adds a new function to libsupc++ which will free the memory still > > in use by the pool used for allocating exceptions when malloc fails. > > > > This is similar to

C++ PATCH for c++/70139 (-fno-elide-constructors breaks regex)

The constexpr code for shortcutting trivial copy ctor/op= didn't get updated for the C++14 constexpr implementation, where we need to consider side effects. For GCC 5.4 I'm just going to disable the shortcut. Tested x86_64-pc-linux-gnu, applying to trunk. commit

Re: [PATCH][PR rtl-optimization/70024] Fix argument to CROSSING_JUMP_P

On 03/16/2016 11:37 AM, Andreas Schwab wrote: Jeff Law writes: PR rtl-optimization/70024 That's probably a typo. Already fixed. jeff

[gomp-nvptx 4/7] nvptx backend: re-enable line info generation

* config/nvptx/nvptx.c (nvptx_option_override): Remove custom handling of debug info options. --- gcc/ChangeLog.gomp-nvptx | 5 + gcc/config/nvptx/nvptx.c | 9 - 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/gcc/config/nvptx/nvptx.c

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245, take 2)

On Thu, Mar 17, 2016 at 11:07:03PM +1030, Alan Modra wrote: > On Thu, Mar 17, 2016 at 12:16:58PM +0100, Jakub Jelinek wrote: > > the rs6000 backend for whatever strange reason I haven't understood > > really wants pointer equality instead of REGNO comparison (even when the > > modes match), one

Re: [RFA][PATCH][PR tree-optimization/64058] Improve and stabilize sorting of coalesce pairs

On 03/15/2016 08:22 AM, Richard Biener wrote: To work around the narrow API in the comparison function we have to either store additional data in each node or have them available in globals. The former would be horribly wasteful, the latter is just ugly. I choose the latter in the lazy

[Patch] [x86_64]: minor latency changes for znver1.md

Hi Uros, The below patch changes the latency values for fp type load reservations. It passes normal bootstrap and bootstrap with BOOT_CFLAGS="-O2 -g - march=znver1 -mno-clzero -mno-sha " on avx2 target. Also compiled and ran SPEC2006 with -march=znver1 and -Ofast . Ok for trunk? ChangeLog

[PATCH, aarch64] Fix target/70120

With -g, and a code section that ends unaligned, the assembler complains of "unaligned opcodes detected". Except there are no such unaligned opcodes, nor dwarf2 code ranges covering the end of the section, which arguably makes this an assembler bug. However, it's reasonably easy to work around

Re: [PATCH] Change replace_rtx if from is a REG (PR target/70245)

On 03/16/2016 01:22 PM, Jakub Jelinek wrote: So, this is what we've converged to on IRC and passed bootstrap/regtest on x86_64-linux and i686-linux. Is this ok for trunk? The explanation was a bit confusing at first, but I think this looks reasonable. The assert worries me, but triggering it

[PATCH] Fix PR c++/70205 (ICE on valid call to qualified static member function)

adjust_result_of_qualified_name_lookup() may erroneously clobber the BASELINK_BINFO of its DECL argument if the BINFO_TYPE of DECL is an ambiguous base of the qualifying scope that's used to refer to DECL. But as the comment in the function suggests, this base ambiguity is not necessarily a

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. + bitmap_set_bit (seen_insns, INSN_UID (insn)); +

Re: Re: [PATCH][Testsuite] Add --param sra-max-scalarization-size-Ospeed to sra-12.c

On 21/10/15 16:59, Jeff Law wrote: > On 10/21/2015 09:52 AM, Alan Lawrence wrote: >> gcc.dg/tree-ssa/sra-12.c is skipped on a bunch of targets, including >> AArch64, >> because the default max-scalarization-size depends on MOVE_RATIO, and >> on those >> targets thus ends up being too small for SRA

Re: [PATCH, PR70183] Propagate dump flags in pass_manager::register_pass

On Wed, Mar 16, 2016 at 12:26 PM, Tom de Vries wrote: > Hi, > > atm dumpfile vzeroupper is not influenced by the flags in > -fdump-rtl-all-flags. > > The patch fixes this by copying the flags in pass_manager::register_pass. > > OK for stage1 if bootstrap and reg-test

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On Wed, Mar 16, 2016 at 7:33 AM, Jason Merrill wrote: > On 03/16/2016 07:55 AM, H.J. Lu wrote: >> >> On Tue, Mar 15, 2016 at 7:51 PM, Jason Merrill wrote: >>> >>> On 03/15/2016 08:25 PM, Joseph Myers wrote: On Tue, 15 Mar 2016, H.J. Lu wrote:

Re: [PATCH] Fix 70199

On 03/16/2016 02:40 AM, Richard Biener wrote: I like patch 2 more - btw, you need to add has_forced_label_in_static streaming to lto-streamer-{in,out}.c, just look for has_nonlocal_label streaming. Also has_label_address_in_static_1 is now unused and should be removed. I'd already found the

Re: [PATCH] PR c/70281: C FE: fix uninitialized range for __builtin_types_compatible_p

On 03/18/2016 10:25 AM, David Malcolm wrote: PR c/70281 reports another case where Valgrind identified an uninitialized src_range in a c_expr in the C frontend, this time in the parsing of __builtin_types_compatible_p. For gcc 7 I hope to fix this more robustly (via poisoning the values in a

Re: Fix 70278 (LRA split_regs followup patch)

On 03/18/2016 06:25 AM, Bernd Schmidt wrote: This fixes an oversight in my previous patch here. I used biggest_mode in the assumption that if the reg was used in the function, it would be set to something other than VOIDmode, but that fails if we have a multiword access - only the first hard reg

[gomp-nvptx 6/7] nvptx backend: change mul.u32 to mul.lo.u32

Recent testing uncovered that PTX JIT may reject attempts to use 'mul.u32' as a non-widening 32-bit multiply instruction. Use 'mul.lo.u32' to fix 32-bit code generation and conform to the PTX spec better. * config/nvptx/nvptx.c (nvptx_init_unisimt_predicate): Emit 'mul.lo.u32'

Re: Please include ada-hurd.diff upstream (try2)

Sorry for manually creating this mail. Replying to Arnauds message crashes my mail client (evolution). On Wed, 2016-03-16 at 11:53 +0100, Arnaud Charlet wrote: > Yes, you are right, somehow I submitted the old patch. Attached is the > updated one, with a proposed ChangeLog entry included. > >

Re: [PATCH, PR70161] Fix fdump-ipa-all-graph

On 18/03/16 10:23, Tom de Vries wrote: On 15/03/16 12:37, Richard Biener wrote: On Mon, 14 Mar 2016, Tom de Vries wrote: Hi, this patch fixes PR70161, a 4.9/5/6 regression. Currently when using -fdump-ipa-all-graph, the compiler ICEs in execute_function_dump when testing for

[PATCH, i386, AVX-512] Emit vpbroadcastq instead if non-existent vbroadcastsd.

Hello, Intel spec [1] states that there're almost all broadcasting intructions variants available, except for (p. 2-4) vbroadcastsd %xmm, %xmm It is safe to emit vpbroadcastq %xmm, %xmm instead. I was uable to extract a testcase, but if this insn is generated - we'll got asm

Re: [RFA][PR rtl-optimization/70263] Fix creation of new REG_EQUIV notes

On 03/17/2016 12:23 PM, Bernd Schmidt wrote: On 03/17/2016 06:37 PM, Jeff Law wrote: + bitmap seen_insns; + seen_insns = BITMAP_ALLOC (NULL); You could save an allocation here by making this a bitmap_head and using bitmap_initialize. Done. + bitmap_set_bit (seen_insns, INSN_UID

Re: [AArch64] Emit square root using the Newton series

On 03/17/16 09:55, James Greenhalgh wrote: On Wed, Mar 16, 2016 at 02:45:37PM -0500, Evandro Menezes wrote: On 03/08/16 16:08, Evandro Menezes wrote: On 02/16/16 14:56, Evandro Menezes wrote: On 12/08/15 15:35, Evandro Menezes wrote: Emit square root using the Newton series 2015-12-03

Re: [PATCH, PR70161] Fix fdump-ipa-all-graph

On 15/03/16 12:37, Richard Biener wrote: On Mon, 14 Mar 2016, Tom de Vries wrote: Hi, this patch fixes PR70161, a 4.9/5/6 regression. Currently when using -fdump-ipa-all-graph, the compiler ICEs in execute_function_dump when testing for pass->graph_dump_initialized, because pass == NULL.

Re: [PATCH] Use simplify_replace_rtx instead of replace_rtx for DEBUG_INSNs in reload

On Fri, 18 Mar 2016, Jakub Jelinek wrote: > Hi! > > This patch fixes one of the spots that use replace_rtx, as this changes > debug insns, it clearly wants to replace just based on regno, not on pointer > equality, and for debug insns simplification is always desirable too. > The other place in

Re: [C PATCH] Fix up composite_types (PR c/70280)

On Thu, 17 Mar 2016, Jakub Jelinek wrote: > Hi! > > Zdenek reported a compare debug issue, where it is dumping used function > prototypes and there is a difference between -g0 and -g in > -2: static int BIO_vsnprintf (char *, size_t, const char *, struct *, > void, ...); > +2: static int

Patch ping

Hi! I'd like to ping a C++ PR70144 patch: http://gcc.gnu.org/ml/gcc-patches/2016-03/msg00653.html Since then, Zdenek has filed another PR with __builtin_constant_p (foo) where foo is FUNCTION_DECL, and only the above version fixes that, not the other one. Jakub

Re: [PATCH] Fix PR64764

On Wed, Mar 16, 2016 at 9:35 AM, Tom de Vries wrote: > On 16/03/16 17:15, H.J. Lu wrote: >> >> On Wed, Mar 16, 2016 at 9:12 AM, H.J. Lu wrote: > > >>> Any particular reason why this test was changed to DOS format? > > > FWIW, the test was in DOS

[PATCH, aarch64] Fix 70048

This fixes only the regression described in the PR. There was quite a bit of follow-up that points to new work that ought to be done during the gcc7 cycle, but isn't really appropriate now. Tested on aarch64-linux; committed as reviewed in the PR. r~ PR target/70048 *

Re: [PATCH][SPARC] sparc: switch -fasynchronous-unwind-tables on by default.

On 02/29/2016 07:50 AM, Jose E. Marchesi wrote: > The backtrace(3) implementation for sparc contains a simple unwinder > that works well in most cases, but that unwinder is not used if > libgcc_s.so can be dlopened and it provides _Unwind_Backtrace. There's no reason that simple unwinder can't be

[PATCH, PR70183] Propagate dump flags in pass_manager::register_pass

Hi, atm dumpfile vzeroupper is not influenced by the flags in -fdump-rtl-all-flags. The patch fixes this by copying the flags in pass_manager::register_pass. OK for stage1 if bootstrap and reg-test succeeds? Thanks, - Tom Propagate dump flags in pass_manager::register_pass 2016-03-16 Tom

Re: PING^1: [PATCH] Add TYPE_EMPTY_RECORD for C++ empty class

On Wed, Mar 16, 2016 at 9:58 AM, Jason Merrill wrote: > On 03/16/2016 08:38 AM, H.J. Lu wrote: >> >> FAIL: g++.dg/abi/pr60336-1.C scan-assembler jmp[\t >> ]+[^$]*?_Z3xxx9true_type >> FAIL: g++.dg/abi/pr60336-5.C scan-assembler jmp[\t >> ]+[^$]*?_Z3xxx9true_type >> FAIL:

Re: [PATCH V3]PR other/70268: map one directory name (old) to another (new) in __FILE__

On 03/18/2016 03:56 PM, Bernhard Reutner-Fischer wrote: On March 18, 2016 6:16:46 AM GMT+01:00, Hongxu Jia wrote: +/* Perform user-specified mapping of __FILE__ prefixes. Return + the new name corresponding to filename. */ + +const char * +remap_file_filename

Re: [PATHCH] Disable inline asm for in-tree mpfr (PR69134)

On Tue, 5 Jan 2016, Richard Biener wrote: On January 5, 2016 2:20:42 PM GMT+01:00, Bernd Edlinger wrote: On 05.01.2016 13:58, Bernd Schmidt wrote: On 01/05/2016 09:44 AM, Bernd Edlinger wrote: Using asm code is generally not desirable for in-tree mpfr builds.

Re: [PATCH, PR70161] Fix fdump-ipa-all-graph

On Thu, 17 Mar 2016, Tom de Vries wrote: > On 15/03/16 12:37, Richard Biener wrote: > > On Mon, 14 Mar 2016, Tom de Vries wrote: > > > > > Hi, > > > > > > this patch fixes PR70161, a 4.9/5/6 regression. > > > > > > Currently when using -fdump-ipa-all-graph, the compiler ICEs in > > >

Re: [PATCH]PR other/70268: map one directory name (old) to another (new) in __FILE__

On Thu, 17 Mar 2016, Hongxu Jia wrote: > + if (add_file_prefix_map(arg) < 0) Bad formatting (missing space before '('). Likewise elsewhere in this patch. > +@item -ffile-prefix-map=@var{old}=@var{new} > +@opindex ffile-prefix-map > +When parsing __FILE__, __BASE_FILE__ and

[AArch64] Emit division using the Newton series

Emit division using the Newton series 2016-03-17 Evandro Menezes gcc/ * config/aarch64/aarch64-tuning-flags.def (AARCH64_EXTRA_TUNE_APPROX_DIV_{SF,DF}: New tuning macros. * config/aarch64/aarch64-protos.h

Re: C PATCH for c/70093 (ICE with nested-function returning VM type)

Ping. On Wed, Mar 09, 2016 at 04:55:23PM +0100, Marek Polacek wrote: > On Wed, Mar 09, 2016 at 04:31:37PM +0100, Jakub Jelinek wrote: > > No, I meant: > > switch (n) > > { > > struct S x; > > case 1: > > fn (); > > break; > > case 2: > > fn2 (); > >

[hsa branch] Use an obstack instead of multiple alloc pools

Hi, when I started working on expansion to HSAIL almost three years ago, I decided to allocate memory for most of the structures from various alloc-pools for reasons that never materialized and the number of pools later grew to unreasonable numbers. So after an internal discussion, Martin Liska

  1   2   >