Re: Recent Go patch fails several tests on 32bit CentOS 5.10

2013-11-17 Thread Uros Bizjak
On Fri, Nov 15, 2013 at 5:57 PM, Ian Lance Taylor i...@google.com wrote: I still see panic in runtime (trace below), segfault in sync, database/sql, net/http and abort in sync/atomic on 32bit CentOS 5.10 library. The problems on 32-bit are a recently introduced middle-end bug:

[wide-int] Remove tree_fits_hwi_p and tree_to_hwi

2013-11-17 Thread Richard Sandiford
AIUI the two-argument tree_fits_hwi_p and tree_to_hwi were replacements for host_integerp and tree_low_cst with variable pos arguments. I removed those uses from trunk this week, and Mike's merge has brought that into branch. The only remaining use is in a branch-local change to the way that

[patch,libgfortran] Fix binary128 ERFC_SCALED

2013-11-17 Thread FX
This patch fixes libgfortran’s binary128 [aka real(kind=16)] variant of ERFC_SCALED. The original code, which I had lifted from netlib, gives only 18 significant decimal digits, which is not enough for binary128 (33 decimal digits). I thus implemented a new variant for binary128. For arguments

[C++ Patch] PR 59123

2013-11-17 Thread Paolo Carlini
Hi, C++11 7.1.5 seems very clear that redeclarations of *variables* differing in constexpr are fine (clang and icc agree). Tested x86_64-linux. Thanks, Paolo. /// /cp 2013-11-17 Paolo Carlini paolo.carl...@oracle.com PR c++/59123 * decl.c

[libgfortran,patch] Silence a warning

2013-11-17 Thread FX
This attach patch adds an assert() in the library to fix PR 51828, i.e. silence a “may be used uninitialized” warning. Built and regtested on x86_64-apple-darwin13. OK to commit? FX libwarning.ChangeLog Description: Binary data libwarning.diff Description: Binary data

Re: [RFA][PATCH]Fix 59019

2013-11-17 Thread Steven Bosscher
On Sun, Nov 17, 2013 at 7:48 AM, Jeff Law wrote: * combine.c (try_combine): If we have created an unconditional trap, make sure to fixup the insn stream CFG appropriately. diff --git a/gcc/combine.c b/gcc/combine.c index 13f5e29..b3d20f2 100644 --- a/gcc/combine.c +++

Re: [PATCH] Time profiler - phase 2

2013-11-17 Thread Martin Liška
Dear Jan, On 16 November 2013 12:24, Jan Hubicka hubi...@ucw.cz wrote: diff --git a/gcc/ChangeLog b/gcc/ChangeLog index c566a85..1562098 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,15 @@ +2013-11-13 Martin Liskamarxin.li...@gmail.com +

Re: [PowerPC] libffi fixes and support for PowerPC64 ELFv2

2013-11-17 Thread David Edelsohn
On Sun, Nov 17, 2013 at 1:25 AM, Alan Modra amo...@gmail.com wrote: On Sat, Nov 16, 2013 at 10:18:05PM +1030, Alan Modra wrote: The following six patches correspond to patches posted to the libffi mailing list a few days ago to add support for PowerPC64 ELFv2. The The ChangeLog just became

Re: [PATCH, rs6000] Emit correct note for DWARF CFI information on LE prolog VSX stores

2013-11-17 Thread David Edelsohn
On Sat, Nov 16, 2013 at 10:32 PM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: Hi, For VSX in little endian we currently split vector register stores into a permute/store pair. For prolog stores, this results in a REG_FRAME_RELATED_EXPR note that doesn't have a simple register for its

Re: [PowerPC] libffi fixes and support for PowerPC64 ELFv2

2013-11-17 Thread Alan Modra
On Sun, Nov 17, 2013 at 07:53:59AM -0500, David Edelsohn wrote: On Sun, Nov 17, 2013 at 1:25 AM, Alan Modra amo...@gmail.com wrote: On Sat, Nov 16, 2013 at 10:18:05PM +1030, Alan Modra wrote: The following six patches correspond to patches posted to the libffi mailing list a few days ago to

Re: [wide-int] Remove tree_fits_hwi_p and tree_to_hwi

2013-11-17 Thread Kenneth Zadeck
On 11/17/2013 05:29 AM, Richard Sandiford wrote: AIUI the two-argument tree_fits_hwi_p and tree_to_hwi were replacements for host_integerp and tree_low_cst with variable pos arguments. I removed those uses from trunk this week, and Mike's merge has brought that into branch. i think that i am a

Re: [wide-int] Remove tree_fits_hwi_p and tree_to_hwi

