RE: [PATCH, combine] Try REG_EQUAL for nonzero_bits

2014-11-24 Thread Zhenqiang Chen
> -Original Message- > From: Eric Botcazou [mailto:ebotca...@adacore.com] > Sent: Saturday, November 22, 2014 6:15 PM > To: Zhenqiang Chen > Cc: gcc-patches@gcc.gnu.org > Subject: Re: [PATCH, combine] Try REG_EQUAL for nonzero_bits > > > The patch tries to use REG_EQUAL to get more preci

Re: [PATCH v3] gcc/ubsan.c: Use 'pretty_print' for 'pretty_name' to avoid memory overflow

2014-11-24 Thread Chen Gang
On 11/24/14 15:41, Jakub Jelinek wrote: > On Sun, Nov 23, 2014 at 09:13:27AM +0800, Chen Gang wrote: [...] >> + else >> +pp_wide_int(&pretty_name, >> +wi::add (wi::to_widest (TYPE_MAX_VALUE (dom)), 1), >> +TYPE_SIGN (TREE_TYPE (

[Ada] Fix ICE on array with unaligned index type at -O3

2014-11-24 Thread Eric Botcazou
This is a regression present on the mainline and 4.9 branch involving an array with an unaligned index type and when -O3 is specified. Tested on x86_64-suse-linux, applied on the mainline and 4.9 branch. 2014-11-24 Eric Botcazou * gcc-interface/trans.c (push_range_check_info): Repla

Re: [PATCH v3] gcc/ubsan.c: Use 'pretty_print' for 'pretty_name' to avoid memory overflow

2014-11-24 Thread Jakub Jelinek
On Mon, Nov 24, 2014 at 04:28:10PM +0800, Chen Gang wrote: > On 11/24/14 15:41, Jakub Jelinek wrote: > > On Sun, Nov 23, 2014 at 09:13:27AM +0800, Chen Gang wrote: > > [...] > > >> +else > >> + pp_wide_int(&pretty_name, > >> + wi::add (wi::to_widest (TYPE_MAX

RE: [PING**4] [PATCH] Fix PR ipa/61190, 2nd edition‏

2014-11-24 Thread Bernd Edlinger
Hello everybody, currently this makes the virtual thunks and the virtual C++ base classes pretty much broken. So it would be really nice if some one would like to review this patch. I attached the patch again, for your convenience. Thanks Bernd. > Fr

[Ada] Small adjustment to return construct

2014-11-24 Thread Eric Botcazou
This slightly improves the way the returns are built in gigi. Tested on x86_64-suse-linux, applied on the mainline. 2014-11-24 Eric Botcazou * gcc-interface/trans.c (build_return_expr): Use INIT_EXPR instead of MODIFY_EXPR to assign to the return object. (finalize_nrv

Re: [ping] account for register spans in expand_builtin_init_dwarf_reg_sizes

2014-11-24 Thread Olivier Hainque
On Nov 18, 2014, at 16:21 , Olivier Hainque wrote: > On Nov 18, 2014, at 03:29 , Jason Merrill wrote: >> What happens when the outer loop hits a register that we've already seen as >> part of a span? > I have a candidate improvement to prevent processing the same regno > multiple times. > > W

[C++ Patch] PR 63203

2014-11-24 Thread Paolo Carlini
Hi, in the audit trail Jon argued that we should have code specifically checking for references initialized with themselves (because one can't rebind references). I added to his draft a STRIP_NOPS, which manages to get back to the decl on the right hand side which we are looking for + minor t

Re: [PATCH, AARCH64] Fix ICE in CCMP (PR64015)

2014-11-24 Thread Richard Henderson
On 11/24/2014 06:11 AM, Zhenqiang Chen wrote: > Expand pass always uses sign-extend to represent constant value. For the > case in the patch, a 8-bit unsigned value "252" is represented as "-4", > which pass the ccmn check. After mode conversion, "-4" becomes "252", which > leads to mismatch. This

[ping] Couple of old patches

2014-11-24 Thread Eric Botcazou
Fix -mlongcall with nested functions on AIX: https://gcc.gnu.org/ml/gcc-patches/2014-10/msg01692.html Fix handling of inlining and nested functions in dwarf2out.c: https://gcc.gnu.org/ml/gcc-patches/2014-11/msg00665.html Thanks in advance. -- Eric Botcazou

Re: [PATCH, combine] Try REG_EQUAL for nonzero_bits

2014-11-24 Thread Eric Botcazou
> Thanks for the comments. I will compare the two nonzero_bits from src and > REG_EQUAL. Then select the smaller one. They are masks so you can simply AND them before ORing the result. > Do you know why it use " SET_SRC (set)" other than "src" for > num_sign_bit_copies? > > If it is "src", I sho

Re: [Android] Enable ifuncs on Android

2014-11-24 Thread Alexander Ivchenko
*ping* thanks, Alexander 2014-11-14 19:49 GMT+03:00 H.J. Lu : > On Thu, Nov 13, 2014 at 8:59 PM, Jeff Law wrote: >> On 11/13/14 21:46, H.J. Lu wrote: >> >>> >>> Hi Jeff, >>> >>> I believe the patch is correct. Not all glibcs support IFUNC. It doesn't >>> mean >>> we should disable IFUNC for all

Re: [PATCH, 1/8] Expand oacc kernels after pass_build_ealias

2014-11-24 Thread Tom de Vries
On 15-11-14 18:19, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial support for the oacc kernels directive. The patch series uses pass_parallelize_loops to implement parallelization of loops in the oacc kernels region. The patch series

[PATCH] Fix PR64039

2014-11-24 Thread Richard Biener
Committed. Richard. 2014-11-24 Richard Biener PR testsuite/64039 * gcc.dg/tree-ssa/ssa-dom-cse-2.c: XFAIL for hppa*-*-*. Index: testsuite/gcc.dg/tree-ssa/ssa-dom-cse-2.c === --- testsuite/gcc.dg/tree-ssa/ssa-dom

Re: [Patch, libstdc++/63920] Fix regex_constants::match_not_null behavior

2014-11-24 Thread Jonathan Wakely
On 18/11/14 11:12 -0800, Tim Shen wrote: Bootstrapped and tested. Thanks! -- Regards, Tim Shen commit f17155183b9ae1283d04f3bbdb61d05d9279ebe4 Author: timshen Date: Tue Nov 18 00:07:28 2014 -0800 PR libstdc++/63920 * include/bits/regex_executor.h: Make _M_begin non const

Re: [PATCH, 8/8] Do simple omp lowering for no address taken var

2014-11-24 Thread Tom de Vries
On 17-11-14 11:13, Richard Biener wrote: On Sat, 15 Nov 2014, Tom de Vries wrote: >On 15-11-14 13:14, Tom de Vries wrote: > >Hi, > > > >I'm submitting a patch series with initial support for the oacc kernels > >directive. > > > >The patch series uses pass_parallelize_loops to implement paralle

Re: [PATCH, 8/8] Do simple omp lowering for no address taken var

2014-11-24 Thread Tom de Vries
On 24-11-14 12:28, Tom de Vries wrote: On 17-11-14 11:13, Richard Biener wrote: On Sat, 15 Nov 2014, Tom de Vries wrote: >On 15-11-14 13:14, Tom de Vries wrote: > >Hi, > > > >I'm submitting a patch series with initial support for the oacc kernels > >directive. > > > >The patch series uses pas

Re: [RFC] First steps towards segregating types.

2014-11-24 Thread Richard Biener
On Sat, Nov 22, 2014 at 12:49 AM, Joseph Myers wrote: > On Fri, 21 Nov 2014, Andrew MacLeod wrote: > >> The biggest issue is what to do with fields which can be either a type or a >> tree... ie TREE_VALUE() of a TREE_LIST can be a type, as can a TREE_VEC >> element or a DECL_CONTEXT. I think

Re: [PATCH] Work around in-tree gmp configure problems

2014-11-24 Thread Richard Biener
On Sat, Nov 22, 2014 at 4:34 PM, Jan-Benedict Glaw wrote: > Hi! > > On Sat, 2014-11-22 14:33:29 +0100, Bernd Edlinger > wrote: >> since r217627 we use an updated AutoMake "missing" script. However >> that revealed a hidden bug in gmp-4.3.2's (up to gmp-6.0.0a) >> configure script. That is: an

Re: [PATCH] Improve shrink wrap for lo_sum

2014-11-24 Thread Richard Biener
On Sun, Nov 23, 2014 at 5:09 AM, Andrew Pinski wrote: > Hi, > As discussed at > https://gcc.gnu.org/ml/gcc-patches/2014-10/msg00997.html, this is the > one line patch which improves shrink wrapping for lo_sum. > > OK? Bootstrapped and tested on x86_64 and built and tested for > aarch64-elf with

Re: [PATCH/TopLevel] Fix compiling libgo with a combined sources

2014-11-24 Thread Richard Biener
On Sun, Nov 23, 2014 at 5:20 AM, Andrew Pinski wrote: > Hi, > The problem here is here is that OBJCOPY is not being set to the > newly built objcopy when compiling libgo. This patch adds > OBJCOPY_FOR_TARGET to the toplevel configure/Makefile so that when > libgo is compiled OBJCOPY is set to O

Re: [RFC,PATCH,testsuite] Set gcc_force_conventional_output even without LTO linker plugin

2014-11-24 Thread Richard Biener
On Sun, Nov 23, 2014 at 10:15 AM, Matthew Fortune wrote: > Hi, > > I have had to use this patch several times when performing cross testing > but I'm not sure if it is the right fix. The MIPS target testsuite runs > all tests regardless of the current compiler configuration and downgrades > run te

Re: [PATCH 2/3] move gimple_canonical_types htab out of gc memory

2014-11-24 Thread Richard Biener
On Mon, Nov 24, 2014 at 5:11 AM, wrote: > From: Trevor Saunders > > Hi, > > The memory allocated for the hash table is never rooted by anything, so > keeping > it in gc memory only works if it never needs to survive a gc. Ok. Thanks, Richard. > Trev > > lto/ > > * lto.c (read_cgraph_

Re: [PATCH, 8/8] Do simple omp lowering for no address taken var

2014-11-24 Thread Richard Biener
On Mon, 24 Nov 2014, Tom de Vries wrote: > On 17-11-14 11:13, Richard Biener wrote: > > On Sat, 15 Nov 2014, Tom de Vries wrote: > > > > > >On 15-11-14 13:14, Tom de Vries wrote: > > > > > >Hi, > > > > > > > > > > > >I'm submitting a patch series with initial support for the oacc > > > > kernels

Re: [PATCH, 8/8] Do simple omp lowering for no address taken var

2014-11-24 Thread Richard Biener
On Mon, 24 Nov 2014, Tom de Vries wrote: > On 24-11-14 12:28, Tom de Vries wrote: > > On 17-11-14 11:13, Richard Biener wrote: > > > On Sat, 15 Nov 2014, Tom de Vries wrote: > > > > > > > >On 15-11-14 13:14, Tom de Vries wrote: > > > > > > >Hi, > > > > > > > > > > > > > >I'm submitting a patch se

[PATCH] Fix PR64044

2014-11-24 Thread Richard Biener
The following fixes an issue I found when more aggressively constant-folding from static initializers. The Java frontend fails to provide an initializer for the classdollar field it creates but nevertheless marks them with TREE_READONLY whilst being static. Thus the following patch removes sett

RE: [RFC,PATCH,testsuite] Set gcc_force_conventional_output even without LTO linker plugin

2014-11-24 Thread Matthew Fortune
Richard Biener writes: > On Sun, Nov 23, 2014 at 10:15 AM, Matthew Fortune > wrote: > > Hi, > > > > I have had to use this patch several times when performing cross testing > > but I'm not sure if it is the right fix. The MIPS target testsuite runs > > all tests regardless of the current compiler

Re: [RFC,PATCH,testsuite] Set gcc_force_conventional_output even without LTO linker plugin

2014-11-24 Thread Richard Biener
On Mon, Nov 24, 2014 at 1:40 PM, Matthew Fortune wrote: > Richard Biener writes: >> On Sun, Nov 23, 2014 at 10:15 AM, Matthew Fortune >> wrote: >> > Hi, >> > >> > I have had to use this patch several times when performing cross testing >> > but I'm not sure if it is the right fix. The MIPS targe

Re: [PATCH] Fix PR64044

2014-11-24 Thread Andrew Haley
On 11/24/2014 12:29 PM, Richard Biener wrote: > > The following fixes an issue I found when more aggressively > constant-folding from static initializers. The Java frontend > fails to provide an initializer for the classdollar field > it creates but nevertheless marks them with TREE_READONLY > w

Re: C++ PATCH for c++/63942 (mangling compat alias problems)

2014-11-24 Thread Jason Merrill
On 11/21/2014 04:32 PM, Jason Merrill wrote: In this testcase, the wrong old mangling for one constructor matches the correct mangling for the other constructor, and the testcase was failing to compile as a result. But compatibility aliases should not be treated as conflicting with real declarat

Re: [C++ Patch] PR 63203

2014-11-24 Thread Jason Merrill
OK. Jason

Re: [PATCH] Work around in-tree gmp configure problems

2014-11-24 Thread Jan-Benedict Glaw
Hi Richard, On Mon, 2014-11-24 12:53:11 +0100, Richard Biener wrote: > On Sat, Nov 22, 2014 at 4:34 PM, Jan-Benedict Glaw wrote: > > On Sat, 2014-11-22 14:33:29 +0100, Bernd Edlinger > > wrote: > > > since r217627 we use an updated AutoMake "missing" script. However > > > that revealed a hid

Re: [PATCH/TopLevel] Fix compiling libgo with a combined sources

2014-11-24 Thread Jan-Benedict Glaw
Hi Andrew, On Sat, 2014-11-22 20:20:08 -0800, Andrew Pinski wrote: > Hi, > The problem here is here is that OBJCOPY is not being set to the > newly built objcopy when compiling libgo. This patch adds > OBJCOPY_FOR_TARGET to the toplevel configure/Makefile so that when > libgo is compiled OBJCO

Re: [Patch, Fortran] Remove gfc_fatal_error_1

2014-11-24 Thread H.J. Lu
On Sun, Nov 23, 2014 at 12:21 PM, Tobias Burnus wrote: > This patch depends on Manuel's %L patch. > > It converts the remaining fatal error calls to the common diagnostic > infrastructure and retires (removes) the old gfc_fatal_error_1. > > I will build and regtest it after Manuel's commit. > OK,

[Patch, Fortran] Move gfc_internal_error to common diagnostics

2014-11-24 Thread Tobias Burnus
Dear all, this patch uses the common diagnostic code for gfc_internal_error; the change is essentially only in error.c, but I additionally changed: * Use gfc_fatal_error for the user-reachable message: if (gfc_option.flag_allow_leading_underscore && letter == '_') gfc_internal_error ("Opt

RE: [PATCH] AArch64: Add TARGET_SCHED_REASSOCIATION_WIDTH

2014-11-24 Thread Wilco Dijkstra
> Richard Earnshaw wrote: > If all cores seem to benefit from FP reassociation set to 4, then it > seems odd that 4 is not also the default for generic. > > Andrew, you may need to pick a target-specific value for ThunderX; I > think Wilco has just picked something that seems plausible because he

[PATCH] check_GNU_style.sh "80 characters exceeded" error fix

2014-11-24 Thread Mantas Mikaitis
check_GNU_style.sh error "Lines should not exceed 80 characters" does not return the file name and line number where error is present, only the line of code. Whereas other kind of errors return full information. This small patch will fix this and make check_GNU_style.sh return full information

[C++ PATCH] Detect UB in shifts in constexpr functions

2014-11-24 Thread Marek Polacek
Constant expressions can't contain undefined behavior, which means that using an expression containing UB in a context requiring a constant expression makes the program invalid and we're required to diagnose that. We fail to diagnose UB if using shifts, because fold_binary_loc returns zero for a sh

RE: [PATCH] Work around in-tree gmp configure problems

2014-11-24 Thread Bernd Edlinger
Hi Richard, On Mon, 24 Nov 2014 14:11:19 +0100, Jan-Benedict Glaw wrote: > > Hi Richard, > > On Mon, 2014-11-24 12:53:11 +0100, Richard Biener > wrote: >> On Sat, Nov 22, 2014 at 4:34 PM, Jan-Benedict Glaw wrote: >>> On Sat, 2014-11-22 14:33:29 +0100, Bernd Edlinger >>> wrote: since r2

Re: [PATCH, MPX runtime 1/2] Integrate MPX runtime library

2014-11-24 Thread Ilya Enkovich
On 21 Nov 23:20, Joseph Myers wrote: > On Fri, 21 Nov 2014, Ilya Enkovich wrote: > > > +# Disable libmpx on unsupported systems. > > +if test -d ${srcdir}/libmpx; then > > +if test x$enable_libmpx = x; then > > + AC_MSG_CHECKING([for libmpx support]) > > + if (srcdir=${srcdir}/libm

Re: [PATCH, MPX wrappers 1/3] Add MPX wrappers library

2014-11-24 Thread Ilya Enkovich
On 21 Nov 23:32, Joseph Myers wrote: > On Fri, 21 Nov 2014, Ilya Enkovich wrote: > > > * c-family/c.opt (static-libmpxwrappers): New. > > New options need documenting in invoke.texi. This includes driver > options. > I'll include it into patch with all checker options documented bacause of

Re: [PATCH, MPX wrappers 2/3] Replace some function calls with wrapper calls during instrumentation

2014-11-24 Thread Ilya Enkovich
On 18 Nov 19:42, Ilya Enkovich wrote: > On 18 Nov 16:23, Joseph Myers wrote: > > On Tue, 18 Nov 2014, Ilya Enkovich wrote: > > > > > +@item -fcheck-pointer-bounds > > > +@opindex fcheck-pointer-bounds > > > +@opindex fno-check-pointer-bounds > > > +Enable Pointer Bounds Checker instrumentation. E

Re: [PATCH RFC]Pair load store instructions using a generic scheduling fusion pass

2014-11-24 Thread James Greenhalgh
On Fri, Nov 14, 2014 at 02:43:12AM +, Bin.Cheng wrote: > On Fri, Nov 7, 2014 at 7:13 AM, Jeff Law wrote: > > On 11/05/14 02:30, Bin.Cheng wrote: > >> Thanks very much for reviewing. I refined the patch according to your > >> comments. Also made two small changes: a) skip breaking dependency

[C PATCH] Fix PR63877

2014-11-24 Thread Marek Polacek
This PR is basically the same as PR54113, except this time it's about -Wmissing-declarations and not about -Wmissing-prototypes. The problem here is that we were emitting bogus warning for a correct use of an inline function. Thus fixed in the same way as PR54113, that is, don't warn for inline f

Re: [PATCH] Improve i?86 address delegitimization after 32-bit pic changes (PR target/63542)

2014-11-24 Thread H.J. Lu
On Tue, Oct 21, 2014 at 9:03 AM, Jakub Jelinek wrote: > On Mon, Sep 29, 2014 at 01:08:56PM +0200, Jakub Jelinek wrote: >> I wonder if during/after reload we just couldn't look at >> ORIGINAL_REGNO of hard regs if ix86_use_pseudo_pic_reg. Or is that >> the other case, where you don't have any PIC

Re: [RS6000] Fix -mlongcall with nested functions on AIX

2014-11-24 Thread David Edelsohn
[Please explicitly copy me on relevant patches] 2014-10-17 Eric Botcazou * config/rs6000/rs6000.c (rs6000_call_aix): For the AIX ABI, do not load the static chain if the call was originally direct. 2014-10-17 Eric Botcazou * gcc.target/powerpc/longcall-2.c: New test. Okay. Thanks, Davi

Re: [C++ PATCH] Detect UB in shifts in constexpr functions

2014-11-24 Thread Jakub Jelinek
On Mon, Nov 24, 2014 at 02:51:14PM +0100, Marek Polacek wrote: > --- gcc/cp/constexpr.c > +++ gcc/cp/constexpr.c > @@ -1451,6 +1451,43 @@ verify_constant (tree t, bool allow_non_constant, bool > *non_constant_p, >return *non_constant_p; > } > > +/* Return true if the shift operation on LHS

Re: [PING**4] [PATCH] Fix PR ipa/61190, 2nd edition�??

2014-11-24 Thread Jan Hubicka
+cgraph_node::call_for_symbol_thunks_and_aliases_1 (bool (*callback) +(cgraph_node *, void *), + void *data, + bool include_overwritable, +

Re: [C++ PATCH] Detect UB in shifts in constexpr functions

2014-11-24 Thread Marek Polacek
On Mon, Nov 24, 2014 at 04:49:25PM +0100, Jakub Jelinek wrote: > On Mon, Nov 24, 2014 at 02:51:14PM +0100, Marek Polacek wrote: > > --- gcc/cp/constexpr.c > > +++ gcc/cp/constexpr.c > > @@ -1451,6 +1451,43 @@ verify_constant (tree t, bool allow_non_constant, > > bool *non_constant_p, > >return

Re: [C++ PATCH] Detect UB in shifts in constexpr functions

2014-11-24 Thread Jakub Jelinek
On Mon, Nov 24, 2014 at 04:58:22PM +0100, Marek Polacek wrote: > > Consider say: > > > > constexpr int p = 1; > > constexpr int foo (int a) > > { > > return a << (int) &p; > > } > > constexpr int bar (int a) > > { > > return ((int) &p) << a; > > } > > constexpr int q = foo (5); > > constexpr i

Improve inliner's handling of aliases

2014-11-24 Thread Jan Hubicka
Hi, the original update of inliner for aliases contined some lazyness. This patch fixes I think last remainder of it where inliner did not remove symbol with aliases on it. This is more important now when ICF uses aliases quite often. Bootstrapped/regtested x86_64-linux, comitted. PR ipa

Re: Track dynamic type changes

2014-11-24 Thread H.J. Lu
On Sun, Oct 5, 2014 at 9:34 AM, Jan Hubicka wrote: > Hi, > this patch makes ipa_polymorphic_call_context::get_dynamic_type to > track if vptr has changed in between function entry and polymorphic > call. > > The patch also add logic skipping if (a==0) checks in multiple > inheritance so I can cons

Re: [C++ PATCH] Detect UB in shifts in constexpr functions

2014-11-24 Thread Marek Polacek
On Mon, Nov 24, 2014 at 05:05:08PM +0100, Jakub Jelinek wrote: > On Mon, Nov 24, 2014 at 04:58:22PM +0100, Marek Polacek wrote: > > > Consider say: > > > > > > constexpr int p = 1; > > > constexpr int foo (int a) > > > { > > > return a << (int) &p; > > > } > > > constexpr int bar (int a) > > > {

Re: Merging configure bits with Binutils

2014-11-24 Thread Jeff Law
On 11/23/14 00:31, Jan-Benedict Glaw wrote: Hi! This is my current patch series for GCC to merge all the config bits from Binutils. It's not yet testet, but I'd like to post it early to get feedback. That way, I hope to not forget something obvious. FWIW, one could easily argue this should be a

Re: [PATCH RFC 1/4] Merge ./configure.ac with Binutils

2014-11-24 Thread Jeff Law
On 11/23/14 00:33, Jan-Benedict Glaw wrote: GCC and Binutils had a common ./configure.ac after this commit: commit 08d081652f50df83f7e9768f8dbb7a99b0df50a2 Author: sandra Date: Wed May 14 23:20:59 2014 + 2014-05-14 Sandra Loosemore * co

Re: [PATCH RFC 3/4] Merge the ./config directory between GCC and Binutils

2014-11-24 Thread Jeff Law
On 11/23/14 00:33, Jan-Benedict Glaw wrote: This pulls in the missing Binutils pieces into the ./config directory. Contains these missing Binutils changes: 2014-08-14 Alan Modra * plugins.m4: Test for dlfcn.h or windows.h here to set default for --enable-plugins. Report erro

Re: [PATCH] Fix regressions in libgomp testsuite: set flag_fat_lto_objects for offload

2014-11-24 Thread Ilya Verbin
On 17 Nov 10:57, Richard Biener wrote: > On Fri, Nov 14, 2014 at 6:08 PM, Ilya Verbin wrote: > > On 14 Nov 09:01, H.J. Lu wrote: > >> On Fri, Nov 14, 2014 at 8:51 AM, Ilya Verbin wrote: > >> > On 14 Nov 08:46, H.J. Lu wrote: > >> >> What happens when -flto is used on command line? Will we > >> >

Re: [PATCH RFC 4/4] Regenerate ./configure

2014-11-24 Thread Jeff Law
On 11/23/14 00:33, Jan-Benedict Glaw wrote: 2014-11-22 Jan-Benedict Glaw * configure: Regenerate. OK. jeff

Re: [PATCH 1/3] remove param_is from nvptx

2014-11-24 Thread Jeff Law
On 11/23/14 21:11, tsaund...@mozilla.com wrote: From: Trevor Saunders Hi, $subject Trev gcc/ * config/nvptx/nvptx.c: Convert htabs to hash_table. This is fine and I think can go in now rather than waiting for the next stage1. It's an obvious cleanup and one could argue I should ha

Re: [patch] Define new std::ios_base::failure with abi_tag("cxx11")

2014-11-24 Thread Tom de Vries
On 14-11-14 13:18, Jonathan Wakely wrote: This adds system_error support to iostreams, including the required base class changes to std::ios_base::failure. The abi_tag is used to make it a distinct type. This changes the type of I/O exceptions thrown by the library but exceptions are very rarely

Re: Merging configure bits with Binutils

2014-11-24 Thread Jan-Benedict Glaw
On Mon, 2014-11-24 09:42:57 -0700, Jeff Law wrote: > On 11/23/14 00:31, Jan-Benedict Glaw wrote: > > This is my current patch series for GCC to merge all the config > > bits from Binutils. It's not yet testet, but I'd like to post it > > early to get feedback. That way, I hope to not forget someth

Re: [PING][PATCH] Change contrib/test_installed for testing cross compilers

2014-11-24 Thread Alan Lawrence
Having just been experimenting with testing of installed compilers - yes something like this could be useful, however: to do cross-testing I found I also (a) had to set my target_list; so either an extra flag for that, or maybe just a generic 'extra_site_flags' parameter? (b) I had to set up som

Re: [PATCH 3/3] remove gengtype support for param_is use_param, if_marked and splay tree allocators

2014-11-24 Thread Jeff Law
On 11/23/14 21:11, tsaund...@mozilla.com wrote: From: Trevor Saunders Hi, The one interesting part of this is what we do about plugins. As you can tell from the diff removing support for these gengtype features makes things much simpler, and I think the only way to do that is to remove the pl

[patch] Restore bootstrap on powerpc*-apple-darwin*

2014-11-24 Thread FX
Bootstrap is currently broken on ppc-darwin due to PR63703 (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63703). This is also broken on 4.9, and actually made it into the 4.9.2 release. This patch fixes it, restores bootstrap (well, it gets cc1 to build, at least… I don’t have a ppc to test the

Pushing recent libtool fix to binutils-gdb and newlib/libgloss

2014-11-24 Thread FX
*ping* Didn’t get any response to my question so far… >> Thanks everyone for the comments and review. >> Committed as r217366 > > I cannot push the change to binutils-gdb as I don’t have write access there. > Also, Joseph Myers said I needed to commit to newlib/libgloss, but their > webpage on

Go patch committed: Don't deref named pointer type for method

2014-11-24 Thread Ian Lance Taylor
This patch by Chris Manghane fixes the Go compiler to not dereference a named pointer type when looking up a method. Before this patch that could happen if the pointer type pointed to a struct with an inherited method. This is http://golang.org/issue/9018. Bootstrapped and ran testsuite on x86_6

Re: [patch] Define new std::ios_base::failure with abi_tag("cxx11")

2014-11-24 Thread Jonathan Wakely
On 24/11/14 17:48 +0100, Tom de Vries wrote: On 14-11-14 13:18, Jonathan Wakely wrote: This adds system_error support to iostreams, including the required base class changes to std::ios_base::failure. The abi_tag is used to make it a distinct type. This changes the type of I/O exceptions thrown

Re: Pushing recent libtool fix to binutils-gdb and newlib/libgloss

2014-11-24 Thread H.J. Lu
Done: https://sourceware.org/ml/binutils/2014-11/msg00318.html On Mon, Nov 24, 2014 at 9:04 AM, FX wrote: > *ping* > > Didn’t get any response to my question so far… > > >>> Thanks everyone for the comments and review. >>> Committed as r217366 >> >> I cannot push the change to binutils-gdb as I

Re: [PATCH] Don't unilaterally include when checking for fpsetmask

2014-11-24 Thread John David Anglin
On 8/7/2014 4:22 PM, John David Anglin wrote: The attached change fixes the build of libgfortran on hppa1.1-hp-hpux10.20 (I know I'm going for the oldest system that will build gfortran). OK? Ping. Dave -- John David Anglindave.ang...@bell.net

Re: [patch] Define new std::ios_base::failure with abi_tag("cxx11")

2014-11-24 Thread Tom de Vries
On 24-11-14 18:12, Jonathan Wakely wrote: On 24/11/14 17:48 +0100, Tom de Vries wrote: On 14-11-14 13:18, Jonathan Wakely wrote: This adds system_error support to iostreams, including the required base class changes to std::ios_base::failure. The abi_tag is used to make it a distinct type. Thi

[PATCH/AARCH64] make bswap vector consistent with scalar name

2014-11-24 Thread Andrew Pinski
I had some local patches in my tree which adds a bswap tree code. This breaks the aarch64 back-end vectorizing of byteswaps as we use the standard mechanism to see if a tree code vectorizes (optabs). Since it make sense to have consistent of the pattern names between the vector version and the scal

Re: [PATCH] Don't unilaterally include when checking for fpsetmask

2014-11-24 Thread FX
>> The attached change fixes the build of libgfortran on hppa1.1-hp-hpux10.20 >> (I know I'm going >> for the oldest system that will build gfortran). OK

Re: Pushing recent libtool fix to binutils-gdb and newlib/libgloss

2014-11-24 Thread FX
> Done: > https://sourceware.org/ml/binutils/2014-11/msg00318.html Thanks!

Re: [PATCH, PR 63551] Use proper type in evaluate_conditions_for_known_args

2014-11-24 Thread Martin Jambor
On Sat, Nov 22, 2014 at 07:36:59PM +0100, Richard Biener wrote: > On November 22, 2014 12:45:58 PM CET, Jakub Jelinek wrote: > >On Sat, Nov 22, 2014 at 12:09:46PM +0100, Martin Jambor wrote: > >> 2014-11-21 Martin Jambor > >> > >>PR ipa/63551 > >>* ipa-inline-analysis.c (evaluate_condi

Re: [PATCH] [AArch64, RTL] Bics instruction generation for aarch64

2014-11-24 Thread Alex Velenko
On 11/11/14 10:38, Alex Velenko wrote: From 98bb6d7323ce79e28be8ef892b919391ed857e1f Mon Sep 17 00:00:00 2001 From: Alex Velenko Date: Fri, 31 Oct 2014 18:43:32 + Subject: [PATCH] [AArch64, RTL] Bics instruction generation for aarch64 Hi, This patch adds rtl patterns for aarch64 to genera

Re: [PATCH, 8/8] Do simple omp lowering for no address taken var

2014-11-24 Thread Tom de Vries
On 24-11-14 13:12, Richard Biener wrote: On Mon, 24 Nov 2014, Tom de Vries wrote: On 24-11-14 12:28, Tom de Vries wrote: On 17-11-14 11:13, Richard Biener wrote: On Sat, 15 Nov 2014, Tom de Vries wrote: On 15-11-14 13:14, Tom de Vries wrote: Hi, I'm submitting a patch series with initial

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-24 Thread Paolo Carlini
Hi, On 11/12/2014 03:02 AM, H.J. Lu wrote: I am checking in this. Looks like devirt-42.C is failing again and reverting your tweak would fix it?!? Thanks, Paolo.

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-24 Thread Paolo Carlini
Hi again, On 11/24/2014 07:33 PM, Paolo Carlini wrote: Hi, On 11/12/2014 03:02 AM, H.J. Lu wrote: I am checking in this. Looks like devirt-42.C is failing again and reverting your tweak would fix it?!? Sorry, in fact now the line failing is "First type is base of second" 3, a different one.

Re: [PATCH] Fix dump scan in test devirt-40.C

2014-11-24 Thread H.J. Lu
On Mon, Nov 24, 2014 at 10:41 AM, Paolo Carlini wrote: > Hi again, > > > On 11/24/2014 07:33 PM, Paolo Carlini wrote: >> >> Hi, >> >> On 11/12/2014 03:02 AM, H.J. Lu wrote: >>> >>> I am checking in this. >> >> Looks like devirt-42.C is failing again and reverting your tweak would fix >> it?!? > >

[C++ Patch/RFC] PR 63757

2014-11-24 Thread Paolo Carlini
Hi, in this rejects-valid, as part of build_user_type_conversion_1, standard_conversion is called by implicit_conversion with a *null* expr, thus the condition in standard_conversion /* [conv.ptr] A null pointer constant can be converted to a pointer type; ... A null pointer const

Re: [PATCH 9/9] ipa-inline uses sreal for as fibonacci_heap template argument.

2014-11-24 Thread H.J. Lu
On Thu, Nov 13, 2014 at 12:10 PM, mliska wrote: > gcc/ChangeLog: > > 2014-11-13 Martin Liska > > * ipa-inline.c (edge_badness): long is replaced by sreal > as fibonacci_heap template type. > (update_edge_key): Likewise. > (inline_small_functions): Likewise. This

Re: [patch] Restore bootstrap on powerpc*-apple-darwin*

2014-11-24 Thread Iain Sandoe
On 24 Nov 2014, at 17:02, FX wrote: > Bootstrap is currently broken on ppc-darwin due to PR63703 > (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=63703). > This is also broken on 4.9, and actually made it into the 4.9.2 release. > > This patch fixes it, restores bootstrap (well, it gets cc1 to b

Re: Merging configure bits with Binutils

2014-11-24 Thread Jeff Law
On 11/24/14 09:52, Jan-Benedict Glaw wrote: On Mon, 2014-11-24 09:42:57 -0700, Jeff Law wrote: On 11/23/14 00:31, Jan-Benedict Glaw wrote: This is my current patch series for GCC to merge all the config bits from Binutils. It's not yet testet, but I'd like to post it early to get feedback. Tha

Re: [PATCH, PR 63551] Use proper type in evaluate_conditions_for_known_args

2014-11-24 Thread Richard Biener
On November 24, 2014 7:12:01 PM CET, Martin Jambor wrote: >On Sat, Nov 22, 2014 at 07:36:59PM +0100, Richard Biener wrote: >> On November 22, 2014 12:45:58 PM CET, Jakub Jelinek > wrote: >> >On Sat, Nov 22, 2014 at 12:09:46PM +0100, Martin Jambor wrote: >> >> 2014-11-21 Martin Jambor >> >> >>

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Jeff Law
On 11/23/14 15:22, Sebastian Pop wrote: The second patch attached limits the search for FSM jump threads to loops. With that patch, we are now down to 470 jump threads in an x86_64-linux bootstrap (and 424 jump threads on powerpc64-linux bootstrap.) Yea, that was one of the things I was going t

Re: [Android] Enable ifuncs on Android

2014-11-24 Thread Jeff Law
On 11/24/14 02:49, Alexander Ivchenko wrote: *ping* thanks, Alexander Can you put a reference to the actual patch you want to ping (or just attach the patch you want to ping)? I'll review given the updated context/changes from HJ. jeff

Re: [PATCH, ciklplus]: Use -ffloat-store for 32bit x86 in cilk-plus/AN/builtin_fn_{custom,mutating}.c

2014-11-24 Thread Jeff Law
On 11/22/14 11:50, Uros Bizjak wrote: Hello! These two tests fix PR target/63847 [1], where x87 excess precision causes testcase to fail. The problem was triggered by -fpic, please see the PR for analysis. The patch adds -ffloat-store for 32bit x86 target, a standard and well tested solution fo

Re: [PATCH/TopLevel] Fix compiling libgo with a combined sources

2014-11-24 Thread Jeff Law
On 11/22/14 21:20, Andrew Pinski wrote: Hi, The problem here is here is that OBJCOPY is not being set to the newly built objcopy when compiling libgo. This patch adds OBJCOPY_FOR_TARGET to the toplevel configure/Makefile so that when libgo is compiled OBJCOPY is set to OBJCOPY_FOR_TARGET. I

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Sebastian Pop
Jeff Law wrote: > On 11/23/14 15:22, Sebastian Pop wrote: > >The second patch attached limits the search for FSM jump threads to loops. > >With > >that patch, we are now down to 470 jump threads in an x86_64-linux bootstrap > >(and 424 jump threads on powerpc64-linux bootstrap.) > > > Yea, that w

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Sebastian Pop
Sebastian Pop wrote: > > Using a SEME region copier is also something I really wanted to do > > long term. In fact, I believe a lot of tree-ssa-threadupdate.c > > ought to be ripped out and replaced with a SEME based copier. > > I did an experiment around these lines over the week-end, and now th

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Jeff Law
On 11/24/14 15:05, Sebastian Pop wrote: I did an experiment around these lines over the week-end, and now that you mention it, I feel less shy to speak about; well the patch does not yet pass bootstrap, and there still are about 20 failing test-cases. I feel better reading the code generation p

Re: PR 13631 Problems in messages

2014-11-24 Thread François Dumont
On 24/11/2014 01:23, Jonathan Wakely wrote: On 24/11/14 00:13 +0100, François Dumont wrote: Hello As we are at doing some evolution in the ABI I would like to take the opportunity to merge branch libstdcxx_so_7-2. The first fix was I don't think we want to merge everything, but it's cert

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Jeff Law
On 11/23/14 15:22, Sebastian Pop wrote: Jeff Law wrote: PS: I have run some perf analysis with the patch: - on a bootstrap of GCC I see 3209 FSM jump threads - libpng and libjpeg contain FSM jump threads, the perf increase is in the 1% (measured on simulators and reduced data sets) - coremark

[committed] Fix !$omp atomic handling (PR fortran/63938)

2014-11-24 Thread Jakub Jelinek
Hi! This fixes Fortran !$omp atomic handling to match C/C++, goa_lhs_expr_p is not able to find out matches of more complex expressions after tree unsharing, so this patch helps it find what it needs. Bootstrapped/regtested on x86_64-linux and i686-linux, committed to trunk. Will backport soon.

[PATCH] Fix linemap_line_start (PR preprocessor/60436)

2014-11-24 Thread Jakub Jelinek
Hi! As mentioned in the PR, when preprocessing very large files, if there are huge numbers of lines where no #line is emitted, we might not detect overflowinging into adhoc locations. Apparently in the add_map case we already handle that fine, by first stopping tracking columns and after another 2

[PATCH] Fix find_base_term in 32-bit -fpic code (PR lto/64025)

2014-11-24 Thread Jakub Jelinek
Hi! The fallback delegitimization I've added as last option mainly for debug info purposes, when we don't know if the base is a PIC register or say a PIC register plus some addend, unfortunately in some tests broke find_base_term, which for PLUS looks only at the first operand and recursion on it

Re: [Patch, Fortran] Remove gfc_fatal_error_1

2014-11-24 Thread Tobias Burnus
Tobias Burnus wrote: It converts the remaining fatal error calls to the common diagnostic infrastructure and retires (removes) the old gfc_fatal_error_1. I will build and regtest it after Manuel's commit. OK, when it succeeds? The patch needed additionally a '...' added in two test-suite file

[PATCH] Fix building of gengtype

2014-11-24 Thread Jakub Jelinek
Hi! My last 2 bootstraps failed, both because of a race while building host gengtype (each time different gengtype*.o). Looking at the history, we have two versions of gengtype, one in build/, another one in . (host one). Initially, the copy in build/ was built with -DGENERATOR_FILE, included bco

[committed] Replace abort instruction with __builtin_trap() in config/pa/linux_atomic.c

2014-11-24 Thread John David Anglin
The attached change uses __builtin_trap() instead of an asm as the compiler can better optimize code using __builtin_trap. Tested on hppa-unknown-linux-gnu. Committed to trunk. Dave -- John David Anglin dave.ang...@bell.net 2014-11-24 John David Anglin * config/pa/linux-at

Re: [Patch] Improving jump-thread pass for PR 54742

2014-11-24 Thread Sebastian Pop
Jeff Law wrote: > On 11/23/14 15:22, Sebastian Pop wrote: > >Jeff Law wrote: > >>>PS: I have run some perf analysis with the patch: > >>>- on a bootstrap of GCC I see 3209 FSM jump threads > >>>- libpng and libjpeg contain FSM jump threads, the perf increase is in the > >>>1% > >>> (measured on

  1   2   >