Turn DECL_SECTION_NAME into string

2014-06-11 Thread Jan Hubicka
Hi, this lenghtly patch makes the legwork to put section names out of tree representation. Originally they were STRING_CST. I ended up implementing on-side reference counted string voclabulary that is done in bit baroque way to be GGC and PCH safe (uff). The memory savings on Firefox are about 60

Re: [PATCH, PR52252] Alternative way of vectorization for load groups of size 2 and 3.

2014-06-11 Thread Evgeny Stupachenko
Testing finished. No new regressions. Is the following patch ok? 2014-06-11 Evgeny Stupachenko * config/i386/i386.c (ix86_reassociation_width): Add alternative for vector case. * config/i386/i386.h (TARGET_VECTOR_PARALLEL_EXECUTION): New. * config/i386/x86-tune.

Re: [Patch ARM/testsuite 00/22] Neon intrinsics executable tests

2014-06-11 Thread Mike Stump
On Jun 10, 2014, at 3:03 PM, Ramana Radhakrishnan wrote: > I am a bit ambivalent between getting folks to add scan-assembler > tests here and worrying between this and getting the behaviour > correct. Additionally if you add the complexity of scanning for > aarch64 as well this starts getting mes

Re: [C++ Patch] PR 19200 follow up

2014-06-11 Thread Jason Merrill
OK. Jason

Re: [patch] libstdc++/29988 Rb_Tree reuse allocated nodes

2014-06-11 Thread Jonathan Wakely
On 11/06/14 21:56 +0200, François Dumont wrote: For the testsuite allocator I though that for an internal allocator used in our tests it was ok. But alright, I will make it better and compatible with SimpleAllocator. I suppose it's OK for now, and we can change it later if we need it to wo

Re: [PING][PATCH, trunk, 4.9, 4.8] Fix PR57653, filename information discarded when using -imacros

2014-06-11 Thread Peter Bergner
On Wed, 2014-06-11 at 23:07 +, Joseph S. Myers wrote: > On Wed, 11 Jun 2014, Peter Bergner wrote: > > > I'd like to ping the following patch that fixes PR57653. This did > > bootstrap and regtest with no regressions on powerpc64-linux. > > > > https://gcc.gnu.org/ml/gcc-patches/2014-04/m

Re: [PING][PATCH, trunk, 4.9, 4.8] Fix PR57653, filename information discarded when using -imacros

2014-06-11 Thread Joseph S. Myers
On Wed, 11 Jun 2014, Peter Bergner wrote: > I'd like to ping the following patch that fixes PR57653. This did > bootstrap and regtest with no regressions on powerpc64-linux. > > https://gcc.gnu.org/ml/gcc-patches/2014-04/msg01571.html > > Is this ok for trunk, 4.9 and 4.8? I think the code

[PATCH] PR rtl-optimization/61047

2014-06-11 Thread Bernd Edlinger
Hi, this patch fixes PR 61047, where rtx_addr_can_trap_p_1 incorrectly assumes, that all SP-relative offsets can be safely read witout causing any traps. And therefore such references are safe to be moved out of an if-block. This patch tries to get safe lower and upper bounds where accesses are a

Re: [PATCH, PR 61186]

2014-06-11 Thread Jan Hubicka
> Hi, > > in PR 61186 valgrind reports use of uninitialized value. The reason > is that walk_polymorphic_call_targets uses cache_token before it > checks return value of possible_polymorphic_call_targets which however > does not produce a cache token if it fails early. This is not the > only pla

Re: [PATCH, PR61446] Fix mode for register copy in REE pass

2014-06-11 Thread Ilya Enkovich
On 11 Jun 22:47, Uros Bizjak wrote: > > This should read: > > /* { dg-do compile { target { ia32 } } } */ > /* { dg-options "-O2 -march=corei7 -mfpmath=387" } */ > > The x86 part is OK with this change. > > Uros. Thanks for comment! Here is a fixed version. Ilya -- gcc/ 2014-06-11 Ilya Enk

fix math wrt volatile-bitfields vs C++ model

2014-06-11 Thread DJ Delorie
If the combined bitfields are exactly the size of the mode, the logic for detecting range overflow is flawed - it calculates an ending "position" that's the position of the first bit in the next field. In the case of "short" for example, you get "16 > 15" without this patch (comparing size to pos

Re: [PING*2][PATCH] Extend mode-switching to support toggle (1/2)

2014-06-11 Thread Oleg Endo
On 11 Jun 2014, at 14:40, Christian Bruel wrote: > > On 06/10/2014 04:03 PM, Joern Rennecke wrote: >> On 13 May 2014 22:41, Oleg Endo wrote: >> >>> Right. I was thinking to add FPSCR.SZ mode switching to SH, in order to >>> do float vector moves. For that SZ and PR need to be switched both

[PING][PATCH, trunk, 4.9, 4.8] Fix PR57653, filename information discarded when using -imacros

