Re: [PATCH][PR65802] Mark ifn_va_arg with ECF_NOTHROW

2015-04-26 Thread Tom de Vries
On 21-04-15 09:50, Richard Biener wrote: On Tue, 21 Apr 2015, Jan Hubicka wrote: Mark ifn_va_arg with ECF_NOTHROW You can definitely make it ECF_LEAF too. I wonder if we can make it ECF_CONST or at leat PURE this would help to keep variadic functions const/pure that may be moderately intere

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

2015-04-26 Thread Mikael Morin
Hello, I'm reviewing the original patch only for now. The added bits in v2 will have to wait. Le 23/04/2015 20:00, Andre Vehreschild a écrit : >>> diff --git a/gcc/fortran/trans-expr.c b/gcc/fortran/trans-expr.c >>> index 9e6432f..80dfed1 100644 >>> --- a/gcc/fortran/trans-expr.c >>> +++ b/gcc/fo

Re: [PATCH] Fix up creation of lowpart subregs in valtrack (PR debug/65678)

2015-04-26 Thread Richard Sandiford
Jakub Jelinek writes: > Hi! > > gen_rtx_SUBREG/gen_lowpart_SUBREG ICE when trying to create a subreg > the backend doesn't like, lowpart_subreg returns NULL. But, for debug info > purposes, all lowpart subregs are valid, a subreg in the debug insns or > var-tracking notes is simply a mode punning

[Patch, C++, PR65882] Check tf_warning flag in build_new_op_1

2015-04-26 Thread Mikhail Maltsev
The attached patch was bootstrapped and regtested on x86_64-unknown-linux-gnu. -- Regards, Mikhail Maltsev gcc/cp/ChangeLog: 2015-04-26 Mikhail Maltsev * call.c (build_new_op_1): Check tf_warning flag in all cases gcc/testsuite/ChangeLog: 2015-04-26 Mikhail Maltsev

Re: [PATCH] Be less conservative in process_{output,input}_constraints (PR target/65689)

2015-04-26 Thread Richard Sandiford
Jakub Jelinek writes: > Hi! > > Right now, stmt.c on constraints not hardcoded in it, and not > define_{register,address,memory}_constraint just assumes the > constraint might allow both reg and mem. Unfortunately, on some > constraints which clearly can't allow either of those leads to errors >

[Patch, fortran] PR65792 - allocation of scalar elemental function with structure constructor fails

2015-04-26 Thread Paul Richard Thomas
Dear All, This patch exploits Mikael's identification of the location of the problem and his testcase. It requires Andre's patch for PR59678 as a prerequisite. It is almost certainly a bit over-conservative but it seems to be free of any memory leakage. The ChangeLog and the comments describe it s

Re: [Patch, fortran] PR65792 - allocation of scalar elemental function with structure constructor fails

