Re: [ping] pending patches

2016-01-19 Thread Eric Botcazou
> 2016-01-05 Eric Botcazou > > * dwarf2out.c (need_endianity_attribute_p): New inline predicate. > (base_type_die): Add REVERSE parameter and attach DW_AT_endianity to > the DIE accordingly. > (modified_type_die): Add REVERSE parameter and pass it recursively, > as

Re: [PATCH][RFC][Offloading] Fix PR68463

2016-01-19 Thread Richard Biener
On Mon, 18 Jan 2016, Ilya Verbin wrote: > On Fri, Jan 15, 2016 at 09:15:01 +0100, Richard Biener wrote: > > On Fri, 15 Jan 2016, Ilya Verbin wrote: > > > II) The __offload_func_table, __offload_funcs_end, __offload_var_table, > > > __offload_vars_end are now provided by the linker script, instead

Re: [PATCH] Fix ICE with asm "m" (stmt-expr) operand (PR middle-end/67653)

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Jakub Jelinek wrote: > Hi! > > Here is an attempt to fix ICE on statement expression in "m" asm input > operand. The problem is that gimplify_asm_expr attempts to mark it > addressable, but that can be just too late, a temporary the stmt-expression > gimplifies to might not

[PATCH] Fix PR69336

2016-01-19 Thread Richard Biener
The following patch enhances the recent change to DOMs memory reference value-numbering to cover PR69336 (all handled components instead of just ones with outermost ARRAY_REF). Bootstrapped and tested on x86_64-unknown-linux-gnu. I'm waiting until I committed the fix for PR69352. Richard. 2016

[PATCH] Fix PR69352

2016-01-19 Thread Richard Biener
The following should fix PR69352 where hash collisions make equal_mem_array_ref_p get refs it wasn't supposed to compare and will not compare correctly (those with variable indices). Profiledbootstrap and testing running on x86_64-unknown-linux-gnu. Richard. 2016-01-19 Richard Biener

Re: [PATCH] Fix PR69352

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Richard Biener wrote: > > The following should fix PR69352 where hash collisions make > equal_mem_array_ref_p get refs it wasn't supposed to compare > and will not compare correctly (those with variable indices). > > Profiledbootstrap and testing running on x86_64-unknown-li

Re: [PATCH][RFC][Offloading] Fix PR68463

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 09:57:01AM +0100, Richard Biener wrote: > On Mon, 18 Jan 2016, Ilya Verbin wrote: > > > On Fri, Jan 15, 2016 at 09:15:01 +0100, Richard Biener wrote: > > > On Fri, 15 Jan 2016, Ilya Verbin wrote: > > > > II) The __offload_func_table, __offload_funcs_end, __offload_var_table

Re: [PING] genattrab.c generate switch

2016-01-19 Thread Richard Biener
On Mon, Jan 18, 2016 at 7:48 PM, Jeff Law wrote: > On 01/18/2016 07:09 AM, Jesper Broge Jørgensen wrote: >> >> Ping patch: >> >> https://gcc.gnu.org/ml/gcc-patches/2016-01/msg00784.html > > I'd put it in my gcc-7 queue. But if Richard, Bernd, Richi or someone else > wants to work though the chang

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-19 Thread Christophe Lyon
On 19 January 2016 at 04:05, H.J. Lu wrote: > On Thu, Dec 24, 2015 at 3:55 AM, Alan Lawrence wrote: >> This version changes the test cases to fix failures on some platforms, by >> rewriting the initializers so that they aren't pushed out to the constant >> pool. >> >> gcc/ChangeLog: >> >>

Re: -z bndplt documentation in GCC manual

2016-01-19 Thread Ilya Enkovich
2016-01-19 5:25 GMT+03:00 Sandra Loosemore : > I think the documentation relating to '-z bndplt' in the GCC manual > description of -fcheck-pointer-bounds is incorrect. It looks like, as of > r225862, the GCC driver is supposed to emit an error message if GCC was > configured with a linker that do

Re: [PATCH, testsuite]: Disable LTO for gcc.c-torture/execute/builtins/{memops,strstr}-asm.c

2016-01-19 Thread Jakub Jelinek
On Mon, Jan 18, 2016 at 11:53:55PM +0100, Uros Bizjak wrote: > Hello! > > As explained by Honza in the PR, these two tests are not suitable for > LTO tests. Also, the dg-options directives are not effective in this > directory. > > 2016-01-19 Uros Bizjak > > PR testsuite/68820 > * gcc

Re: [hsa merge 00/10] Merge of HSA branch

2016-01-19 Thread Martin Jambor
Hi, On Wed, Jan 13, 2016 at 06:39:25PM +0100, Martin Jambor wrote: > Hi, > > this is hopefully the last big re-post of the HSA patches... I have committed the combined patch as revision 232549 after bootstrapping and testing all languages on x86_64-linux and i686-linux and verifying I did not br

[PATCH, PR tree-optimization/69328] Fix vectorization of boolean vector comparision in COND_EXPR

2016-01-19 Thread Ilya Enkovich
Hi, Currently vectorizer incorrectly handles a case when COND_EXPR has boolean vector comparison. Firstly masked COND_EXPR is determined incorrectly. Also we don't check vector types of compared values are compatible. This patch fixes these problems. Bootstrapped and regtested for x86_64-pc-lin

Remove outdated text from lto.texi