2014-06-11 Thread Peter Bergner
I'd like to ping the following patch that fixes PR57653. This did bootstrap and regtest with no regressions on powerpc64-linux. https://gcc.gnu.org/ml/gcc-patches/2014-04/msg01571.html Is this ok for trunk, 4.9 and 4.8? Peter

[C++ Patch] PR 19200 follow up

2014-06-11 Thread Paolo Carlini
Hi, just reminded the hard way that, for friend *template* declarations, cp_parser_init_declarator (vs cp_parser_member_declaration) has to pass the actual friend_p information. I'm finishing testing the below. Thanks, Paolo. // /cp 2014-06-11 Paolo Carlini PR

[rx] adjust alignments for RX100/200 families

2014-06-11 Thread DJ Delorie
The RX100/200 families have 4 byte cache lines, vs 8 bytes for the RX600 family. Applied. * config/rx/rx.h (FUNCTION_BOUNDARY): Adjust for RX100/200 4-byte cache lines. * config/rx/rx.c (rx_option_override): Likewise. (rx_align_for_label): Likewise. Index: config

Re: [PATCH, rs6000][trunk, 4.9, 4.8] Fix PR target/61415, long double 128 issues

2014-06-11 Thread Peter Bergner
On Fri, 2014-06-06 at 11:37 -0400, David Edelsohn wrote: > On Thu, Jun 5, 2014 at 3:57 PM, Peter Bergner wrote: > > Is this also ok for the FSF 4.9 and FSF 4.8 branches? Without the gcc/ > > changes, we hit an ICE whenever we call __builtin_pack_longdouble and > > __builtin_unpack_longdouble when

Re: [PATCH, PR61446] Fix mode for register copy in REE pass

2014-06-11 Thread Uros Bizjak
On Wed, Jun 11, 2014 at 6:11 PM, Ilya Enkovich wrote: > On 11 Jun 14:59, Uros Bizjak wrote: >> On Tue, Jun 10, 2014 at 3:45 PM, Dominique Dhumieres >> wrote: >> >> This patch fixes PR61446. ... >> > >> > Confirmed, it also allows to bootstrap Core* targets. >> > Could it be reviewed and committe

Re: [PATCH] PR 61642

2014-06-11 Thread Martin Liška
On 06/11/2014 02:01 PM, Richard Biener wrote: On Wed, Jun 11, 2014 at 1:58 PM, Martin Liška wrote: Hello, I send patch for PR61642. 61462? Hello, that's bogus, I am not a visionary that creates a patch for a future bug :) Thanks, Martin Changelog: 2014-06-11 Martin Liska

[PATCH, PR 61186]

2014-06-11 Thread Martin Jambor
Hi, in PR 61186 valgrind reports use of uninitialized value. The reason is that walk_polymorphic_call_targets uses cache_token before it checks return value of possible_polymorphic_call_targets which however does not produce a cache token if it fails early. This is not the only place where we ma

Re: [patch] libstdc++/29988 Rb_Tree reuse allocated nodes

2014-06-11 Thread François Dumont
For the testsuite allocator I though that for an internal allocator used in our tests it was ok. But alright, I will make it better and compatible with SimpleAllocator. On 11/06/2014 14:02, Jonathan Wakely wrote: Index: include/bits/stl_tree.h =

Re: [PATCH] Trust TREE_ADDRESSABLE

2014-06-11 Thread Jan Hubicka
> > Why not just make the change to may_be_aliased in LTO mode, with a comment > saying that TREE_PUBLIC and DECL_EXTERNAL aren't fully correct any longer? They are fully correct to the partition being compiled BTW. Honza

Re: [PATCH] Trust TREE_ADDRESSABLE

2014-06-11 Thread Jan Hubicka
> > Sure. Still currently TREE_ADDRESSABLE on TREE_PUBLIC/DECL_EXTERNAL > > VAR_DECLs carries no useful information, so I consider the bit unused. > > I guess that's true for the middle-end in non-LTO mode at this point. > But then the new approach shouldn't be make correctness depend on its sett

[rx] further optimize alignment with -Os

2014-06-11 Thread DJ Delorie
Minor tweak to space optimization. Committed. * config/rx/rx.c (rx_max_skip_for_label): Don't skip anything if -Os. Index: config/rx/rx.c === --- config/rx/rx.c (revision 211479) +++ config/rx/rx.c (working copy)

Re: [PATCH][RX] Patch to correct the functionality of compiler option -falign-labels=n

2014-06-11 Thread DJ Delorie
> Please review the patch and let me know if there should be any modifications > in it? Have you checked the other alignment macros to see if they need to be fixed too? * config/rx/rx.h (Corrected macro LABEL_ALIGN). This should be : * config/rx/rx.h (LABEL_ALIGN): It's alway

RFC: PR target/61296: Excessive alignment in ix86_data_alignment

2014-06-11 Thread H.J. Lu
ix86_data_alignment returns 32 bytes or the cache line size when getting the optimized alignment for a static aggregate variable bigger than 32 bytes or the cache line size. The increased alignment only applies to the static variable in the same compilation unit while not optimizing for space. If