2013-11-17 Thread Richard Sandiford
Kenneth Zadeck zad...@naturalbridge.com writes: On 11/17/2013 05:29 AM, Richard Sandiford wrote: AIUI the two-argument tree_fits_hwi_p and tree_to_hwi were replacements for host_integerp and tree_low_cst with variable pos arguments. I removed those uses from trunk this week, and Mike's merge

Re: [wide-int] Remove tree_fits_hwi_p and tree_to_hwi

2013-11-17 Thread Kenneth Zadeck
On 11/17/2013 10:58 AM, Richard Sandiford wrote: Kenneth Zadeck zad...@naturalbridge.com writes: On 11/17/2013 05:29 AM, Richard Sandiford wrote: AIUI the two-argument tree_fits_hwi_p and tree_to_hwi were replacements for host_integerp and tree_low_cst with variable pos arguments. I removed

Re: [C++ Patch] PR 59123

2013-11-17 Thread Jason Merrill
OK. Jason

Re: [PATCH] Time profiler - phase 2

2013-11-17 Thread Jan Hubicka
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 5cb07b7..754f882 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,13 @@ +2013-11-17 Martin Liska marxin.li...@gmail.com + Jan Hubicka j...@suse.cz + + * cgraphunit.c (node_cmp): New function. +

[PATCH, rs6000] Fix little-endian access to sdmode_stack_slot

2013-11-17 Thread Ulrich Weigand
Hello, when accessing the sdmode_stack_slot, code in rs6000_emit_move would unconditionally use rtx mem = adjust_address_nv (operands[0], mode, 4); This is wrong in little-endian mode; we always need to access the low word there too. Fixed by the patch below, which fixes a large number

Re: [PATCH][1-3] New configure option to enable Position independent executable as default.

2013-11-17 Thread Magnus Granberg
lördag 16 november 2013 20.37.58 skrev Ryan Hill: On Wed, 13 Nov 2013 23:28:45 +0100 Magnus Granberg zo...@gentoo.org wrote: Hi This patchset will add a new configure options --enable-default-pie. With the new option enable will make it pass -fPIE and -pie from the gcc and g++

Re: [libgfortran,patch] Silence a warning

2013-11-17 Thread Janne Blomqvist
On Sun, Nov 17, 2013 at 1:05 PM, FX fxcoud...@gmail.com wrote: This attach patch adds an assert() in the library to fix PR 51828, i.e. silence a “may be used uninitialized” warning. Built and regtested on x86_64-apple-darwin13. OK to commit? FX Ok, thanks. -- Janne Blomqvist

[PATCH, i386]: Fix PR59153, ICE: in memory_address_length, at config/i386/i386.c

2013-11-17 Thread Uros Bizjak
Hello! ix86_decompose_address is called from many places in i386.c, also to calculate various attributes of the insn (length, etc). The testcase failed since addsi_1 pattern was declared as TYPE_LEA and its pattern (involving subregs of SFmode) was passed to length attribute calculation as memory

Re: [PATCH, rs6000] Fix little-endian access to sdmode_stack_slot

2013-11-17 Thread David Edelsohn
On Sun, Nov 17, 2013 at 4:12 PM, Ulrich Weigand uweig...@de.ibm.com wrote: Hello, when accessing the sdmode_stack_slot, code in rs6000_emit_move would unconditionally use rtx mem = adjust_address_nv (operands[0], mode, 4); This is wrong in little-endian mode; we always need to

Re: [PATCH] Time profiler - phase 2

2013-11-17 Thread Jan Hubicka
Hello, there is a new version of the patch, I disabled the branch with profile-generate. Could you please advise me how should I force to use profile-reorder-functions just with enable LTO optimization? I also attach reordering results: o gimp-reoder-latest.html (latest patch) o

Re: Add value range support into memcpy/memset expansion

2013-11-17 Thread Jan Hubicka
Hi, this is version I comitted. It also adds a testcase and enables the support in i386 backend. Honza * doc/md.texi (setmem, movstr): Update documentation. * builtins.c (determine_block_size): New function. (expand_builtin_memcpy): Use it and pass it to

Re: Some wide-int review comments

2013-11-17 Thread Kenneth Zadeck
On 11/08/2013 05:30 AM, Richard Sandiford wrote: Some comments from looking through the diff with the merge point, ignoring wide-int.h and wide-int.cc. A few more to follow in the form of patchses. dwarf2out.c has: +case CONST_WIDE_INT: + if (mode == VOIDmode) +

[PATCH, rs6000] Fix libcpp/lex.c Altivec code to be correct for little endian

2013-11-17 Thread Bill Schmidt
Hi, As Ulrich Weigand discovered, libcpp/lex.c contains some code optimized for use with Altivec that is incorrect for little endian targets. This breaks bootstrap on powerpc64le-unknown-linux-gnu when configured with --with-cpu=power7. This patch makes appropriate modifications for little