2016-01-19 Thread Kugan
Hi, lto.texi has "Currently, the linker plugin works only in combination with the Gold linker, but a GNU ld implementation is under development". I don't think this is true any more. Attached patch removes this. is this OK for trunk? Thanks, Kugan gcc/ChangeLog: 2016-01-19 Kugan Vivekanandaraj

Re: [PATCH] ARM PR68620 (ICE with FP16 on armeb)

2016-01-19 Thread Christophe Lyon
On 18 January 2016 at 20:01, Alan Lawrence wrote: > Thanks for working on this, Christophe, and sorry I missed the PR. You got > further in fixing more things than I did though :). A couple of comments: > >> For the vec_set_internal and neon_vld1_dup patterns, I >> switched to an existing iterator

Do not ICE with -fdump-ipa-cgraph when building firefox

2016-01-19 Thread Jan Hubicka
Hi, cgraph dumpnow segfaults as we have symbols without DECL_NAME set. This patch makes us to print node by asm name or as if everything fails. Honza * symtab.c (symtab_node::asm_name): Do not call printable name directly. (symtab_node::name): Report name as unnamed if DECL_NA

Re: reject decl with incomplete struct/union type in check_global_declaration()

2016-01-19 Thread Marek Polacek
Sorry for speaking up late, but I think we could do better with formatting in this patch: On Sat, Jan 16, 2016 at 03:45:22PM +0530, Prathamesh Kulkarni wrote: > diff --git a/gcc/c/c-decl.c b/gcc/c/c-decl.c > index 915376d..d36fc67 100644 > --- a/gcc/c/c-decl.c > +++ b/gcc/c/c-decl.c > @@ -4791,6 +

[chkp] Do not stream bodies of instrumentation thunks

2016-01-19 Thread Jan Hubicka
Hi, instrumentation thunks are de-facto transparent aliases and thus their bodies should not be streamed (longer term plan is to reorganize them to aliases). This avoids an ICE when mixing instrumented and non-instrumented files in LTO. Bootstrapped/regtested x86_64-linux, plan to commit it if the

Fix ICE in lto-symtab

2016-01-19 Thread Jan Hubicka
Hi, this patch fixes ICE with abstract decls. Those need not to be linked. Bootstrapped/regtested x86_64-linux, will commit it shortly. Honza * lto-symtab.c (lto_symtab_prevailing_virtual_decl): Abstract decls have no assemblernames. * g++.dg/torture/pr69136.C: New testc

Re: [Ping^3][PATCH][GCC][ARM] testcase memset-inline-10.c uses -mfloat-abi=hard but does not check whether target supports it

2016-01-19 Thread Andre Vieira (lists)
On 05/01/16 17:40, Andre Vieira wrote: On 27/11/15 14:28, Andre Vieira wrote: On 12/11/15 15:16, Andre Vieira wrote: On 12/11/15 15:08, Andre Vieira wrote: Hi, This patch changes the memset-inline-10.c testcase to make sure that it is only compiled for ARM targets that support -mfloat-abi=

Re: [PATCH] Fix RTL DSE (PR rtl-optimization/68955, take 2)

2016-01-19 Thread Eric Botcazou
> 2016-01-19 Jakub Jelinek > > PR rtl-optimization/68955 > PR rtl-optimization/64557 > * dse.c (record_store, check_mem_read_rtx): Don't call get_addr > here. Fix up formatting. > * alias.c (get_addr): Handle VALUE + CONST_INT. VALUE +/- CONST_INT (and actually a

Fix ICE in get_untransformed_body