[wwwdocs, patch, committed] Changed some http://gcc.gnu.org to https

2014-06-11 Thread Tobias Burnus
Dear all, given that the webserver now redirects http accesses to https, it makes sense that all references already use https. Committed as obvious for the most important files. Tobias Index: style.mhtml === RCS file: /cvs/gcc/ww

Re: [GOOGLE] Emit linkage_name when built with -gmlt and for abstract decls

2014-06-11 Thread Dehao Chen
On Wed, Jun 11, 2014 at 10:38 AM, Cary Coutant wrote: >> This will increase c++ g1/g2 binary size a little. For all spec >> cint2006 benchmarks, the binary size change is shown below. >> >> 400 0.00% 0.00% 0.00% 0.00% >> 401 0.00% 0.00% 0.00% 0.00% >> 403 0.00% 0.00% 0.00% 0.00% >> 429 0.00% 0.00%

Re: [GOOGLE] Emit linkage_name when built with -gmlt and for abstract decls

2014-06-11 Thread Cary Coutant
> This will increase c++ g1/g2 binary size a little. For all spec > cint2006 benchmarks, the binary size change is shown below. > > 400 0.00% 0.00% 0.00% 0.00% > 401 0.00% 0.00% 0.00% 0.00% > 403 0.00% 0.00% 0.00% 0.00% > 429 0.00% 0.00% 0.00% 0.00% > 445 0.00% 0.00% 0.00% 0.00% > 456 0.00% 0.00% 0

Re: [GOOGLE] Emit linkage_name when built with -gmlt and for abstract decls

2014-06-11 Thread Dehao Chen
ping... Dehao On Fri, May 30, 2014 at 4:13 PM, Dehao Chen wrote: > This will increase c++ g1/g2 binary size a little. For all spec > cint2006 benchmarks, the binary size change is shown below. > > 400 0.00% 0.00% 0.00% 0.00% > 401 0.00% 0.00% 0.00% 0.00% > 403 0.00% 0.00% 0.00% 0.00% > 429 0.00

[c++-concepts] Terse notation

2014-06-11 Thread Andrew Sutton
Sorry if you receive multiple versions of this. The original message bounced (formatting). Attached is a patch that greatly improves terse notation support for generic functions and the use of concept names in constrained-type-specifiers. There are actually 2 patches. The second includes unit test

Re: [PATCH] GCC/MMIX: Remove orphan mmix_asm_output_source_line prototype

2014-06-11 Thread Maciej W. Rozycki
On Wed, 11 Jun 2014, Hans-Peter Nilsson wrote: > > I've noticed mmix_asm_output_source_line is declared, but nowhere > > defined. OK to remove the prototype? > > Sure; in fact, obvious. Applied, thanks. Maciej

Re: [ARM] Fix build failure due to movsi_compare0 (PR 61430)

2014-06-11 Thread Chung-Lin Tang
On 2014/6/11 下午 06:32, James Greenhalgh wrote: > > Hi, > > A recent change somewhere exposed a latent bug between LRA and the definition > of the movsi_compare0 pattern. > > This pattern ties the source and destination register of a set together > a (match_dup) and register constraints: > >

Re: ipa-visibility TLC 2/n

2014-06-11 Thread Jan Hubicka
> > Unfortunately, AIX isn't the only target massively affected by your > recent patches. This all started with r210597 > > 2014-05-17 Jan Hubicka > > * tree-pass.h (make_pass_ipa_comdats): New pass. > * timevar.def (TV_IPA_COMDATS): New timevar. > * passes.def (pass_ip

Re: [PATCH 8/8] Add a common .md file and define standard constraints there

2014-06-11 Thread Richard Sandiford
Thanks for the reviews. Jeff Law writes: > Does the comment before indep_constraints in genoutput need updating? > The constraints in common.md are machine independent, but aren't listed > in indep_constraints in genoutput.c Yeah, good catch. I changed it to: /* All machine-independent const

Re: [C++ Path/RFC] PR 60265

2014-06-11 Thread Jason Merrill
On 06/11/2014 12:49 PM, Paolo Carlini wrote: .. Core/1742 seems relevant to the difference scoped/unscoped. It's still open, though. And priority 2, so unlikely to be resolved any time soon. The patch is OK. Jason

Re: [C++ Path/RFC] PR 60265

2014-06-11 Thread Paolo Carlini
.. Core/1742 seems relevant to the difference scoped/unscoped. It's still open, though. Thanks, Paolo.

[C++ Path/RFC] PR 60265

2014-06-11 Thread Paolo Carlini
Hi, this issue is probably easy to resolve, one way or another, but I find it rather curious: as the submitted testcase explains, we reject the trivial: enum E { V }; using E::V; because "'E' is not a namespace", but I don't see anything in C++11 explicitly outlawing enums. In fact, curren

Re: [patch,avr] Fix ICE PR61443

2014-06-11 Thread Denis Chertykov
2014-06-11 13:05 GMT+04:00 Georg-Johann Lay : > This is a fix for unrecognizable insn ICE where the push expander generated > (subreg(mem)) for non-generic address-space locations. > > Instead of loading byte-wise and legitimizing such expressions, it's easier > and results in better code when we l

