Re: Calculate TYPE_CANONICAL only for types that can be accessed in memory

On Fri, May 22, 2015 at 5:00 AM, Jan Hubicka hubi...@ucw.cz wrote: Now we have it spelled out 4 times ... makes sense to create a new macro for it? (though I cannot think of a good name... UNACCESSIBLE_TYPE_P ()?) Yep, actually I already made that version of patch yesterday but then got

Re: Move ABS detection from fold-const.c to match.pd

I forgot to mention I optimistically tried to write something like this: (match (negated_value_for_comparison @0) (negate @0)) (match (negated_value_for_comparison (negate @0)) @0) (match (negated_value_for_comparison (minus @0 @1)) (if (!HONOR_SIGN_DEPENDENT_ROUNDING (type)) (minus @1

Re: [RFC / CFT] PR c++/66192 - Remove TARGET_RELAXED_ORDERING and use load acquires.

Hello! This patch removes the special casing for targets with relaxed memory ordering and handles guard accesses with equivalent atomic load acquire operations. In this process we change the algorithm to load the guard variable with an atomic load that has ACQUIRE semantics. I'm not

Re: Calculate TYPE_CANONICAL only for types that can be accessed in memory

On Sun, May 24, 2015 at 5:49 AM, H.J. Lu hjl.to...@gmail.com wrote: On Fri, May 22, 2015 at 5:00 AM, Jan Hubicka hubi...@ucw.cz wrote: Now we have it spelled out 4 times ... makes sense to create a new macro for it? (though I cannot think of a good name... UNACCESSIBLE_TYPE_P ()?) Yep,

Re: [Patch, fortran] PR66257 [5/6 regression] elemental typebound calls rejected as actual argument

Hi Mikael, Yes, that's fine for 5 and 6. Thanks for the patch. Paul On 23 May 2015 at 23:35, Mikael Morin mikael.mo...@sfr.fr wrote: Hello, For PR 63727, a check was introduced, rejecting procedure pointer components used as actual arguments: foo(obj%proc_comp) but it had the

Simplify -x0 to x0

Hello, I noticed we were only doing this transformation for floats and not for integers, so I took the chance to move it to match.pd. Regtested on ppc64le-redhat-linux. 2015-05-25 Marc Glisse marc.gli...@inria.fr * match.pd (swapped_tcc_comparison): New operator list. (-A

Move ABS detection from fold-const.c to match.pd

Hello, I don't think this pattern is done in the branch. Here I am trying to match what is done in fold-const, but the idea is that we can later add an extra block where we replace (cmp (minus @2 @3) zerop) with (cmp @2 @3), maybe with some adjustments (convert? all over the place), to help

[PATCH v3 2/4] libcc1: set debug compile: Display GCC driver filename

Hi, GDB currently searches for /usr/bin/ARCH-OS-gcc and chooses one but it does not display which one. It cannot, GCC method set_arguments() does not yet know whether 'set debug compile' is enabled or not. Jan include/ChangeLog 2015-05-24 Jan Kratochvil jan.kratoch...@redhat.com *

[PATCH v3 1/4] libcc1: Introduce GCC_FE_VERSION_1

Hi, the libcc1 API change formerly approved for GCC was rejected by GDB, therefore here is a new API + its implementation. Jan include/ChangeLog 2015-05-24 Jan Kratochvil jan.kratoch...@redhat.com * gcc-interface.h (enum gcc_base_api_version): Add GCC_FE_VERSION_1.

[PATCH v3 3/4] libcc1: Add 'set compile-gcc'

Hi, GDB currently searches for /usr/bin/ARCH-OS-gcc and chooses one but one cannot override which one. GDB would provide new option 'set compile-gcc'. Jan include/ChangeLog 2015-05-24 Jan Kratochvil jan.kratoch...@redhat.com * gcc-interface.h (enum gcc_base_api_version): Update

Re: [Patch, fortran] PR66079 - [6 Regression] memory leak with source allocation in internal subprogram

Hi Paul, thanks for the answers. I did not want to be nasty, but was just wondering, if I only I had those problems. Given furthermore, that reviews are sparse lately, I thought it might help, when a glitch was removed before a reviewer put his head into it :-) I am curious though, why we loose

[PATCH v3 4/4] libcc1: 'set debug compile': Display absolute GCC driver filename

Hi, already approved, maybe it could be already checked in outside of the series: https://gcc.gnu.org/ml/gcc-patches/2015-04/msg01298.html With the patches so far after (gdb) set debug compile 1 one would get: searching for compiler matching regex

Teach gimple_canonical_types_compatible_p about incomplete types

Richard, here is next patch of the series. It adds all the logic for defining type equivalence that globs all complete types together in order to make incomplete type equivalent to every complete variant. Effect of recursing on pointers === This is, of course,

Re: [PATCH] Don't combine param and return value copies

On Sat, May 23, 2015 at 08:45:23AM -0500, Segher Boessenkool wrote: Thanks. Did you see improvements around return as well, or mostly / only related to the function start? The rlwinm vs. rldicl change was in dwarf2out.c add_ranges_num for r3 one insn before a blr. I'm sure that was due to not

[PATCH] Fixes combined gcc-binutils builds.

Combined builds has been broken for about 10 months, because some binutils configure.in files were renamed to configure.ac, but gcc's references to them were not updated. There is a corresponding patch submitted to binutils-gdb, which renames its few remaining configure.in files to configure.ac.

GIMPLE syntax highlighting for vim

Hi all! The attached vim script can be used to highlight syntax in GIMPLE dumps making them somewhat easier to read. I would like to add this script to gcc/contrib directory. Is that OK? -- Regards, Mikhail Maltsev Syntax highlighting rules for GIMPLE dump files Copyright (C) 2015 Free

[C++ Patch] Fix 66243, silent cast of scoped enum

This is a pretty obvious patch. We were permitting a scoped enum initializer of an another enum to silently decay to int. That's not right, only unscoped enums have that privilege. committed. nathan 2015-05-24 Nathan Sidwell nat...@acm.org cp/ PR c++/66243 * decl.c (build_enumerator):

Re: [PATCH] Don't combine param and return value copies

On Mon, May 25, 2015 at 10:26:35AM +0930, Alan Modra wrote: looking at gcc/*.o I haven't yet seen any regressions in code quality.) Well that didn't last very long. There are regressions, and just from looking at disassembled object files it would appear to be frame pointer related. So the

[Ping]: [Patch] [AArch64] PR target 66049: fix add/extend gcc test suite failures

Ping! -Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches-ow...@gcc.gnu.org] On Behalf Of Kumar, Venkataramanan Sent: Tuesday, May 19, 2015 9:07 PM To: James Greenhalgh (james.greenha...@arm.com); gcc-patches@gcc.gnu.org Cc: Kyrill Tkachov

RE: [PATCH] Fix PR66168: ICE due to incorrect invariant register info

From: Jeff Law [mailto:l...@redhat.com] Sent: Saturday, May 23, 2015 6:54 AM - if (!can_move_invariant_reg (loop, inv, reg)) + if (!can_move_invariant_reg (loop, inv, dest)) Won't this run into into the same problem if DEST is a SUBREG? One of the very first test in

RE: [RFC: Patch, PR 60158] gcc/varasm.c : Pass actual alignment value to output_constant_pool_2

-Original Message- From: Richard Biener [mailto:rguent...@suse.de] Sent: Monday, May 18, 2015 1:06 PM To: Jeff Law Cc: Dharmakan Rohit-B30502; gcc-patches@gcc.gnu.org; Jakub Jelinek; Alan Modra; David Edelsohn; Wienskoski Edmar-RA8797 Subject: Re: [RFC: Patch, PR 60158] gcc/varasm.c

Re: [PATCH GCC]Improve how we handle overflow for type conversion in scev/ivopts, part I

On Fri, May 22, 2015 at 7:45 PM, Richard Biener richard.guent...@gmail.com wrote: On Wed, May 20, 2015 at 11:41 AM, Bin Cheng bin.ch...@arm.com wrote: Hi, As we know, GCC is too conservative when checking overflow behavior in SCEV and loop related optimizers. Result is some variable can't be

Re: [C frontend] Fix construction of TYPE_STUB_DECL

Hi, this patch implements the idea. It uses anon for anonymous namespace types and updates ipa-devirt to rely on it (and also sanity check that). The patch has bootstrapped/regtested powerpc64le-linux, will commit it tomorrow if there are no complains to unbreak the Ada LTO bootstrap. Honza

Re: [Patch, fortran] PR66079 - [6 Regression] memory leak with source allocation in internal subprogram

Dear Andre, I'll put both points right. Thanks for pointing them out. Cheers Paul On 23 May 2015 at 19:52, Andre Vehreschild ve...@gmx.de wrote: Hi Paul, does this patch apply to current trunk cleanly? I get an issue with the last hunk, because all of the prerequisites are gone since

New Swedish PO file for 'gcc' (version 5.1.0)

Hello, gentle maintainer. This is a message from the Translation Project robot. A revised PO file for textual domain 'gcc' has been submitted by the Swedish team of translators. The file is available at: http://translationproject.org/latest/gcc/sv.po (This file, 'gcc-5.1.0.sv.po', has