Re: [dwarf, RFC] Emitting per-function dwarf info

2015-05-19 Thread Senthil Kumar Selvaraj
Ping! Regards Senthil On Fri, Apr 10, 2015 at 12:19:36PM +0530, Senthil Kumar Selvaraj wrote: > Hi, > > This (rather big) patch is an attempt to generate per function DWARF > information for functions that go into their own sections (through > -ffunction-section or otherwise). This is so th

Cleanup and improve canonical type construction in LTO

2015-05-19 Thread Jan Hubicka
Richard, this is my attempt to make sense of TYPE_CANONICAL at LTO. My undrestanding is that gimple_canonical_types_compatible_p needs to return true for all pairs of types that are considered compatible across compilation unit for any of languages we support (and in a sane way for cross language,

[PATCH] Fix PR target/65730

2015-05-19 Thread Max Filippov
2015-05-20 Max Filippov gcc/ * config/xtensa/xtensa.c (init_alignment_context): Replace MULT by BITS_PER_UNIT with ASHIFT by exact_log2 (BITS_PER_UNIT). --- gcc/config/xtensa/xtensa.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/gcc/config/xtensa/xten

Re: [AArch64][PR65375] Fix RTX cost for vector SET

2015-05-19 Thread Kugan
On 07/05/15 17:24, James Greenhalgh wrote: > On Wed, May 06, 2015 at 03:12:33AM +0100, Kugan wrote: gcc/ChangeLog: 2015-04-24 Kugan Vivekanandarajah Jim Wilson * config/arm/aarch-common-protos.h (struct mem_cost_table): Added new fields loadv

Re: [patch, gcc 5 regression] re-enable biarch for powerpc-linux-gnu

2015-05-19 Thread David Edelsohn
On Tue, May 19, 2015 at 8:03 PM, Sandra Loosemore wrote: > On 05/19/2015 05:34 PM, Alan Modra wrote: >> >> On Tue, May 19, 2015 at 02:18:23PM -0400, David Edelsohn wrote: >>> >>> This seems reasonable to me. >>> >>> Alan, any thoughts from you? >> >> >> Looks good. > > > Thanks. I've checked the

Fix minor memory leak in jump threader

2015-05-19 Thread Jeff Law
Honza pointed out we were leaking some memory in the jump threading code. There's at least two leaks, this fixes the first. Bootstrapped and regression tested on x86_64-unknown-linux-gnu. Installed on the trunk. commit f4517a9d2f17b0890d9fe1159a3981985bed5672 Author: Jeff Law Date: Tue Ma

fix class enum lookup

2015-05-19 Thread Nathan Sidwell
I've committed this obvious patch for 65954. We failed to issue a diagnostic for a failed class enum lookup, and returned NULL, rather than error_mark_node, leading to a seg fault. booted & tested on x86_64-linux. nathan 2015-05-19 Nathan sidwell cp/ PR c++/65954 * typeck.c (finish_cla

[PING] [C++17] Implement N3928 - Extending static_assert

2015-05-19 Thread Ed Smith-Rowland
On 05/02/2015 04:16 PM, Ed Smith-Rowland wrote: This extends' static assert to not require a message string. I elected to make this work also for C++11 and C++14 and warn only with -pedantic. I think many people just write static_assert(thing, ""); . I took the path of building an empty stri

Re: [PATCH 4/4] Split-stack arg pointer init refinement

2015-05-19 Thread Alan Modra
Thanks for reviewing! Somehow I missed seeing your OK in yesterday's email. I have far too many emails in my inbox.. Patch series committed as revisions 223424, 223425, 223426, and 223427. I made a change to the comment for rs6000_supports_split_stack, instead of /* -fsplit-stack uses a field

Re: [PATCH 1/4] rs6000_stack_info changes for -fsplit-stack

2015-05-19 Thread Alan Modra
On Mon, May 18, 2015 at 02:05:59PM -0400, David Edelsohn wrote: > On Sun, May 17, 2015 at 10:54 PM, Alan Modra wrote: > > This patch changes rs6000_stack_info to keep save areas offsets even > > when not used. I need lr_save_offset valid for split-stack, and it > > seemed reasonable to treat the

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Rich Felker
On Tue, May 19, 2015 at 05:10:11PM -0700, H.J. Lu wrote: > On Tue, May 19, 2015 at 1:54 PM, Rich Felker wrote: > > On Tue, May 19, 2015 at 01:27:06PM -0700, H.J. Lu wrote: > >> On Tue, May 19, 2015 at 1:15 PM, Rich Felker wrote: > >> > On Tue, May 19, 2015 at 12:17:18PM -0700, H.J. Lu wrote: > >>

[PATCH] fixup hash table descriptor in winnt.c

2015-05-19 Thread tbsaunde+gcc
From: Trevor Saunders Hi, This is a straight forward fixup of the hash table descriptor in winnt.c causing the PR. Tested a cross to i686-cygwin now builds, and committing to trunk. Trev gcc/ChangeLog: 2015-05-19 Trevor Saunders PR c++/65835 * config/i386/winnt.c (struct

Re: [PATCH 3/4] split-stack for powerpc64

2015-05-19 Thread Alan Modra
On Tue, May 19, 2015 at 07:40:15AM -0500, Lynn A. Boger wrote: > Questions on the use of the options for split stack: > > - The way this is implemented, split stack is generated if the > target platform supports split stack, on ppc64/ppc64le as well > as on x86, and the use of -fno-split-stack doe

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 1:54 PM, Rich Felker wrote: > On Tue, May 19, 2015 at 01:27:06PM -0700, H.J. Lu wrote: >> On Tue, May 19, 2015 at 1:15 PM, Rich Felker wrote: >> > On Tue, May 19, 2015 at 12:17:18PM -0700, H.J. Lu wrote: >> >> On Tue, May 19, 2015 at 12:11 PM, Richard Henderson >> >> wro

Re: [patch, gcc 5 regression] re-enable biarch for powerpc-linux-gnu

2015-05-19 Thread Sandra Loosemore
On 05/19/2015 05:34 PM, Alan Modra wrote: On Tue, May 19, 2015 at 02:18:23PM -0400, David Edelsohn wrote: This seems reasonable to me. Alan, any thoughts from you? Looks good. Thanks. I've checked the patch into mainline (r223418). I assume this is also a candidate for backporting to the

Re: [patch, gcc 5 regression] re-enable biarch for powerpc-linux-gnu

2015-05-19 Thread Alan Modra
On Tue, May 19, 2015 at 02:18:23PM -0400, David Edelsohn wrote: > This seems reasonable to me. > > Alan, any thoughts from you? Looks good. -- Alan Modra Australia Development Lab, IBM

[SH][committed] Fix pr64366.c test case

2015-05-19 Thread Oleg Endo
Hi, The SH testcase pr64366 would fail on SH2A because -m4 -ml is specified in dg-options of the test case. Target options such as CPU and endianness are usually not specified in the test cases directly, but when invoking the test suite. Fixed with the attached patch. Committed as r223417. Che

Re: [PATCH] add self-tuning to x86 hardware fast path in libitm

2015-05-19 Thread Torvald Riegel
On Mon, 2015-05-18 at 23:27 -0400, Nuno Diegues wrote: > On Mon, May 18, 2015 at 5:29 PM, Torvald Riegel wrote: > > > > Are there better options for the utility function, or can we tune it to > > be less affected by varying txn length and likelihood of txnal vs. > > nontxnal code? What are the th

Re: ODR merging and implicit typedefs

2015-05-19 Thread Jan Hubicka
> On 05/19/2015 01:33 PM, Jan Hubicka wrote: > >I tracked down that those are implicit typedef created by > >create_implicit_typedef. > >My patch made them no longer anonymous that in turn triggers the bogus > >diagnostics. > >I do not think it is fully correct though - those types are not anonym

Re: ODR merging and implicit typedefs

2015-05-19 Thread Jason Merrill
On 05/19/2015 01:33 PM, Jan Hubicka wrote: I tracked down that those are implicit typedef created by create_implicit_typedef. My patch made them no longer anonymous that in turn triggers the bogus diagnostics. I do not think it is fully correct though - those types are not anonymous. Hmm? Th

Re: [match-and-simplify] fix incorrect code-gen in 'for' pattern

2015-05-19 Thread Prathamesh Kulkarni
On 19 May 2015 at 14:34, Richard Biener wrote: > On Tue, 19 May 2015, Prathamesh Kulkarni wrote: > >> On 18 May 2015 at 20:17, Prathamesh Kulkarni >> wrote: >> > On 18 May 2015 at 14:12, Richard Biener wrote: >> >> On Sat, 16 May 2015, Prathamesh Kulkarni wrote: >> >> >> >>> Hi, >> >>> genmatch

Re: Fwd: Re: [PATCH, RFC]: Next stage1, refactoring: propagating rtx subclasses

2015-05-19 Thread Jeff Law
On 05/18/2015 11:07 PM, Mikhail Maltsev wrote: It seems rather weird to me: I have definitely sent this message, but now I realized that it did not get to the mailing list archive (and it was not bounced either). Thus, resending. I got the original, not sure why it didn't show up in the archives

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Rich Felker
On Tue, May 19, 2015 at 01:27:06PM -0700, H.J. Lu wrote: > On Tue, May 19, 2015 at 1:15 PM, Rich Felker wrote: > > On Tue, May 19, 2015 at 12:17:18PM -0700, H.J. Lu wrote: > >> On Tue, May 19, 2015 at 12:11 PM, Richard Henderson > >> wrote: > >> > On 05/19/2015 12:06 PM, H.J. Lu wrote: > >> >> O

Re: Demangle symbols in debug assertion messages

2015-05-19 Thread François Dumont
Hello Is it ok ? François On 04/05/2015 22:31, François Dumont wrote: Hi Here is the patch to demangle symbols in debug messages. I have also simplify code in formatter.h. Here is an example of assertion message: /home/fdt/dev/gcc/build/x86_64-unknown-linux-gnu/libstdc++-v3/i

Re: miter_base simplification

2015-05-19 Thread François Dumont
On 03/05/2015 22:19, François Dumont wrote: On 30/04/2015 13:18, Jonathan Wakely wrote: On 30/04/15 10:40 +0200, François Dumont wrote: On 27/04/2015 13:55, Jonathan Wakely wrote: (Alternatively, could the same simplification be made for __miter_base? Do we need _Miter_base<> or just two overl

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 1:15 PM, Rich Felker wrote: > On Tue, May 19, 2015 at 12:17:18PM -0700, H.J. Lu wrote: >> On Tue, May 19, 2015 at 12:11 PM, Richard Henderson wrote: >> > On 05/19/2015 12:06 PM, H.J. Lu wrote: >> >> On Tue, May 19, 2015 at 11:59 AM, Richard Henderson >> >> wrote: >> >>>

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Rich Felker
On Tue, May 19, 2015 at 12:17:18PM -0700, H.J. Lu wrote: > On Tue, May 19, 2015 at 12:11 PM, Richard Henderson wrote: > > On 05/19/2015 12:06 PM, H.J. Lu wrote: > >> On Tue, May 19, 2015 at 11:59 AM, Richard Henderson > >> wrote: > >>> On 05/19/2015 11:06 AM, Rich Felker wrote: > I'm still

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Richard Henderson
On 05/19/2015 12:35 PM, Rich Felker wrote: > Why would you recompute it (this requires a fairly expensive call that > reads or pops its own return address) rather than simply spilling the > already-computed value and reloading it from the stack? > > The only example I can think of where it might m

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Richard Henderson
On 05/19/2015 12:17 PM, H.J. Lu wrote: >> But my point is that the only time the compiler should present you with the >> form of indirect branch you're looking for is when there's no place to hoist >> the load. >> >> At which point, is it really worth adding a new relocation to the ABI? Is it >> r

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Rich Felker
On Tue, May 19, 2015 at 11:59:00AM -0700, Richard Henderson wrote: > On 05/19/2015 11:06 AM, Rich Felker wrote: > > I'm still mildly worried that concerns for supporting > > relaxation might lead to decisions not to optimize code in ways that > > would be difficult to relax (e.g. certain types of a

[C PATCH] Use AGGREGATE_TYPE_P

2015-05-19 Thread Marek Polacek
Bootstrapped/regtested on x86_64-linux, applying to trunk. 2015-05-19 Marek Polacek * c-typeck.c (start_init): Use AGGREGATE_TYPE_P. diff --git gcc/c/c-typeck.c gcc/c/c-typeck.c index 7f54490..cf5322f 100644 --- gcc/c/c-typeck.c +++ gcc/c/c-typeck.c @@ -7126,10 +7126,7 @@ start_init (

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 8:33 AM, Joseph Myers wrote: > On Tue, 19 May 2015, H.J. Lu wrote: > >> > I think the whole thing should be posted as one patch, with both the >> > target-independent changes and the target-specific changes for all >> > targets. >> > >> >> That is what makes me concerned.

Re: [PATCH] Fix memory orders description in atomic ops built-ins docs.

2015-05-19 Thread Torvald Riegel
On Mon, 2015-05-18 at 17:36 +0100, Matthew Wahab wrote: > Hello, > > On 15/05/15 17:22, Torvald Riegel wrote: > > This patch improves the documentation of the built-ins for atomic > > operations. > > The "memory model" to "memory order" change does improve things but I think > that > the patch h

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 12:11 PM, Richard Henderson wrote: > On 05/19/2015 12:06 PM, H.J. Lu wrote: >> On Tue, May 19, 2015 at 11:59 AM, Richard Henderson wrote: >>> On 05/19/2015 11:06 AM, Rich Felker wrote: I'm still mildly worried that concerns for supporting relaxation might lead to

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Richard Henderson
On 05/19/2015 12:06 PM, H.J. Lu wrote: > On Tue, May 19, 2015 at 11:59 AM, Richard Henderson wrote: >> On 05/19/2015 11:06 AM, Rich Felker wrote: >>> I'm still mildly worried that concerns for supporting >>> relaxation might lead to decisions not to optimize code in ways that >>> would be difficul

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Rich Felker
On Tue, May 19, 2015 at 06:01:07PM +0200, Michael Matz wrote: > Hi, > > On Tue, 19 May 2015, Jeff Law wrote: > > > > > Forget lazy binding. It's dead anyway because serious distros want > > > > PIE+relro+bindnow+... > > > > > > You keep saying this, but I can't help the feeling it's mostly beca

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 11:59 AM, Richard Henderson wrote: > On 05/19/2015 11:06 AM, Rich Felker wrote: >> I'm still mildly worried that concerns for supporting >> relaxation might lead to decisions not to optimize code in ways that >> would be difficult to relax (e.g. certain types of address loa

Move dependency for shared libgcc

2015-05-19 Thread Eric Botcazou
Probably a misapplied patch: the dependency of the shared libgcc on the shared libunwind is in a wrong place in Makefile. The patch also removes a useless endif/ifneq pair. Tested on x86_64-suse-linux and ia64-suse-linux, applied as obvious. 2015-05-19 Eric Botcazou * Makefile.in

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Richard Henderson
On 05/19/2015 11:06 AM, Rich Felker wrote: > I'm still mildly worried that concerns for supporting > relaxation might lead to decisions not to optimize code in ways that > would be difficult to relax (e.g. certain types of address load > reordering or hoisting) but I don't understand GCC internals

RE: [PATCH, MIPS]: Fix internal compiler error: in check_bool_attrs, at recog.c:2218 for micromips attribute

2015-05-19 Thread Robert Suchanek
Hi, The original patch had a missing declaration of micromips_globals in mips.h that appears to be the cause of segmentation faults when building mips-mti-linux-gnu. I didn't get any failures just before the submission neither on mips-img-linux-gnu nor mips64el-linux-gnu and the test case is to

Re: [patch, gcc 5 regression] re-enable biarch for powerpc-linux-gnu

2015-05-19 Thread David Edelsohn
This seems reasonable to me. Alan, any thoughts from you? Thanks, David On Mon, May 18, 2015 at 12:22 PM, Sandra Loosemore wrote: > We've found that configuring a powerpc-linux-gnu cross toolchain with > --enable-targets=all no longer enables -m64 support in GCC 5, due to the > patch for PR ta

[PING] [RFC 12/13] S/390 Vector ABI GNU Attribute.

2015-05-19 Thread Andreas Krebbel
On 05/11/2015 03:23 PM, Andreas Krebbel wrote: > With this patch .gnu_attribute is used to mark binaries with a vector > ABI tag. This is required since the z13 vector support breaks the ABI > of existing vector_size attribute generated vector types: > > 1. vector_size(16) and bigger vectors are

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Sriraman Tallam
On Tue, May 19, 2015 at 10:22 AM, Yury Gribov wrote: > On 05/19/2015 09:16 AM, Sriraman Tallam wrote: >> >> We have the following problem with selectively compiling modules with >> -m options and I have provided a solution to solve this. I would >> like to hear what you think. >> >> Multi version

Re: [PATCH/libiberty] fix build of gdb/binutils with clang.

2015-05-19 Thread Ian Lance Taylor
On Tue, May 19, 2015 at 11:08 AM, Yunlian Jiang wrote: > > I could do that and it make the compilation of libiberty passes. > However, I have some other problem when using clang to build gdb > because of libiberty. > > Some c file from other component may include 'libiberty.h' which contains > th

Re: [PATCH/libiberty] fix build of gdb/binutils with clang.

2015-05-19 Thread DJ Delorie
> If the other c file only includes libiberty.h and does not include the > libiberty/config.h and In general, such "other c files" should have their own config.h that does the same test and has its own HAVE_DECL_ASPRINTF. That way, the config.h matches the compiler options being used, and not th

Re: [PATCH/libiberty] fix build of gdb/binutils with clang.

2015-05-19 Thread Yunlian Jiang
I could do that and it make the compilation of libiberty passes. However, I have some other problem when using clang to build gdb because of libiberty. Some c file from other component may include 'libiberty.h' which contains the following #if !HAVE_DECL_ASPRINTF /* Like sprintf but provides a p

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Rich Felker
On Tue, May 19, 2015 at 04:43:53PM +0200, Michael Matz wrote: > Hi, > > On Fri, 15 May 2015, Rich Felker wrote: > > > Forget lazy binding. It's dead anyway because serious distros want > > PIE+relro+bindnow+... > > You keep saying this, but I can't help the feeling it's mostly because > musl do

Use a couple of macros in stor-layout.c

2015-05-19 Thread Eric Botcazou
Self-explanatory, tested on x86_64-suse-linux, applied as obvious. 2015-05-19 Eric Botcazou * stor-layout.c (finalize_type_size): Use AGGREGATE_TYPE_P. (layout_type): Use RECORD_OR_UNION_TYPE_P. -- Eric BotcazouIndex: stor-layout.c ==

Re: ODR merging and implicit typedefs

2015-05-19 Thread Markus Trippelsdorf
On 2015.05.19 at 19:33 +0200, Jan Hubicka wrote: > > Jason, > I just noticed that there are bogus ODR violation warnings during > LTO-bootstrap > (that breaks -Werror builds). It was caused by my work-around for > type_in_anonymous_namespace > for the issue discussed in: > https://gcc.gnu.org/m

Re: [Patch][loop-invariant.c] Fix a couple of bugs regarding loop invariant motion discovered by spec2k6 on aarch64

2015-05-19 Thread Jeff Law
On 05/18/2015 02:16 AM, David Sherwood wrote: Hi Jeff, Thanks for the suggestion. I did a bootstrap x86_64 build before and after my patch and looked for differences in the last stage object files and there were plenty of them. I chose a nice simple function (check_callers) from ipa-inline-analy

ODR merging and implicit typedefs

2015-05-19 Thread Jan Hubicka
Jason, I just noticed that there are bogus ODR violation warnings during LTO-bootstrap (that breaks -Werror builds). It was caused by my work-around for type_in_anonymous_namespace for the issue discussed in: https://gcc.gnu.org/ml/gcc-patches/2015-05/msg01245.html (i.e. the TYPE_STUB_DECL disuc

Re: [PATCH] Add SPECIAL_FLOAT_MODE to enable adding IEEE 128-bit floating point to PowerPC

2015-05-19 Thread Michael Meissner
On Fri, May 08, 2015 at 01:05:59PM -0600, Jeff Law wrote: > On 05/06/2015 11:29 AM, Michael Meissner wrote: > >On Wed, May 06, 2015 at 04:03:00PM +0100, Richard Sandiford wrote: > >>Jeff Law writes: > >>>So my worry here is that folks writing these loops to iterate over modes > >>>are going to eas

Re: [RFC]: Remove Mem/address type assumption in combiner

2015-05-19 Thread Jeff Law
On 05/16/2015 07:55 PM, Hans-Peter Nilsson wrote: On Sat, 16 May 2015, Segher Boessenkool wrote: On Sat, May 16, 2015 at 12:36:38PM -0400, Hans-Peter Nilsson wrote: On Sat, 16 May 2015, Segher Boessenkool wrote: On Fri, May 15, 2015 at 10:40:48PM -0400, Hans-Peter Nilsson wrote: I confess the

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Yury Gribov
On 05/19/2015 09:16 AM, Sriraman Tallam wrote: We have the following problem with selectively compiling modules with -m options and I have provided a solution to solve this. I would like to hear what you think. Multi versioning at module granularity is done by compiling a subset of modules with

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Sriraman Tallam
On Tue, May 19, 2015 at 2:39 AM, Richard Biener wrote: > On Tue, May 19, 2015 at 8:16 AM, Sriraman Tallam wrote: >> We have the following problem with selectively compiling modules with >> -m options and I have provided a solution to solve this. I would >> like to hear what you think. >> >> Mult

[committed] linear/lastprivate clause fixes

2015-05-19 Thread Jakub Jelinek
Hi! When working on taskloop, I've noticed various issues in the OpenMP 4.0 handling of the linear/lastprivate (explicit as well as implicit) clauses. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk, plan to backport to 5/4.9 after a while. 2015-05-19 Jakub Jelinek

[ping**2] Handle MULTILIB_REUSE in auto-generated SYSROOT_SUFFIX_SPEC macro

2015-05-19 Thread Sandra Loosemore
Re-pinging a patch from last year that never got reviewed: https://gcc.gnu.org/ml/gcc-patches/2014-06/msg00511.html This problem still exists in GCC 5.1 and the above patch still fixes it. I haven't tried mainline head yet, but it doesn't look like anything else has touched this since we bran

RE: Refactor gimple_expr_type

2015-05-19 Thread Aditya K
> Date: Tue, 19 May 2015 11:33:16 +0200 > Subject: Re: Refactor gimple_expr_type > From: richard.guent...@gmail.com > To: hiradi...@msn.com > CC: tbsau...@tbsaunde.org; gcc-patches@gcc.gnu.org > > On Tue, May 19, 2015 at 12:04 AM, Aditya K wrote: >> >> >>

Fix PR48052: loop not vectorized if index is "unsigned int"

2015-05-19 Thread Aditya K
w.r.t. the PR48052, here is the patch which finds out if scev would wrap or not. The patch symbolically evaluates if valid_niter>= loop->nb_iterations is true. In that case the scev would not wrap (??). Currently, we only look for two special 'patterns', which are sufficient to analyze the simple

Re: [RFC] COMDAT Safe Module Level Multi versioning

2015-05-19 Thread Xinliang David Li
> > Hm. But which options are unsafe? Also wouldn't it be better to simply > _not_ have unsafe options produce comdats but always make local clones > for them (thus emit the comdat with "unsafe" flags dropped)? Always localize comdat functions may lead to text size increase. It does not work if

Re: [Patch ARM-AArch64/testsuite 00/13] Neon intrinsics executable tests

2015-05-19 Thread Christophe Lyon
On 19 May 2015 at 15:32, James Greenhalgh wrote: > On Tue, May 12, 2015 at 09:30:48PM +0100, Christophe Lyon wrote: >> This patch series is a follow-up to the tests I already contributed, >> converted from my original testsuite. >> >> This series consists in 13 new files, which can be committed >>

Re: [Patch AArch64] Add cpu_defines.h for AArch64.

2015-05-19 Thread Ramana Radhakrishnan
On Tue, May 19, 2015 at 4:54 PM, wrote: > > > > >> On May 19, 2015, at 5:54 AM, Ramana Radhakrishnan >> wrote: >> >> Hi, >> >> Like the ARM port, the AArch64 ports needs to set glibc_integral_traps to >> false as integer divide instructions do not trap. >> >> Bootstrapped and regression tested

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Michael Matz
Hi, On Tue, 19 May 2015, Jeff Law wrote: > > > Forget lazy binding. It's dead anyway because serious distros want > > > PIE+relro+bindnow+... > > > > You keep saying this, but I can't help the feeling it's mostly because > > musl doesn't support it ;-) > > FWIW, Red Hat is pushing PIE & parti

Re: [Patch AArch64] Add cpu_defines.h for AArch64.

2015-05-19 Thread pinskia
> On May 19, 2015, at 5:54 AM, Ramana Radhakrishnan > wrote: > > Hi, > > Like the ARM port, the AArch64 ports needs to set glibc_integral_traps to > false as integer divide instructions do not trap. > > Bootstrapped and regression tested on aarch64-none-linux-gnu > > Ok to apply ? Not r

Re: breakage with series "[0/9] Record number of hard registers in a REG"

2015-05-19 Thread Richard Sandiford
Hans-Peter Nilsson writes: > g++ -c -g -O2 -DIN_GCC -DCROSS_DIRECTORY_STRUCTURE -fno-exceptions > -fno-rtti -fasynchronous-unwind-tables -W -Wall -Wwrite-strings > -Wcast-qual -Wmissing-format-attribute -Woverloaded-virtual -pedantic > -Wno-long-long -Wno-variadic-macros -Wno-overlength-strings -f

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

2015-05-19 Thread Kumar, Venkataramanan
Hi Maintainers, Please find the attached patch, that fixes add/extend gcc test suite failures in Aarch64 target. Ref: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66049 These tests started to fail after we prevented combiner from converting shift RTX to mult RTX, when the RTX is not inside a

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

2015-05-19 Thread Kyrill Tkachov
Hi Venkat, On 19/05/15 16:37, Kumar, Venkataramanan wrote: Hi Maintainers, Please find the attached patch, that fixes add/extend gcc test suite failures in Aarch64 target. Ref: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66049 These tests started to fail after we prevented combiner from conv

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 8:33 AM, wrote: > > > From: gcc-patches-ow...@gcc.gnu.org [gcc-patches-ow...@gcc.gnu.org] on behalf > of H.J. Lu [hjl.to...@gmail.com] > Sent: Tuesday, May 19, 2015 11:27 AM > To: Joseph Myers > Cc: Magnus Granberg; GCC Patches > S

RE: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-19 Thread Paul_Koning
From: gcc-patches-ow...@gcc.gnu.org [gcc-patches-ow...@gcc.gnu.org] on behalf of H.J. Lu [hjl.to...@gmail.com] Sent: Tuesday, May 19, 2015 11:27 AM To: Joseph Myers Cc: Magnus Granberg; GCC Patches Subject: Re: PING^3: [PATCH]: New configure options that m

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-19 Thread Joseph Myers
On Tue, 19 May 2015, H.J. Lu wrote: > > I think the whole thing should be posted as one patch, with both the > > target-independent changes and the target-specific changes for all > > targets. > > > > That is what makes me concerned. I have some simple target-specified > patches which weren't re

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-19 Thread H.J. Lu
On Tue, May 19, 2015 at 8:21 AM, Joseph Myers wrote: > On Mon, 18 May 2015, H.J. Lu wrote: > >> > Have updates for all affected specs for all targets been posted? I just >> > saw a small and apparently arbitrary subset of targets with patches, and >> > no explanation of how those targets were ide

Re: PING^3: [PATCH]: New configure options that make the compiler use -fPIE and -pie as default option

2015-05-19 Thread Joseph Myers
On Mon, 18 May 2015, H.J. Lu wrote: > > Have updates for all affected specs for all targets been posted? I just > > saw a small and apparently arbitrary subset of targets with patches, and > > no explanation of how those targets were identified or why the other > > targets with specs mentioning t

breakage with series "[0/9] Record number of hard registers in a REG"

2015-05-19 Thread Hans-Peter Nilsson
> From: Richard Sandiford > Date: Mon, 18 May 2015 20:09:19 +0200 > While looking at a profile of gcc, I noticed one thing fairly high > up the list was a loop iterating over all the registers in a REG, > apparently due to the delay in computing the index for hard_regno_nregs > and then loading t

Re: [PATCH, alpha]: Some cleanups

2015-05-19 Thread Uros Bizjak
On Tue, May 19, 2015 at 5:10 PM, Uros Bizjak wrote: > No functional changes. > > 2015-05-18 Uros Bizjak > > * config/alpha/alpha.c (alpha_legitimize_reload_address) > (alpha_preferred_reload_class, alpha_legitimate_constant_p): Use > CONST_INT_P, CONST_SCALAR_INT_P and CONST_DOUBLE_

[PATCH, alpha]: Some cleanups

2015-05-19 Thread Uros Bizjak
No functional changes. 2015-05-18 Uros Bizjak * config/alpha/alpha.c (alpha_legitimize_reload_address) (alpha_preferred_reload_class, alpha_legitimate_constant_p): Use CONST_INT_P, CONST_SCALAR_INT_P and CONST_DOUBLE_P predicates. (alpha_split_reload_pair) : Use CASE_CONST_

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Jeff Law
On 05/19/2015 08:43 AM, Michael Matz wrote: Hi, On Fri, 15 May 2015, Rich Felker wrote: Forget lazy binding. It's dead anyway because serious distros want PIE+relro+bindnow+... You keep saying this, but I can't help the feeling it's mostly because musl doesn't support it ;-) FWIW, Red Hat is

Re: [PATCH 02/13] optabs: Fix vec_perm -> V16QI middle end lowering.

2015-05-19 Thread Richard Henderson
On 05/19/2015 01:41 AM, Andreas Krebbel wrote: > On 05/18/2015 07:35 PM, Richard Henderson wrote: >> On 05/11/2015 06:23 AM, Andreas Krebbel wrote: >>> @@ -6784,14 +6784,18 @@ expand_vec_perm (machine_mode mode, rtx v0, rtx v1, >>> rtx sel, rtx target) >>> { >>>/* Multiply each elemen

Re: [PATCH i386] Allow sibcalls in no-PLT PIC

2015-05-19 Thread Michael Matz
Hi, On Fri, 15 May 2015, Rich Felker wrote: > Forget lazy binding. It's dead anyway because serious distros want > PIE+relro+bindnow+... You keep saying this, but I can't help the feeling it's mostly because musl doesn't support it ;-) No, you don't have to use bindnow to get the effects of re

Re: [PATCH 4/4] Split-stack arg pointer init refinement

2015-05-19 Thread David Edelsohn
This small refinement to the -fsplit-stack prologue arg pointer initialization improves code generation. Compare the -O2 gcc/testsuite/gcc.dg/split-3.c code for down() below. beforeafter mflr 0mflr 0 std 31,-8(1)std 31,-8(1) std 0,16(1)mr 12,1

Re: [PATCH 2/4] prologue and epilogue tidy and -mno-vrsave bug fix

2015-05-19 Thread David Edelsohn
On Sun, May 17, 2015 at 10:54 PM, Alan Modra wrote: > This patch tidies the prologue and epilogue altivec code a little. > A number of places using info->altivec_size unnecessarily also test > TARGET_ALTIVEC_ABI, when rs6000_stack_info() guarantees that > info->altivec_size is zero if !TARGET_ALTI

Re: [patch] libstdc++/66055 add missing constructors to unordered containers

2015-05-19 Thread Jonathan Wakely
On 17/05/15 22:21 +0200, François Dumont wrote: Ok, I just commit fixing some other lines length except those having a long hyperlink, I didn't want to break those. Yep, thanks. I think we should backport Nathan's patch and your one to the gcc-5-branch too. I'll make a note to do that before t

[PATCH] Fix duplicated warning with __attribute__((format)) (PR c/64223)

2015-05-19 Thread Marek Polacek
This PR points out that we output same -Wformat warning twice when using __attribute__ ((format)). The problem was that attribute_value_equal (called when processing merge_attributes) got two lists: "format printf, 1, 2" and "__format__ __printf__, 1, 2", these should be equal. But since attribut

[patch] Optimize std::list when using new ABI

2015-05-19 Thread Jonathan Wakely
This fixes some missed optimizations I should have made when adding the new std::__cxx11::list, making use of the O(1) list::size() when it saves work. In the equality comparisons two lists can't be equal if their sizes differ. When resizing a list we don't need to walk the list to find whether

Re: [Patch, fortran, pr65548, 2nd take, v5] [5/6 Regression] gfc_conv_procedure_call

2015-05-19 Thread Mikael Morin
Le 19/05/2015 10:50, Andre Vehreschild a écrit : > Hi all, > > find attached latest version to fix 65548. > > Bootstraps and regtests ok on x86_64-linux-gnu/f21. > OK. Thanks. Mikael

Re: [Patch, Fortran, PR58586, v4] ICE with derived type with allocatable component passed by value

2015-05-19 Thread Andre Vehreschild
Hi, attached is the most recent version of the patch for 58586. It adapts to recent trunk and addresses the caveats so far, i.e. the testcases in the comments now compile and run again w/o errors. Bootstraps and regtests fine on x86_64-linux-gnu/f21. Comments? - Andre On Fri, 8 May 2015 16:11:

Re: [Patch ARM-AArch64/testsuite 00/13] Neon intrinsics executable tests

2015-05-19 Thread James Greenhalgh
On Tue, May 12, 2015 at 09:30:48PM +0100, Christophe Lyon wrote: > This patch series is a follow-up to the tests I already contributed, > converted from my original testsuite. > > This series consists in 13 new files, which can be committed > independently. > > Another series (hopefully final) wi

[patch, avr] Restore base register if not marked dead/unused

2015-05-19 Thread Sivanupandi, Pitchumani
Test gcc.c-torture/execute/memcpy-bi.c (-O2) failed for attiny40 device. Cause seems to be in "load from memory" as it is not restoring base register after load instructions generated. Function avr_out_load_psi_reg_no_disp_tiny in avr.c: It returns just after emitting instructions to load from mem

[Patch AArch64] Add cpu_defines.h for AArch64.

2015-05-19 Thread Ramana Radhakrishnan
Hi, Like the ARM port, the AArch64 ports needs to set glibc_integral_traps to false as integer divide instructions do not trap. Bootstrapped and regression tested on aarch64-none-linux-gnu Ok to apply ? regards Ramana 2015-05-17 Ramana Radhakrishnan * configure.host: Define cpu

[Patch ARM] Add cpu_defines.h for ARM

2015-05-19 Thread Ramana Radhakrishnan
Hardware Integer divide instructions do not trap. Define this to be so for the ARM port. Applied to trunk after a build and test across architecture ranges and a bootstrap and regression run on a Cortex-A15 - a v7ve core that has hardware divide instructions. A patch for AArch64 follows. re

Re: [PATCH] plugin event for C/C++ function definitions

2015-05-19 Thread Andres Tiraboschi
2015-05-18 16:51 GMT-03:00 : > Hi, this patch adds two new plugin events PLUGIN_START_PARSE_FUNCTION and > PLUGIN_FINISH_PARSE_FUNCTION. These events are invoked at start_function and > finish_function in gcc/c/c-decl.c and gcc/cp/decl.c respectively in the C and > C++ frontends. > PLUGIN_START

Re: [Patch, fortran, 64674, v2] [OOP] ICE in ASSOCIATE with class array

2015-05-19 Thread Andre Vehreschild
Hi, so here is the update on pr 64674. Besides adapting to current trunk nothing has changed from the previous version. The links for getting the patches this one depends on are: PR65548 v5: https://gcc.gnu.org/ml/fortran/2015-05/msg00123.html and PR44672 v6: https://gcc.gnu.org/ml/fortran/2015-0

Commit: MSP430: Enhance the zero_extendhisi2 pattern

2015-05-19 Thread Nick Clifton
Hi Guys, I am applying the patch below to enhance the zero_extendhisi2 pattern in the MSP430 backend so that it can cope with separate source and destination registers. This makes zero extending into another register more efficient and it also helps to work around a reload bug reported

Re: [PATCH] Handle multiple vector sizes in BB vectorization

2015-05-19 Thread Richard Biener
On Tue, 19 May 2015, Rainer Orth wrote: > Richard Biener writes: > > > Well, not really - but at least don't fail vectorization because of that > > but allow it to proceed the "build up from scalar pieces" path. > > > > Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. > >

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

2015-05-19 Thread Steven Bosscher
On Tue, May 19, 2015 at 12:17 PM, Thomas Preud'homme wrote: > 2015-05-18 Thomas Preud'homme > > PR rtl-optimization/66168 > * loop-invariant.c (move_invariant_reg): Set inv->reg to destination > of inv->insn when moving an invariant without introducing a temporary >

Re: [PATCH][AArch64] PR target/65491: Classify V1TF vectors as AAPCS64 short vectors rather than composite types

2015-05-19 Thread James Greenhalgh
On Mon, Apr 20, 2015 at 11:16:02AM +0100, Kyrylo Tkachov wrote: > Hi all, > > The ICE in the PR happens when we pass a 1x(128-bit float) vector as an > argument. > The aarch64 backend erroneously classifies it as a composite type when in > fact it > is a short vector according to AAPCS64 > (sectio

Re: [PATCH] Handle multiple vector sizes in BB vectorization

2015-05-19 Thread Rainer Orth
Richard Biener writes: > Well, not really - but at least don't fail vectorization because of that > but allow it to proceed the "build up from scalar pieces" path. > > Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. The testcase FAILs on Solaris/SPARC: FAIL: gcc.dg/vect/b

Re: Fix PR48052: loop not vectorized if index is "unsigned int"

2015-05-19 Thread Bin.Cheng
On Wed, May 6, 2015 at 7:02 PM, Richard Biener wrote: > On Mon, May 4, 2015 at 9:47 PM, Abderrazek Zaafrani > wrote: >> This is an old thread and we are still running into similar issues: >> Code is not being vectorized on 64-bit target due to scev not being >> able to optimally analyze overflow

Re: [gomp4] Lack of OpenACC NVPTX devices is not an error during scanning

2015-05-19 Thread Jakub Jelinek
On Tue, May 19, 2015 at 11:36:58AM +0100, Julian Brown wrote: > This patch fixes an oversight whereby if the CUDA libraries are > available for some reason on a system that doesn't actually contain an > nVidia card, an OpenACC program will raise an error if the NVPTX > backend is picked as a defaul

[gomp4] Lack of OpenACC NVPTX devices is not an error during scanning

2015-05-19 Thread Julian Brown
Hi, This patch fixes an oversight whereby if the CUDA libraries are available for some reason on a system that doesn't actually contain an nVidia card, an OpenACC program will raise an error if the NVPTX backend is picked as a default instead of falling back to some other device instead. OK for g

[gomp4] Add OpenACC vector-single/vector-partitioned tests

2015-05-19 Thread Julian Brown
Hi, This patch adds several tests of vector-single/vector-partitioned mode, as part of work implementing the OpenACC execution model. Pre-approved for gomp4 branch. I will apply there shortly. Thanks, Julian ChangeLog libgomp/ * testsuite/libgomp.oacc-c-c++-common/vec-single-{1,2,3,4,

  1   2   >