2015-04-26 Thread Steve Kargl
On Sun, Apr 26, 2015 at 08:35:06PM +0200, Paul Richard Thomas wrote: > > --- 7062,7091 > { > if (expr->expr_type != EXPR_STRUCTURE) > { > + tree dealloc = NULL_TREE; > gfc_init_se (&se, NULL); > gfc_conv_expr (&se, expr); > gfc_add_block_to_blo

Re: [ARM][PR65768] Keep constants in register when expanding

2015-04-26 Thread Kugan
> While I support your goals, I think your approach needs some refinement. > > In particular, we DO NOT want another function that starts looking at > constant values and tries to decide, on a case by case basis, what to do > with that value. We need to keep the logic for that, as much as > poss

Re: [Patch] Add regex_constants::__polynomial

2015-04-26 Thread Tim Shen
On Sat, Apr 25, 2015 at 1:10 AM, Tim Shen wrote: > Bootstrapped and tested. I didn't test with _GLIBCXX_DEBUG though. Updated the patch for removing DFS restriction for ECMAScript. -- Regards, Tim Shen commit 32cd325c4acffdcdf16caca4233a2455ea483d69 Author: Tim Shen Date: Sat Apr 25 00:23:5

[PATCH] Emit -Waddress warnings for comparing address of reference against NULL

2015-04-26 Thread Patrick Palka
Here is an updated version of the patch with, hopefully, all your suggestions made. I decided to add calls to STRIP_NOPS before emitting the warning so that we properly warn for cases where there's a cast in between the whole thing, e.g. if (!&(int &)a) I also added guards to emit the warnings

[PATCH] Refactor handle_section_attribute to reduce nesting and distinguish error cases

2015-04-26 Thread Josh Triplett
handle_section_attribute contains many levels of nested conditionals and branching code flow paths, with the error cases sometimes in the else case and sometimes in the if case. Simplify the code flow into a series of potential failure cases ending with the successful path, with no nesting and no

[PATCH 0/8] remove more conditional compilation

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders Hi, Here's another series to remove some conditional compilation. each patch was bootstrapped and regtested on x86_64-linux-gnu, and the series was run through config-list.mk. I think this all falls within Jeff's preapproval so I'll commit it tomorrow (today actually) nig

[PATCH 1/8] add default for NO_FUNCTION_CSE

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-26 Trevor Saunders * calls.c (prepare_call_address): Remove ifdef NO_FUNCTION_CSE. * cse.c (fold_rtx): Likewise. * config/alpha/alpha.h (NO_FUNCTION_CSE): Define to 1. * config/arc/arc.h (NO_FUNCTION_CSE): Likewise.

[PATCH 2/8] add default for HARD_REGNO_RENAME_OK

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-27 Trevor Saunders * defaults.h (HARD_REGNO_RENAME_OK): Add default definition to true. * regrename.c (check_new_reg_p): Remove check if HARD_REGNO_RENAME_OK is defined. * sel-sched.c (sel_hard_regno_rename_

[PATCH 3/8] add default for PCC_BITFIELD_TYPE_MATTERS

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-27 Trevor Saunders * defaults.h (PCC_BITFIELD_TYPE_MATTERS): Add default definition to false. * dwarf2out.c (field_byte_offset): REmove check if PCC_BITFIELD_TYPE_MATTERS is defined. * stor-layout.c (layout_

[PATCH 4/8] add default for EPILOGUE_USES

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-27 Trevor Saunders * defaults.h (EPILOGUE_USES): Add default definition of false. * df-scan.c (EPILOGUE_USES): Remove check if its undefined. * resource.c (init_resource_info): Likewise. --- gcc/defaults.h | 4 gcc/df

[PATCH 5/8] always define HAVE_simple_return and HAVE_return

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-27 Trevor Saunders * bb-reorder.c (HAVE_return): Don't check if its undefined. * defaults.h (gen_simple_return): New function. (gen_simple_return): Likewise. (HAVE_return): Add default definition to false. (

[PATCH 6/8] always define HAVE_epilogue

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-27 Trevor Saunders * defaults.h (gen_epilogue): New function. (HAVE_epilogue): Add default definition to false. * alias.c (init_alias_analysis): don't check if HAVE_epilogue is defined. * cfgrtl.c (cfg_layout_finali

[PATCH 8/8] remove #if ARGS_GROW_DOWNWARD

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-27 Trevor Saunders * calls.c (save_fixed_argument_area): Don't check ARGS_GROW_DOWNWARD with the preprocessor. (restore_fixed_argument_area): Likewise. (mem_overlaps_already_clobbered_arg_p): Likewise. (chec

[PATCH 7/8] always define ARGS_GROW_DOWNWARD

2015-04-26 Thread tbsaunde+gcc
From: Trevor Saunders gcc/ChangeLog: 2015-04-27 Trevor Saunders * config/pa/pa.h (ARGS_GROW_DOWNWARD): Define to 1. * defaults.h (ARGS_GROW_DOWNWARD): Define it to 0 by default. * calls.c (save_fixed_argument_area): Don't chekc if ARGS_GROW_DOWNWARD is defined

Re: [PATCH, rs6000, testsuite, PR65456] Changes for unaligned vector load/store support on POWER8

2015-04-26 Thread Bin.Cheng
On Mon, Mar 30, 2015 at 1:42 AM, Bill Schmidt wrote: > Hi, > > This is a follow-up to > https://gcc.gnu.org/ml/gcc-patches/2015-03/msg00103.html, which adds > support for faster unaligned vector memory accesses on POWER8. As > pointed out in https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65456, the