Re: [patch] don't link shared RTPs with libc_internal.a on VxWorks

2014-06-11 Thread Olivier Hainque
On Jun 11, 2014, at 17:49 , Olivier Hainque wrote: > * config/vxworks.h (VXWORKS_LIBGCC_SPEC): Don't link shared RTPs with > libc_internal. Oops. I think I sent a bogus version of the patch, as the system libc.so embeds libgcc as well. Let me revisit. Olivier

Re: [PATCH, PR61446] Fix mode for register copy in REE pass

2014-06-11 Thread Ilya Enkovich
On 11 Jun 14:59, Uros Bizjak wrote: > On Tue, Jun 10, 2014 at 3:45 PM, Dominique Dhumieres > wrote: > >> This patch fixes PR61446. ... > > > > Confirmed, it also allows to bootstrap Core* targets. > > Could it be reviewed and committed ASAP? > > > 2014-06-09 Ilya Enkovich > > > > PR 61446 > >

[PATCH, Pointer Bounds Checker 35/x] Fix object size emitted for structures with flexible arrays

2014-06-11 Thread Ilya Enkovich
Hi, This patch fixes problem with size emitted for static structures with flexible array. I found a couple of trackers in guzilla for this problem but all of them are marked as fixed and problem still exists. For a simple testcase struct S { int a; int b[0]; } s = { 1, { 0, 0} }; current tru

[patch] don't link shared RTPs with libc_internal.a on VxWorks