RE: [patch] [arm] ARM Cortex-M3/M4 tuning

2013-11-17 Thread Joey Ye
Sorry about this. I should have run x86 make check. - Joey -Original Message- From: Richard Biener [mailto:richard.guent...@gmail.com] Sent: Thursday, November 14, 2013 22:16 To: H.J. Lu Cc: Joey Ye; Janis Johnson; GCC Patches; Ramana Radhakrishnan Subject: Re: [patch] [arm] ARM

Re: cilking away

2013-11-17 Thread Mike Stump
On Nov 15, 2013, at 8:23 PM, Iyer, Balaji V balaji.v.i...@intel.com wrote: This is already done in my patch for _Cilk-spawn and _Cilk_sync support for C++. The patch was submitted ~3-4 weeks ago. Ping once a day until reviewed. :-) You should form a new patch and post it and be sure to cc

_Cilk_spawn and _Cilk_sync for C++

2013-11-17 Thread Iyer, Balaji V
Hello Jason et al., Mike Stump mentioned that my _Cilk_spawn and _Cilk_sync for C++ may have been lost in the email pile. So, attached is an updated _Cilk_spawn and _Cilk_sync for C++ patch. Is this Ok to install? Here are the ChangeLog entries (they shouldn't have changed since the

Re: [PATCH, rs6000] Fix libcpp/lex.c Altivec code to be correct for little endian

2013-11-17 Thread David Edelsohn
On Sun, Nov 17, 2013 at 8:33 PM, Bill Schmidt wschm...@linux.vnet.ibm.com wrote: Hi, As Ulrich Weigand discovered, libcpp/lex.c contains some code optimized for use with Altivec that is incorrect for little endian targets. This breaks bootstrap on powerpc64le-unknown-linux-gnu when

Re: [RFA][PATCH]Fix 59019

2013-11-17 Thread Jeff Law
On 11/17/13 04:28, Steven Bosscher wrote: TRAP_CONDITION (PATTERN (i3)) == const1_rtx But shouldn't the check be on const_true_rtx? Or does combine put a const1_rtx there? I took const1_rtx from control_flow_insn_p. That's ultimately what we need to be consistent with. +{ +

Re: [0/10] Replace host_integerp and tree_low_cst

2013-11-17 Thread Jeff Law
On 11/16/13 05:53, Richard Sandiford wrote: After the patch that went in yesterday, all calls to host_integerp and tree_low_cst pass a constant pos argument. This series replaces each function with two separate ones: [ ... ] So I've almost entirely ignored the whole wide-int conversion

[PATCH] Fix static libasan link

2013-11-17 Thread Yury Gribov
Hi, This patch is supposed to fix PR59106 (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59106). The bug manifests when we link static sanitizer libs (asan, tsan or ubsan) against pure C programs. The patch adds -fno-rtti to LDFLAGS of sanitizer runtime libs (based on Kcc's recommendation).

Re: [ia64] [PR target/57491] internal compiler error: in ia64_split_tmode -O2, quadmath

2013-11-17 Thread Kirill Yukhin
On 16 Nov 10:03, Eric Botcazou wrote: As far as I understand semantics of this insn: (insn 200 199 0 (set (reg:DI 15 r15) (mem:DI (post_dec:DI (reg/f:DI 15 r15 [447])) [3 *_61[_12]{lb: 1 sz: 64}.text+8 S8 A64])) -1 (nil)) What is done is (in that sequence). 1. Calculate

Re: Group static constructors and destructors in specific subsections

2013-11-17 Thread Martin Liška
Dear Cary, I've been merging my patches to GCC mainline and I would really appreciate new section naming convention that was suggested by you in the previous post. Is there any progress in implementation? Should I participate in this change and write a patch that will introduce this new section

Re: [PATCH] Time profiler - phase 2

2013-11-17 Thread Martin Liška
Hello, there's new version of the patch. I wrote email to Cary to negotiate how will implement gold's linker patch. Thanks, Martin On 18 November 2013 00:37, Jan Hubicka hubi...@ucw.cz wrote: Hello, there is a new version of the patch, I disabled the branch with profile-generate. Could

Re: [PATCH] Fix static libasan link

2013-11-17 Thread Jakub Jelinek
On Mon, Nov 18, 2013 at 10:45:16AM +0400, Yury Gribov wrote: 2013-11-18 Yury Gribov y.gri...@samsung.com PR sanitizer/59106 * asan/Makefile.am: Disable RTTI. * asan/Makefile.am (AM_CXXFLAGS): Add -fno-rtti. * interception/Makefile.am: Likewise. *