2016-01-19 Thread Jan Hubicka
Hi, this patch fixes confusion of cgraph_node::get_untransformed_body about thunks so we do not try to stream them in twice. Honza * cgraphunit.c (cgraph_node::expand_thunk): When forcing gimple assume that the node has body. * cgraph.c (cgraph_node::get_untransformed_body

Re: [PING] genattrab.c generate switch

2016-01-19 Thread Jesper Broge Jørgensen
On 19/01/16 10:44, Richard Biener wrote: On Mon, Jan 18, 2016 at 7:48 PM, Jeff Law wrote: On 01/18/2016 07:09 AM, Jesper Broge Jørgensen wrote: Ping patch: https://gcc.gnu.org/ml/gcc-patches/2016-01/msg00784.html I'd put it in my gcc-7 queue. But if Richard, Bernd, Richi or someone else wa

[RFC] [nvptx] Try to cope with cuLaunchKernel returning CUDA_ERROR_LAUNCH_OUT_OF_RESOURCES

2016-01-19 Thread Thomas Schwinge
Hi! With nvptx offloading, in one OpenACC test case, we're running into the following fatal error (GOMP_DEBUG=1 output): [...] info: Function properties for 'LBM_performStreamCollide$_omp_fn$0': info: used 87 registers, 0 stack, 8 bytes smem, 328 bytes cmem[0], 80 bytes cmem[

Re: [PATCH][GCC][ARM] testcase memset-inline-10.c uses -mfloat-abi=hard but does not check whether target supports it

2016-01-19 Thread Ramana Radhakrishnan
On Thu, Nov 12, 2015 at 3:16 PM, Andre Vieira wrote: > On 12/11/15 15:08, Andre Vieira wrote: >> >> Hi, >> >>This patch changes the memset-inline-10.c testcase to make sure that >> it is only compiled for ARM targets that support -mfloat-abi=hard using >> the fact that all non-thumb1 targets d

Re: [PATCH] Fix RTL DSE (PR rtl-optimization/68955, take 2)

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 12:43:53PM +0100, Eric Botcazou wrote: > > 2016-01-19 Jakub Jelinek > > > > PR rtl-optimization/68955 > > PR rtl-optimization/64557 > > * dse.c (record_store, check_mem_read_rtx): Don't call get_addr > > here. Fix up formatting. > > * alias.c (get_ad

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jan Hubicka
Hi, here is updated patch. It has same effect as the former version. Bootstrapped/regtested x86_64-linux, OK? Honza * tree-ssa-live.c (remove_unused_scope_block_p): Also remove reudndant typedefs. Index: tree-ssa-live.c

Re: [PATCH 1/2] DWARF: process all TYPE_DECL nodes when iterating on scopes

2016-01-19 Thread Pierre-Marie de Rodat
On 01/18/2016 12:18 PM, Richard Biener wrote: Looking for TYPE_DECL_IS_STUB uses I come along dwarf2out_ignore_block which you'd need to change as well I think. That is true, thank you! First, I’ll answer your last point: Btw, not sure how you get at the "wrong" debug info gen order, I can't

[C++] Add -fnull-this-pointer

2016-01-19 Thread Jan Hubicka
Hi, according to Trevor, the assumption about THIS pointer being non-NULL breaks several bigger C++ packages (definitly including Firefox, but I believe kdevelop was mentioned, too). This patch makes the feature to be controlable by a dedicated flag. I am not sure about the default. We now have u

Re: [PATCH] Fix RTL DSE (PR rtl-optimization/68955, take 2)

2016-01-19 Thread Eric Botcazou
> So shall I take out the CONST_WIDE_INT/CONST_DOUBLE handling and just > check for CONST_INT_P instead of CONST_SCALAR_INT_P ? I thought it is just > easy thing to handle, though for DSE which cares about addresses it really > does not matter. Or can I leave it in? Your call. > DSE will only c

[chkp] Clear instrumented_version and thunk_info in cgraph_node::reset

2016-01-19 Thread Jan Hubicka
Hi, this patch makes the code turning instrumentation thunks into transparent aliases to work. Bootstrapped/regtested x86_64-linux, will commit it later today. Honza * cgraphunit.c (cgraph_node::reset): Clear thunk info and instrumented_version, too. Index: cgraphunit.c =

Re: genattrab.c generate switch

2016-01-19 Thread Bernd Schmidt
On 01/18/2016 11:44 PM, Jesper Broge Jørgensen wrote: I found a formatting tool called uncrustify that comes with a gnu style config https://github.com/bengardner/uncrustify/blob/master/etc/gnu-indent.cfg that needed a few tweaks to format code that looked what is already in gcc/genattrtab.c The

Re: [C++] Add -fnull-this-pointer

2016-01-19 Thread Markus Trippelsdorf
On 2016.01.19 at 13:11 +0100, Jan Hubicka wrote: > according to Trevor, the assumption about THIS pointer being non-NULL breaks > several bigger C++ packages (definitly including Firefox, but I believe > kdevelop was mentioned, too). This patch makes the feature to be controlable > by a dedicated

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Jan Hubicka wrote: > Hi, > here is updated patch. It has same effect as the former version. > > Bootstrapped/regtested x86_64-linux, OK? But what about the comment? We track no information on whether given type is used or not, so we have to keep t

Re: [PATCH] Fix debug info handling in prepare_shrink_wrap (PR debug/65779)

2016-01-19 Thread Bernd Schmidt
On 01/19/2016 12:33 AM, Jakub Jelinek wrote: + if (MAY_HAVE_DEBUG_INSNS) +{ + for (dinsn = BB_END (bb); dinsn != insn; dinsn = PREV_INSN (dinsn)) + if (DEBUG_INSN_P (dinsn)) + { + df_ref use; + FOR_EACH_INSN_USE (use, dinsn) + if (refers_to_

Re: [testsuite][ARM target attributes] Fix effective_target tests

2016-01-19 Thread Kyrill Tkachov
Hi Christophe, On 04/01/16 14:21, Christophe Lyon wrote: On 4 January 2016 at 15:20, Christophe Lyon wrote: On 18 December 2015 at 15:16, Kyrill Tkachov wrote: Hi Christophe, On 17/12/15 22:17, Christophe Lyon wrote: Hi, Here is an updated version of this patch. I did test it with -mthum

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jan Hubicka
> On Tue, 19 Jan 2016, Jan Hubicka wrote: > > > Hi, > > here is updated patch. It has same effect as the former version. > > > > Bootstrapped/regtested x86_64-linux, OK? > > But what about the comment? > > We track no > information on whether given type is used or not, so we h

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jan Hubicka
> > On Tue, 19 Jan 2016, Jan Hubicka wrote: > > > > > Hi, > > > here is updated patch. It has same effect as the former version. > > > > > > Bootstrapped/regtested x86_64-linux, OK? > > > > But what about the comment? > > > > We track no > > information on whether given type i

[committed] Readd __tls_get_addr and __interceptor___tls_get_addr to libtsan for ABI compatibility (PR sanitizer/68824)

2016-01-19 Thread Jakub Jelinek
Hi! The fix for this PR landed upstream, so I've cherry picked it, and after bootstrapping/regtesting it on x86_64-linux last night committed to trunk. 2016-01-19 Jakub Jelinek PR sanitizer/68824 * tsan/tsan_interceptors.cc (NEED_TLS_GET_ADDR, __tls_get_addr, Initializ

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 01:42:00PM +0100, Jan Hubicka wrote: > > One solution I do envision, which might help, would be to try to figure > > out which types are unused, and discard those. Say, scan all variables > > within a lexical scope (including nested blocks), deciding which ones > > can be d

Re: Prune BLOCK_VARs lists in free_lang_data

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Jan Hubicka wrote: > > On Tue, 19 Jan 2016, Jan Hubicka wrote: > > > > > Hi, > > > here is updated patch. It has same effect as the former version. > > > > > > Bootstrapped/regtested x86_64-linux, OK? > > > > But what about the comment? > > > > We track no > >

Re: [PATCH] Fix debug info handling in prepare_shrink_wrap (PR debug/65779)

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 01:27:32PM +0100, Bernd Schmidt wrote: > Is there a way to merge these two blocks (e.g. by moving this to the start > of the loop and testing for insn or BB_HEAD)? Sure, like this? 2016-01-19 Jakub Jelinek PR debug/65779 * shrink-wrap.c: Include valtrac

Re: [PATCH] Fix debug info handling in prepare_shrink_wrap (PR debug/65779)

2016-01-19 Thread Bernd Schmidt
On 01/19/2016 02:08 PM, Jakub Jelinek wrote: On Tue, Jan 19, 2016 at 01:27:32PM +0100, Bernd Schmidt wrote: Is there a way to merge these two blocks (e.g. by moving this to the start of the loop and testing for insn or BB_HEAD)? Sure, like this? That's ok. I'm assuming you know best how to u

Re: [C++] Add -fnull-this-pointer

2016-01-19 Thread Trevor Saunders
On Tue, Jan 19, 2016 at 01:11:44PM +0100, Jan Hubicka wrote: > Hi, > according to Trevor, the assumption about THIS pointer being non-NULL breaks That was Markus, not me. > several bigger C++ packages (definitly including Firefox, but I believe > kdevelop was mentioned, too). This patch makes th

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-19 Thread Alan Lawrence
On 19/01/16 09:46, Christophe Lyon wrote: On 19 January 2016 at 04:05, H.J. Lu wrote: On Thu, Dec 24, 2015 at 3:55 AM, Alan Lawrence wrote: This version changes the test cases to fix failures on some platforms, by rewriting the initializers so that they aren't pushed out to the constant pool.

Re: [PATCH][RFC][Offloading] Fix PR68463

2016-01-19 Thread Ilya Verbin
On Tue, Jan 19, 2016 at 10:36:28 +0100, Jakub Jelinek wrote: > On Tue, Jan 19, 2016 at 09:57:01AM +0100, Richard Biener wrote: > > On Mon, 18 Jan 2016, Ilya Verbin wrote: > > > On Fri, Jan 15, 2016 at 09:15:01 +0100, Richard Biener wrote: > > > > On Fri, 15 Jan 2016, Ilya Verbin wrote: > > > > > II

Re: [PATCH] Fix ICE with asm "m" (stmt-expr) operand (PR middle-end/67653)

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 10:00:00AM +0100, Richard Biener wrote: > On Tue, 19 Jan 2016, Jakub Jelinek wrote: > > Here is an attempt to fix ICE on statement expression in "m" asm input > > operand. The problem is that gimplify_asm_expr attempts to mark it > > addressable, but that can be just too la

Re: [PATCH 2/4] Equate MEM_REFs and ARRAY_REFs in tree-ssa-scopedtables.c

2016-01-19 Thread Christophe Lyon
On 19 January 2016 at 14:22, Alan Lawrence wrote: > On 19/01/16 09:46, Christophe Lyon wrote: >> >> On 19 January 2016 at 04:05, H.J. Lu wrote: >>> >>> On Thu, Dec 24, 2015 at 3:55 AM, Alan Lawrence >>> wrote: This version changes the test cases to fix failures on some platforms,

Re: [RFC] [nvptx] Try to cope with cuLaunchKernel returning CUDA_ERROR_LAUNCH_OUT_OF_RESOURCES

2016-01-19 Thread Alexander Monakov
On Tue, 19 Jan 2016, Thomas Schwinge wrote: > Hi! > > With nvptx offloading, in one OpenACC test case, we're running into the > following fatal error (GOMP_DEBUG=1 output): > > [...] > info: Function properties for 'LBM_performStreamCollide$_omp_fn$0': > info: used 87 registe

Re: [RFC] [nvptx] Try to cope with cuLaunchKernel returning CUDA_ERROR_LAUNCH_OUT_OF_RESOURCES

2016-01-19 Thread Nathan Sidwell
On 01/19/16 06:49, Thomas Schwinge wrote: (One problem certainly might be that we're currently not doing any register allocation for nvptx, as far as I remember based on the idea that PTX is only a "virtual ISA", and the PTX JIT compiler would "fix this up" for us -- which I'm not sure it actual

Re: C++ PATCH for c++/68586 (rejects-valid with enum in C++11)

2016-01-19 Thread Jason Merrill
OK, thanks. Jason

RFA: MIPS: Fix race condition causing PR 69129

2016-01-19 Thread Nick Clifton
Hi Catherine, Hi Eric, Hi Matthew, GCC PR 69129 reports a problem with the MIPS backend: https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69129 I traced the problem down to a race condition in mips_compute_frame_info. This calls mips_global_pointer, which through a torturous chain of inferi

Re: [RFC] [nvptx] Try to cope with cuLaunchKernel returning CUDA_ERROR_LAUNCH_OUT_OF_RESOURCES

2016-01-19 Thread Alexander Monakov
On Tue, 19 Jan 2016, Alexander Monakov wrote: > > ... to determine an optimal number of threads per block given the number > > of registers (maybe just querying CU_FUNC_ATTRIBUTE_MAX_THREADS_PER_BLOCK > > would do that already?). > > I have implemented that for OpenMP offloading, but also since CU

Re: C++ PATCH to suppress bogus -Wunused warning for parameter packs (PR c++/68965)

2016-01-19 Thread Marek Polacek
On Mon, Jan 18, 2016 at 11:17:04AM -0500, Jason Merrill wrote: > But we do currently print > > wa.C:1:24: warning: unused parameter ‘xs#0’ [-Wunused-parameter] > wa.C:1:24: warning: unused parameter ‘xs#1’ [-Wunused-parameter] > wa.C:1:24: warning: unused parameter ‘xs#2’ [-Wunused-parameter] > >

Re: [PATCH] Fix ICE in vectorizable_store ().

2016-01-19 Thread Kirill Yukhin
Hello Richard, On 15 Jan 12:54, Richard Biener wrote: > On Fri, 15 Jan 2016, Kirill Yukhin wrote: > > > Hello, > > Thet patch in the bottom adds check if rhs is "useless_type_conversion_p" > > in vectorizable_store () to avoid subsequent gcc_assert. > > > > This change is very similar to [1]. > >

Re: C++ PATCH to suppress bogus -Wunused warning for parameter packs (PR c++/68965)

2016-01-19 Thread Jason Merrill
OK, thanks. Jason

[PATCH] Fix bootstrap with older and non-GCC host compilers

2016-01-19 Thread Richard Biener
It also seems we're wrongly using values defined for the host while looking at GIMPLE IL for the target. Bootstrap / regtest running on x86_64-unknown-linux-gnu. Ok? Richard. 2016-01-19 Richard Biener * hsa-gen.c (get_memory_order_name): Use MEMMODEL_ constants and name.

Re: [PATCH] Fix ICE in vectorizable_store ().

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Kirill Yukhin wrote: > Hello Richard, > On 15 Jan 12:54, Richard Biener wrote: > > On Fri, 15 Jan 2016, Kirill Yukhin wrote: > > > > > Hello, > > > Thet patch in the bottom adds check if rhs is "useless_type_conversion_p" > > > in vectorizable_store () to avoid subsequent gcc

Re: [PATCH][ARM] PR target/69135: Mark ARMv8 vcvt instructions as unconditional

2016-01-19 Thread Ramana Radhakrishnan
On Fri, Jan 15, 2016 at 3:05 PM, Kyrill Tkachov wrote: > Hi all, > > In this PR the ARMv8 vcvt instructions end up being conditionalised when > they don't have a conditional form. > setting the predicable attribute to "no" is not enough. We need to set the > "conds" attribute to unconditional as w

Re: [C++] Add -fnull-this-pointer

2016-01-19 Thread Richard Biener
On Tue, Jan 19, 2016 at 2:18 PM, Trevor Saunders wrote: > On Tue, Jan 19, 2016 at 01:11:44PM +0100, Jan Hubicka wrote: >> Hi, >> according to Trevor, the assumption about THIS pointer being non-NULL breaks > > That was Markus, not me. > >> several bigger C++ packages (definitly including Firefox,

Re: [hsa merge 00/10] Merge of HSA branch

2016-01-19 Thread Richard Biener
On Tue, Jan 19, 2016 at 11:45 AM, Martin Jambor wrote: > Hi, > > On Wed, Jan 13, 2016 at 06:39:25PM +0100, Martin Jambor wrote: >> Hi, >> >> this is hopefully the last big re-post of the HSA patches... > > I have committed the combined patch as revision 232549 after > bootstrapping and testing all

Re: [PATCH] fix gimplification of call parameters (PR cilkplus/69267)

2016-01-19 Thread Ryan Burn
Does this look ok? > On Jan 15, 2016, at 5:41 PM, Ryan Burn wrote: > > This patch changes the function cilk_gimplify_call_params_in_spawned_fn to > use gimplify_arg instead of gimplify_expr. It fixes an ICE when calling a > function with a constructed empty class as the argument. > > Bootstra

Re: [PATCH] Fix bootstrap with older and non-GCC host compilers

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 03:19:17PM +0100, Richard Biener wrote: > > It also seems we're wrongly using values defined for the host while > looking at GIMPLE IL for the target. > > Bootstrap / regtest running on x86_64-unknown-linux-gnu. > > Ok? > > Richard. > > 2016-01-19 Richard Biener > >

Re: [C++] Add -fnull-this-pointer

2016-01-19 Thread Markus Trippelsdorf
On 2016.01.19 at 15:23 +0100, Richard Biener wrote: > On Tue, Jan 19, 2016 at 2:18 PM, Trevor Saunders > wrote: > > On Tue, Jan 19, 2016 at 01:11:44PM +0100, Jan Hubicka wrote: > >> Hi, > >> according to Trevor, the assumption about THIS pointer being non-NULL > >> breaks > > > > That was Markus

Re: [RFC] [nvptx] Try to cope with cuLaunchKernel returning CUDA_ERROR_LAUNCH_OUT_OF_RESOURCES

2016-01-19 Thread Thomas Schwinge
Hi! On Tue, 19 Jan 2016 08:47:02 -0500, Nathan Sidwell wrote: > On 01/19/16 06:49, Thomas Schwinge wrote: > > int axis = get_oacc_ifn_dim_arg (call); > > + if (axis == GOMP_DIM_WORKER) > > +{ > > + /* libgomp's nvptx plugin might potentially modify > > +dims[GOMP_DIM_WORKER]. *

Re: [C++] Add -fnull-this-pointer

2016-01-19 Thread Jakub Jelinek
On Tue, Jan 19, 2016 at 03:37:02PM +0100, Markus Trippelsdorf wrote: > > Agreed. As we already have a flag that can be used as a workaround I don't > > see > > a reason to add another more specific one. That just makes it a > > lesser incentive > > for people to fix their code. > > Well, -fno-d

Re: [PATCH] ARM PR68620 (ICE with FP16 on armeb)

2016-01-19 Thread Alan Lawrence
On 19/01/16 11:15, Christophe Lyon wrote: For neon_vdupn, I chose to implement neon_vdup_nv4hf and neon_vdup_nv8hf instead of updating the VX iterator because I thought it was not desirable to impact neon_vrev32. Well, the same instruction will suffice for vrev32'ing vectors of HF just as well

Re: [RFC] [nvptx] Try to cope with cuLaunchKernel returning CUDA_ERROR_LAUNCH_OUT_OF_RESOURCES

2016-01-19 Thread Thomas Schwinge
Hi! On Tue, 19 Jan 2016 17:07:17 +0300, Alexander Monakov wrote: > On Tue, 19 Jan 2016, Alexander Monakov wrote: > > > ... to determine an optimal number of threads per block given the number > > > of registers (maybe just querying CU_FUNC_ATTRIBUTE_MAX_THREADS_PER_BLOCK > > > would do that alre

Re: [PATCH] Fix PR c++/69283 (auto deduction fails when ADL is required)

2016-01-19 Thread Jason Merrill
On 01/18/2016 10:55 PM, Patrick Palka wrote: mark_used is wrongly diagnosing a use of a TEMPLATE_DECL (e.g. the call to f1 in function f3 of auto-fn29.C below) for having an undeduced 'auto' return type. This doesn't make sense, because an 'auto' used inside a template doesn't get deduced until

Re: [ping] pending patches

2016-01-19 Thread Jason Merrill
On 01/05/2016 04:30 AM, Eric Botcazou wrote: It doesn't look to me like DW_AT_endianity is applicable to array types or members in DWARF 3/4; instead, it should be applied to the underlying base type. OK, the attached patch does that so is more invasive as expected. Please document the revers

Re: [PATCH] Fix bootstrap with older and non-GCC host compilers

2016-01-19 Thread Richard Biener
On Tue, 19 Jan 2016, Jakub Jelinek wrote: > On Tue, Jan 19, 2016 at 03:19:17PM +0100, Richard Biener wrote: > > > > It also seems we're wrongly using values defined for the host while > > looking at GIMPLE IL for the target. > > > > Bootstrap / regtest running on x86_64-unknown-linux-gnu. > > >

Re: [RFC] [nvptx] Try to cope with cuLaunchKernel returning CUDA_ERROR_LAUNCH_OUT_OF_RESOURCES

2016-01-19 Thread Alexander Monakov
On Tue, 19 Jan 2016, Thomas Schwinge wrote: > Hi! > > On Tue, 19 Jan 2016 17:07:17 +0300, Alexander Monakov > wrote: > > On Tue, 19 Jan 2016, Alexander Monakov wrote: > > > > ... to determine an optimal number of threads per block given the number > > > > of registers (maybe just querying > >

Re: [PATCH][ARM] PR target/69135: Mark ARMv8 vcvt instructions as unconditional

2016-01-19 Thread Kyrill Tkachov
On 19/01/16 14:21, Ramana Radhakrishnan wrote: On Fri, Jan 15, 2016 at 3:05 PM, Kyrill Tkachov wrote: Hi all, In this PR the ARMv8 vcvt instructions end up being conditionalised when they don't have a conditional form. setting the predicable attribute to "no" is not enough. We need to set the

Re: [PATCH][ARM,AARCH64] target/PR68674: relayout vector_types in expand_expr

2016-01-19 Thread Richard Biener
On Tue, Jan 19, 2016 at 4:13 PM, Christian Bruel wrote: > > > On 01/19/2016 04:01 PM, Christian Bruel wrote: >> >> Hi Richard, >> >> thanks for your input, >> >> On 01/18/2016 12:36 PM, Richard Biener wrote: >>> >>> On Fri, Jan 8, 2016 at 2:29 PM, Christian Bruel >>> wrote: When compili

RE: [Patch, MIPS] Remove definition of TARGET_PROMOTE_PROTOTYPES

2016-01-19 Thread Maciej W. Rozycki
On Sat, 12 Dec 2015, Matthew Fortune wrote: > > > * config/mips/mips.c (mips_promote_function_mode): New function. > > > (TARGET_PROMOTE_FUNCTION_MODE): Define as above function. > > > (TARGET_PROMOTE_PROTOTYPES): Remove. > > I'm OK with this change on the basis that MIPS has been providing

Re: [RFC][PATCH, ARM 7/8] ARMv8-M Security Extension's cmse_nonsecure_call: use __gnu_cmse_nonsecure_call]

2016-01-19 Thread Andre Vieira (lists)
On 16/01/16 14:49, Senthil Kumar Selvaraj wrote: User-agent: mu4e 0.9.13; emacs 24.5.1 Hi, Apologies for the bad posting style (I don't have the original email handy), but shouldn't _gnu_cmse_nonsecure_call be defined with the .global directive in the below hunk (to make it visible when linking

Re: [PATCH] Fix PR c++/69283 (auto deduction fails when ADL is required)

2016-01-19 Thread Patrick Palka
On Tue, Jan 19, 2016 at 9:56 AM, Jason Merrill wrote: > On 01/18/2016 10:55 PM, Patrick Palka wrote: >> >> mark_used is wrongly diagnosing a use of a TEMPLATE_DECL (e.g. the call >> to f1 in function f3 of auto-fn29.C below) for having an undeduced >> 'auto' return type. This doesn't make sense,

[wwwdocs] Add special math functions to libstdc++ changes

2016-01-19 Thread Jonathan Wakely
This documents Ed's recent addition to trunk. Committed to cvs. Index: htdocs/gcc-6/changes.html === RCS file: /cvs/gcc/wwwdocs/htdocs/gcc-6/changes.html,v retrieving revision 1.46 diff -u -r1.46 changes.html --- htdocs/gcc-6/changes

Re: [PATCH 4/4][AArch64] Cost CCMP instruction sequences to choose better expand order

2016-01-19 Thread H.J. Lu
On Tue, Dec 15, 2015 at 2:33 AM, Wilco Dijkstra wrote: > ping > >> -Original Message- >> From: Wilco Dijkstra [mailto:wilco.dijks...@arm.com] >> Sent: 13 November 2015 16:03 >> To: 'gcc-patches@gcc.gnu.org' >> Subject: [PATCH 4/4][AArch64] Cost CCMP instruction sequences to choose >> bett

[wwwdocs] Update changes.html for LTO and IPA

2016-01-19 Thread Jan Hubicka
Hi, this patch mentiones few user visible changes I can think of. I will add some quality data on firefox once stage3 closes. It seems that the optimization section of changes.html deserve some work :) Honza Index: changes.html ===

[wwwdocs] Document C++17 static_assert feature

2016-01-19 Thread Jonathan Wakely
Patch for the /projects/cxx1z.html page to document the unary static_assert in C++17. Committed to CVS. Index: htdocs/projects/cxx1z.html === RCS file: /cvs/gcc/wwwdocs/htdocs/projects/cxx1z.html,v retrieving revision 1.8 diff -u -r1

Re: [PATCH 4/4][AArch64] Cost CCMP instruction sequences to choose better expand order

2016-01-19 Thread Wilco Dijkstra
H.J. Lu wrote: > It breaks bootstrap on Linux/x86: Sorry about that - it looks like the warning levels seem to have changed since that patch was tested... I have a trivial fix which I'll get checked in soon. Wilco

Do not redirect calls to cxa_pure_virtual into bultin_unreachable in some cases

2016-01-19 Thread Jan Hubicka
Hi, currently we optimize calls to pure virtual functions to __builtin_unreachable. While this is correct it does make code harder to debug. This patch makes ipa-devirt to treat cxa_pure_virtual specially and devirtualize to it when it is the only posible target. Bootstrapped/regtested x86_64-lin

Re: Remove outdated text from lto.texi

2016-01-19 Thread Jeff Law
On 01/19/2016 03:56 AM, Kugan wrote: Hi, lto.texi has "Currently, the linker plugin works only in combination with the Gold linker, but a GNU ld implementation is under development". I don't think this is true any more. Attached patch removes this. is this OK for trunk? Thanks, Kugan gcc/Chang

Re: Do not redirect calls to cxa_pure_virtual into bultin_unreachable in some cases

2016-01-19 Thread Bernd Schmidt
On 01/19/2016 05:50 PM, Jan Hubicka wrote: +static bool +is_cxa_pure_virtual_p (tree target) +{ + return target && TREE_CODE (TREE_TYPE (target)) != METHOD_TYPE +&& DECL_NAME (target) +&& !strcmp (IDENTIFIER_POINTER (DECL_NAME (target)), +"__cxa_pure_virtual"

Re: [PATCH, AArch64] Fix for PR67896 (C++ FE cannot distinguish __Poly{8,16,64,128}_t types)

2016-01-19 Thread James Greenhalgh
On Tue, Jan 19, 2016 at 06:28:18AM +0100, Roger Ferrer Ibáñez wrote: > Hi, > > aarch64-builtins.c defines several SIMD builtin types. Among these > SIMD types there are the polynomials __Poly{8,16,64,128}_t. These are > built by a call to build_distinct_type_copy > (unsigned_int{Q,H,D,T}I_type_nod

Re: [wwwdocs] gcc-6/changes.html: diagnostics, Levenshtein, -Wmisleading-indentation, jit (v2)

2016-01-19 Thread Gerald Pfeifer
On Fri, 15 Jan 2016, David Malcolm wrote: > Here's an updated version of the above, which the W3C validator > reports as being clean (fixing various "&" and "<" and a missing > end-tag). Nice - and a lot of nice changes you implemented since GCC 5! > OK to commit? Yep. I was going to flag "miss

RE: [PATCH] [ARC] Add basic support for double load and store instructions

2016-01-19 Thread Claudiu Zissulescu
Hi, I've prepared a new patch based on the received review (attached). I also added a mod on invoke.texi regarding mll64 documentation. This mod was missing in the first patch. I have tested it with dg.exp for arc700, archs and archs+ll64. Please let me know if everything is alright. Thank yo

[PATCH] c++/59759 - ICE in unify, using std::enable_if on classes

2016-01-19 Thread Martin Sebor
Attached is the patch to avoid the ICE that Kai posted below with the test case Marek asked for in his response. I didn't see any further followup on the list. https://gcc.gnu.org/ml/gcc-patches/2015-05/msg02325.html Martin gcc/testsuite/ChangeLog: 2016-01-19 Martin Sebor PR c++/59759 *

Re: [PATCH 4/4][AArch64] Cost CCMP instruction sequences to choose better expand order

2016-01-19 Thread Wilco Dijkstra
H.J. Lu wrote: > It breaks bootstrap on Linux/x86: Committed trivial fix as r232576: Index: gcc/ChangeLog === --- gcc/ChangeLog(revision 232575) +++ gcc/ChangeLog(working copy) @@ -1,3 +1,7 @@ +2016-01-19 Wilco Dijkstra +

Re: [C++] Add -fnull-this-pointer

2016-01-19 Thread Mike Stump
On Jan 19, 2016, at 6:41 AM, Jakub Jelinek wrote: > But then perhaps it will be better incentive for the projects to fix their > cruft. With a specialized option they will keep broken code forever. Flags are forever.

C++ PATCH for c++/59759 (ICE with variable as default template argument)

2016-01-19 Thread Jason Merrill
The comment removed by this patch asserts that we can't see a VAR_DECL in unify, because any use of a variable as a template argument will have been adjusted to be either its constant value or its address. But that isn't actually true when the type of the non-type template parameter depends on

Re: [patch, fortran] Inline MATMUL(A,TRANSPOSE(B)), PR 66094

2016-01-19 Thread Toon Moene
On 01/18/2016 08:55 PM, Toon Moene wrote: On 01/17/2016 01:44 PM, Thomas Koenig wrote: So... comments? Toon, would this help you? Could yo maybe give this a spin? Thanks, the nightly test at my home computer will build with your patch. That was the plan; unfortunately, the system crashed

Re: [wwwdocs] gcc-6/changes.html: diagnostics, Levenshtein, -Wmisleading-indentation, jit (v2)

2016-01-19 Thread Manuel López-Ibáñez
On 19/01/16 17:08, Gerald Pfeifer wrote: On Fri, 15 Jan 2016, David Malcolm wrote: Here's an updated version of the above, which the W3C validator reports as being clean (fixing various "&" and "<" and a missing end-tag). Nice - and a lot of nice changes you implemented since GCC 5! Am I th

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-19 Thread Torvald Riegel
On Sat, 2016-01-16 at 10:57 +0100, Dominique d'Humières wrote: > > Addressed these, fixed a problem with using GLIBCXX_WEAK_DEFINITION > > (which is only set on Darwin despite the generic-sounding name -- so > > just use __attribute__((weak)) directly), and also updated > > testsuite_abi.cc so that

Re: [PATCH v2] libstdc++: Make certain exceptions transaction_safe.

2016-01-19 Thread Jonathan Wakely
On 19/01/16 20:10 +0100, Torvald Riegel wrote: On Sat, 2016-01-16 at 10:57 +0100, Dominique d'Humières wrote: > Addressed these, fixed a problem with using GLIBCXX_WEAK_DEFINITION > (which is only set on Darwin despite the generic-sounding name -- so > just use __attribute__((weak)) directly), a

Re: [wwwdocs] gcc-6/changes.html: diagnostics, Levenshtein, -Wmisleading-indentation, jit (v2)

2016-01-19 Thread Mike Stump
On Jan 19, 2016, at 11:05 AM, Manuel López-Ibáñez wrote: > > Am I the only one who doesn't see the colors at > https://gcc.gnu.org/gcc-6/changes.html#c-family nor > https://gcc.gnu.org/gcc-5/changes.html#fortran ? Yes. The darkslategrey of the headings is very close to black, but the links s

Re: [wwwdocs] gcc-6/changes.html: diagnostics, Levenshtein, -Wmisleading-indentation, jit (v2)

2016-01-19 Thread Manuel López-Ibáñez
On 19 January 2016 at 19:31, Mike Stump wrote: > On Jan 19, 2016, at 11:05 AM, Manuel López-Ibáñez > wrote: >> >> Am I the only one who doesn't see the colors at >> https://gcc.gnu.org/gcc-6/changes.html#c-family nor >> https://gcc.gnu.org/gcc-5/changes.html#fortran ? > > Yes. The darkslategr

[PATCH] Fix c/68513 for GCC5 (match.pd and SAVE_EXPRs)

2016-01-19 Thread Marek Polacek
Recently on IRC we've concluded that for GCC 5 the simplest solution will be to just disable the problematic pattern on GENERIC. So done in the following. (The problem was that the match.pd pattern created SAVE_EXPRs which then leaked into gimplification.) Bootstrapped/regtested on x86_64-linux,

[PATCH][committed] libitm: Remove dead code.

2016-01-19 Thread Torvald Riegel
I missed dead code when I removed the cacheline stuff. local_type_traits hasn't been updated either, apparently leading to bootstrap issues. So we just remove more dead code. Tested fine on x86_64-linux. Committed. commit c608b69c3c49c7d29033faf328fd4d117f31fd9f Author: Torvald Riegel Date: T

  1   2   >