2014-06-11 Thread Olivier Hainque
Hello, For VxWorks RTPs, the config/vxworks.h has: /* For VxWorks, the system provides libc_internal.a. This is a superset of libgcc.a; we want to use it. Make sure not to dynamically export any of its symbols, though. Always look for libgcc.a first so that we get the latest versi

Re: libgo patch committed: Merge from revision 18783 of master

2014-06-11 Thread Rainer Orth
Ian Lance Taylor writes: > On Wed, Jun 11, 2014 at 6:57 AM, Rainer Orth > wrote: >> >> Ok, I'll have a look unless some other solution can be found. > > There are other, less efficient, ways that this could be compiled, but > a common symbol seems like the right solution. > > Basically, every ty

[patch] improve sloc assignment on bind_expr entry/exit code

2014-06-11 Thread Olivier Hainque
Hello, For blocks requiring it, the gimplifier generates stack pointer save/restore operations on entry/exit, per: gimplify_bind_expr (...) if (gimplify_ctxp->save_stack) { gimple stack_restore; /* Save stack on entry and restore it on exit. Add a try_finally block

Re: [C++ Patch] PR 19200

2014-06-11 Thread Jason Merrill
OK. Jason

Re: libgo patch committed: Merge from revision 18783 of master

2014-06-11 Thread Ian Lance Taylor
On Wed, Jun 11, 2014 at 6:57 AM, Rainer Orth wrote: > > Ok, I'll have a look unless some other solution can be found. There are other, less efficient, ways that this could be compiled, but a common symbol seems like the right solution. Basically, every type descriptor now points to a zero value

Re: ipa-visibility TLC 2/n

2014-06-11 Thread Rainer Orth
Jan Hubicka writes: >> Honza, >> >> I am not sure that the problem is caused only by aliases and thunks. >> The large increase in AIX linker warnings about branches not followed >> by nop also worry me. >> >> Your patch was about visibility. How does the more aggressive > > ipa-visibility is a

[PATCH][Ping] Add patch for debugging compiler ICEs

2014-06-11 Thread Maxim Ostapenko
Ping. Original Message Subject:[PATCH] Add patch for debugging compiler ICEs Date: Mon, 02 Jun 2014 19:21:14 +0400 From: Maxim Ostapenko To: GCC Patches CC: Yury Gribov , Slava Garbuzov , Jakub Jelinek , tsaund...@mozilla.com, chefm...@gmail.com Hi, A

[PATCH, i386, Pointer Bounds Checker 34/x] Vararg functions support

2014-06-11 Thread Ilya Enkovich
Hi, This patch introduces initialization of incoming bounds for vararg function on i386 target. Bootstrapped and tested on linux-x86_64. Thanks, Ilya -- gcc/ 2014-06-11 Ilya Enkovich * config/i386/i386.c (ix86_setup_incoming_varargs): New. (ix86_va_start): Initialize bounds

[PATCH, i386, Pointer Bounds Checker 31/x] Pointer Bounds Checker builtins for i386 target

2014-06-11 Thread Ilya Enkovich
Hi, This patch adds i386 target builtins for Pointer Bounds Checker. Bootstrapped and tested on linux-x86_64. Thanks, Ilya -- gcc/ 2014-06-11 Ilya Enkovich * config/i386/i386-builtin-types.def (BND): New. (ULONG): New. (BND_FTYPE_PCVOID_ULONG): New. (VOID_FTY

Re: libgo patch committed: Merge from revision 18783 of master

2014-06-11 Thread Rainer Orth
Ian Lance Taylor writes: > On Wed, Jun 11, 2014 at 5:01 AM, Rainer Orth > wrote: >> Ian Lance Taylor writes: >> >>> On Fri, Jun 6, 2014 at 2:12 AM, Rainer Orth >>> wrote: Ian Lance Taylor writes: > I have committed a patch to libgo to merge from revision > 18783:00cce3a34d7e

[PATCH, i386, Pointer Bounds Checker 33/x] MPX ABI

2014-06-11 Thread Ilya Enkovich
Hi, This patch adds MPX ABI support for i386 target. Bootstrapped and tested on linux-x86_64. Thanks, Ilya -- gcc/ 2014-06-11 Ilya Enkovich * config/i386/i386.c (ix86_option_override_internal): Do not support x32 with MPX. is not available. (init_cumulative_a

[PATCH, i386, Pointer Bounds Checker 32/x] Pointer Bounds Checker hooks for i386 target

2014-06-11 Thread Ilya Enkovich
Hi, This patch adds i386 target hooks for Pointer Bounds Checker. Bootstrapped and tested on linux-x86_64. Thanks, Ilya -- gcc/ 2014-06-11 Ilya Enkovich * config/i386/i386.c: Include tree-iterator.h. (ix86_function_value_bounds): New. (ix86_builtin_mpx_function): New

Re: [PATCH, Pointer Bounds Checker 28/x] IPA CP

2014-06-11 Thread Ilya Enkovich
On 11 Jun 15:07, Ilya Enkovich wrote: > 2014-06-11 13:45 GMT+04:00 Martin Jambor : > > Hi, > > > > On Wed, Jun 11, 2014 at 12:24:57PM +0400, Ilya Enkovich wrote: > >> Hi, > >> > >> This patch fixes IPA CP pass to handle instrumented code correctly. > >> > >> Bootstrapped and tested on linux-x86_64.

Re: [PATCH, PR61446] Fix mode for register copy in REE pass

2014-06-11 Thread Dominique Dhumieres
> Yes, these are bootstraps with non-default configurations. Core2 is the default for darwin and bootstrap is also broken for it without the patch. Dominique

Re: [Patch ARM/testsuite 00/22] Neon intrinsics executable tests

2014-06-11 Thread Christophe Lyon
On 11 June 2014 00:03, Ramana Radhakrishnan wrote: > On Thu, Jun 5, 2014 at 11:04 PM, Christophe Lyon > wrote: >> This is patch series is a more complete version of the patch I sent >> some time ago: >> https://gcc.gnu.org/ml/gcc-patches/2013-10/msg00624.html >> >> I have created a series of patc

Re: [Patch ARM/testsuite 00/22] Neon intrinsics executable tests

2014-06-11 Thread Christophe Lyon
On 6 June 2014 22:15, Christophe Lyon wrote: > On 6 June 2014 17:57, Ramana Radhakrishnan > wrote: >> On 06/06/14 15:40, Christophe Lyon wrote: >>> >>> On 6 June 2014 01:32, Joseph S. Myers wrote: Have these been tested for both big and little endian (especially for tests where m

Re: libgo patch committed: Merge from revision 18783 of master

2014-06-11 Thread Ian Lance Taylor
On Wed, Jun 11, 2014 at 5:01 AM, Rainer Orth wrote: > Ian Lance Taylor writes: > >> On Fri, Jun 6, 2014 at 2:12 AM, Rainer Orth >> wrote: >>> Ian Lance Taylor writes: >>> I have committed a patch to libgo to merge from revision 18783:00cce3a34d7e of the master library. This revision

Re: [PATCH, PR61446] Fix mode for register copy in REE pass

2014-06-11 Thread Uros Bizjak
On Wed, Jun 11, 2014 at 3:19 PM, Dominique Dhumieres wrote: >> (I am not RTL reviewer, so I can't approve the patch). > > Is https://gcc.gnu.org/ml/gcc-regression/2014-06/ accepatble? Yes, these are bootstraps with non-default configurations. Uros.

Re: [PATCH, PR61446] Fix mode for register copy in REE pass

2014-06-11 Thread Dominique Dhumieres
> (I am not RTL reviewer, so I can't approve the patch). Is https://gcc.gnu.org/ml/gcc-regression/2014-06/ accepatble? Dominique

Re: [PATCH, PR61446] Fix mode for register copy in REE pass

2014-06-11 Thread Uros Bizjak
On Tue, Jun 10, 2014 at 3:45 PM, Dominique Dhumieres wrote: >> This patch fixes PR61446. ... > > Confirmed, it also allows to bootstrap Core* targets. > Could it be reviewed and committed ASAP? > 2014-06-09 Ilya Enkovich > > PR 61446 > * ree.c (find_and_remove_re): Narrow mode for register cop

Re: [PATCH 2/9] Add comments

2014-06-11 Thread Richard Earnshaw
On 11/06/14 11:19, Charles Baylis wrote: > 2014-05-22 Charles Baylis > > * config/arm/bpabi.S (__aeabi_uldivmod, __aeabi_ldivmod): Add comment > describing register usage on function entry and exit. OK. R. > --- > libgcc/config/arm/bpabi.S | 16 > 1 file changed

Re: [PATCH 1/9] Whitespace

2014-06-11 Thread Richard Earnshaw
On 11/06/14 11:19, Charles Baylis wrote: > 2014-05-22 Charles Baylis > > * config/arm/bpabi.S (__aeabi_uldivmod): Fix whitespace. > (__aeabi_ldivmod): Fix whitespace. This is OK, but please wait until the others are ready to go in. R. > --- > libgcc/config/arm/bpabi.S | 36 +

Re: [PATCH] Support asan-instrumentation-with-call-threshold in GCC (second try)

2014-06-11 Thread Yury Gribov
On 06/11/2014 01:31 PM, Jakub Jelinek wrote: The plan (we had already for 4.9, but didn't get to that yet) is in the end not to lower the checks in asan pass that much, and lower it in sanopt pass later on after performing some inter-bb optimizations. ... The reason for the plan is that it will b

[PATCH][RX] Patch to correct the functionality of compiler option -falign-labels=n

2014-06-11 Thread Sandeep Kumar Singh
Hi, The RX compiler align wrong labels with option falign-labels=n, where n is number of bytes. The relationship between -falign-labels and expected .balign output supposed to be: -falign-labels=4 --> .balign 4 -falign-labels=8 --> .balign 8 -falign-labels=16 --> .balign 16 However, listed pse

Re: [PING*2][PATCH] Extend mode-switching to support toggle (1/2)

2014-06-11 Thread Christian Bruel
On 06/10/2014 04:03 PM, Joern Rennecke wrote: > On 13 May 2014 22:41, Oleg Endo wrote: > >> Right. I was thinking to add FPSCR.SZ mode switching to SH, in order to >> do float vector moves. For that SZ and PR need to be switched both at >> the same time (only SH4A has both, fpchg and fschg). S

[Ada] Fix handling of pragma/aspect Independent[_Components]

2014-06-11 Thread Arnaud Charlet
This fixes several errors in the handling of the pragmas Independent and Independent_Components. The implementation now matches the RM definition 100%. The following compiles without errors: 1. package Independ is 2.type A1 is array (1 .. 10) of Integer; 3.pragma Independent

[Ada] Consistent processing of preelaborated units across language versions

2014-06-11 Thread Arnaud Charlet
The processing of pragma Preelaborate_05 might cause inconsistent compiler behaviour when a given unit having the pragma appears in the dependencies of both an Ada 95 and and Ada 2005 unit in the same closure. This is addressed by making runtime units Preelaborate in all cases. Tested on x86_64-pc

[Ada] Better handling of variant records with No_Implicit_Conditionals

2014-06-11 Thread Arnaud Charlet
Previously, an attempt to declare a variant record type was rejected if restriction No_Implicit_Conditionals was active, since the resulting generated equality and initialization routines contained implicit tests. Now such declarations are allowed, but these routines are not generated if the restri

[Ada] Error not detected in illegal selected component

2014-06-11 Thread Arnaud Charlet
This patch corrects an error in the resolution of selected components when the prefix is overloaded and none of the interpretations matches the context. Compiling resolve_func_deref_comp.adb must yield: resolve_func_deref_comp.adb:14:18: no interpretation matches type access to "T" defined

Ping x 6: [PATCH] [libgomp] make it possible to use OMP on both sides of a fork

2014-06-11 Thread Nathaniel Smith
Hi all, Ping. Again, this patch fixes a limitation in GOMP which makes it impossible for programmers who care about gcc compatibility to safely use OpenMP in libraries; GOMP is the last OpenMP implementation with this limitation. -n On Wed, May 14, 2014 at 3:47 PM, Nathaniel Smith wrote: > Hi a

[Ada] gnat link and shared libraries

2014-06-11 Thread Arnaud Charlet
When "gnat link" is invoked and there are shared libraries, the link may be incorrect on some platforms, such as Windows. This is fixed by this patch. Tested on x86_64-pc-linux-gnu, committed on trunk 2014-06-11 Vincent Celier * gnatcmd.adb (Process_Link): Do not invoke gnatlink with

[Ada] Analyze contracts of subprogram body stubs

2014-06-11 Thread Arnaud Charlet
This patch ensures that contract of subprogram body stubs are analyzed in timely fashion. -- Source -- -- pack.ads package Pack with SPARK_Mode, Abstract_State => State, Initializes=> (Var_1, State) is Var_1 : Integer := 0; procedure Double

Re: [patch] libstdc++/29988 Rb_Tree reuse allocated nodes

2014-06-11 Thread Jonathan Wakely
Index: include/bits/stl_tree.h === --- include/bits/stl_tree.h (revision 211388) +++ include/bits/stl_tree.h (working copy) @@ -330,6 +330,111 @@ const _Rb_tree_const_iterator<_Val>& __y) _GLIBCXX_NOEXCEPT {

Re: [PATCH] PR 61642

2014-06-11 Thread Richard Biener
On Wed, Jun 11, 2014 at 1:58 PM, Martin Liška wrote: > Hello, >I send patch for PR61642. 61462? > Changelog: > 2014-06-11 Martin Liska > PR ipa/61462 > * ipa-prop.c (ipa_make_edge_direct_to_target): Check that gimple call > statement is reachable. > > Bootstrapped and regteste

Re: libgo patch committed: Merge from revision 18783 of master

2014-06-11 Thread Rainer Orth
Ian Lance Taylor writes: > On Fri, Jun 6, 2014 at 2:12 AM, Rainer Orth > wrote: >> Ian Lance Taylor writes: >> >>> I have committed a patch to libgo to merge from revision >>> 18783:00cce3a34d7e of the master library. This revision was committed >>> January 7. I picked this revision to merge

Re: [PING*2][PATCH] Extend mode-switching to support toggle (1/2)

2014-06-11 Thread Christian Bruel
On 06/11/2014 06:17 AM, Joern Rennecke wrote: > >>> Joern, is this new target macro interface OK with you ? > Yes, this interface should allow me to do switches between rounding > and truncating > floating-point modes with an add/subtract immediate. > > However, the implentation, as posted, doesn'

[PATCH] PR 61642

2014-06-11 Thread Martin Liška
Hello, I send patch for PR61642. Changelog: 2014-06-11 Martin Liska * ipa-prop.c (ipa_make_edge_direct_to_target): Check that gimple call statement is reachable. Bootstrapped and regtested on x86_64-unknown-linux-gnu. Ready for trunk? Martin diff --git a/gcc/ipa-prop.c b/gcc/ipa

Re: [PATCH][libsanitizer] Cherry-pick r210012

2014-06-11 Thread Paolo Carlini
On 06/11/2014 01:33 PM, Richard Biener wrote: Committed. Thanks! Paolo,

[PATCH][libsanitizer] Cherry-pick r210012

2014-06-11 Thread Richard Biener
Committed. Richard. Index: libsanitizer/ChangeLog === --- libsanitizer/ChangeLog (revision 211449) +++ libsanitizer/ChangeLog (working copy) @@ -1,3 +1,7 @@ +2014-06-11 Richard Biener + + * asan/asan_linux.cc: Che

Re: [PATCH] Trust TREE_ADDRESSABLE

2014-06-11 Thread Eric Botcazou
> Sure. Still currently TREE_ADDRESSABLE on TREE_PUBLIC/DECL_EXTERNAL > VAR_DECLs carries no useful information, so I consider the bit unused. I guess that's true for the middle-end in non-LTO mode at this point. But then the new approach shouldn't be make correctness depend on its setting in th

RE: RFA: Make LRA temporarily eliminate addresses before testing constraints

2014-06-11 Thread Robert Suchanek
Hi Richard, >> Robert: you also had an LRA change, but is it still needed after this one? >> If so, could you repost it and explain the case it handles? For just turning the LRA for the MIPS backend is not needed but we have issues with the code size for MIPS16. LRA inserted a lot of reloads and

Re: [PATCH] [ARM] [RFC] Fix longstanding push_minipool_fix ICE (PR49423, lp1296601)

2014-06-11 Thread Charles Baylis
Ping? On 6 May 2014 17:05, Charles Baylis wrote: > Ping? > > At this stage looking for general feedback on whether the define_split > approach in this patch is appropriate. If it is, I'll do a clean patch > for full review. > > Archive link: http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00078.html

[match-and-simplify] Merge from trunk r211444

2014-06-11 Thread Richard Biener
Bootstrapped on x86_64-unknown-linux-gnu. Richard. 2014-06-11 Richard Biener Merge from trunk r208477 through r211444.

Re: [PATCH, Pointer Bounds Checker 28/x] IPA CP

2014-06-11 Thread Ilya Enkovich
2014-06-11 13:45 GMT+04:00 Martin Jambor : > Hi, > > On Wed, Jun 11, 2014 at 12:24:57PM +0400, Ilya Enkovich wrote: >> Hi, >> >> This patch fixes IPA CP pass to handle instrumented code correctly. >> >> Bootstrapped and tested on linux-x86_64. >> >> Thanks, >> Ilya >> -- >> gcc/ >> >> 2014-06-11 I

Re: [PATCH] Delete temporary string within demangler even in failure cases.

2014-06-11 Thread Pedro Alves
On 06/10/2014 07:08 PM, Andrew Burgess wrote: > I don't have gcc write permissions, would you (or any other interested > maintainer) mind committing this for me please. Done, and merged to binutils-gdb git repo. Thanks, -- Pedro Alves

[Ada] Cleanup handling of info and warning messages

2014-06-11 Thread Arnaud Charlet
This is a fairly major internal reorganization of how info and warning messages are handled. Info messages for elaboration are now tagged as [-gnatel] if warning tagging is activated (-gnatw.d), and info messages coming from instantiations are consistently labeled as such as shown by this example,

Re: [patch i386]: Combine memory and indirect jump

2014-06-11 Thread Kai Tietz
2014-06-11 11:57 GMT+02:00 Steven Bosscher : > On Wed, Jun 11, 2014 at 10:32 AM, Kai Tietz wrote: >> this patch adds simple combining of indirect-jumps on memory-address. >> This patch is pretty similar to sibcall-combing. >> ChangeLog >> >> 2014-06-11 Kai Tietz >> >> * config/i386/i386.md (

[ARM] Fix build failure due to movsi_compare0

2014-06-11 Thread James Greenhalgh
Hi, A recent change somewhere exposed a latent bug between LRA and the definition of the movsi_compare0 pattern. This pattern ties the source and destination register of a set together a (match_dup) and register constraints: [(set (reg:CC CC_REGNUM) (compare:CC (match_operand:SI 1 "s

Re: libsanitizer merge from upstream r208536

2014-06-11 Thread Paolo Carlini
Hi, On 05/22/2014 09:02 PM, Jakub Jelinek wrote: In file included from ../../../../trunk/libsanitizer/asan/asan_interceptors.cc:147:0: ../../../../trunk/libsanitizer/sanitizer_common/sanitizer_common_interceptors.inc: In function ‘int __interceptor_accept4(int, void*, unsigned int*, int)’: ../..

Re: [PATCH] Trust TREE_ADDRESSABLE

2014-06-11 Thread Richard Biener
On Wed, 11 Jun 2014, Eric Botcazou wrote: > > Because that's not the point and because it feels like a hack ;) > > Well, if we keep the current semantics of TREE_ADDRESSABLE and decide that > the > predicate for aliasing is may_be_aliased, the implementation for the latter > becomes a detail.

[PATCH 9/9] Remove __gnu_uldivmod_helper

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.c (__gnu_uldivmod_helper): Remove. --- libgcc/config/arm/bpabi.c | 14 -- 1 file changed, 14 deletions(-) diff --git a/libgcc/config/arm/bpabi.c b/libgcc/config/arm/bpabi.c index 7b155cc..e90d044 100644 --- a/libgcc/config/arm/b

[PATCH 7/9] Fix cfi annotations

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.S (__aeabi_ldivmod, __aeabi_uldivmod, push_for_divide, pop_for_divide): Use .cfi_* directives for DWARF annotations. Fix DWARF information. --- libgcc/config/arm/bpabi.S | 21 - 1 file changed, 16 insertio

[PATCH 6/9] Optimise __aeabi_ldivmod

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.S (__aeabi_ldivmod): Perform division using __udivmoddi4, and fixups for negative operands. --- libgcc/config/arm/bpabi.S | 41 - 1 file changed, 40 insertions(+), 1 deletion(-) diff --git a/l

[PATCH 8/9] Use __udivmoddi4 for v6M aeabi_uldivmod

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi-v6m.S (__aeabi_uldivmod): Perform division using __udivmoddi4. --- libgcc/config/arm/bpabi-v6m.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libgcc/config/arm/bpabi-v6m.S b/libgcc/config/arm/bpabi-v6m.S index

[PATCH 4/9] Optimise __aeabi_uldivmod

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.S (__aeabi_uldivmod): Perform division using call to __udivmoddi4. --- libgcc/config/arm/bpabi.S | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libgcc/config/arm/bpabi.S b/libgcc/config/arm/bpabi.S index 67246b0.

[PATCH 5/9] Optimise __aeabi_ldivmod (stack manipulation)

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.S (__aeabi_ldivmod): Optimise stack manipulation. --- libgcc/config/arm/bpabi.S | 14 +++--- 1 file changed, 3 insertions(+), 11 deletions(-) diff --git a/libgcc/config/arm/bpabi.S b/libgcc/config/arm/bpabi.S index 927e37f..3f9ece5

[PATCH 2/9] Add comments

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.S (__aeabi_uldivmod, __aeabi_ldivmod): Add comment describing register usage on function entry and exit. --- libgcc/config/arm/bpabi.S | 16 1 file changed, 16 insertions(+) diff --git a/libgcc/config/arm/bpabi.S b/

[PATCH 1/9] Whitespace

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.S (__aeabi_uldivmod): Fix whitespace. (__aeabi_ldivmod): Fix whitespace. --- libgcc/config/arm/bpabi.S | 36 ++-- 1 file changed, 18 insertions(+), 18 deletions(-) diff --git a/libgcc/config/arm/bpabi

[PATCH 3/9] Optimise __aeabi_uldivmod (stack manipulation)

2014-06-11 Thread Charles Baylis
2014-05-22 Charles Baylis * config/arm/bpabi.S (__aeabi_uldivmod): Optimise stack pointer manipulation. --- libgcc/config/arm/bpabi.S | 54 +-- 1 file changed, 43 insertions(+), 11 deletions(-) diff --git a/libgcc/config/arm/bpabi.S

  1   2   >