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 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: > http://gcc.gnu.or

[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 mat

[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 PR c++/59123 * decl.c (validate_constexpr_redeclaration): Redecla

[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 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 Liska >> + Jan Hubicka

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 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 easier to w

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 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 > RHS, which it needs to h

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 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 sup

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 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 has >> brought th

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 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 thi

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 > + Jan Hubicka > + > + * cgraphunit.c (node_cmp): New function. > + (expand_all_functions): Function ordering

[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 o

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 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++ frontend. Hav

Re: [libgfortran,patch] Silence a warning

2013-11-17 Thread Janne Blomqvist
On Sun, Nov 17, 2013 at 1:05 PM, FX 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 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 access the > low w

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 emit_bloc

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 endi

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] A

Re: cilking away

2013-11-17 Thread Mike Stump
On Nov 15, 2013, at 8:23 PM, "Iyer, Balaji V" 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 jason on that email.

Re: Clean up atomic tests

2013-11-17 Thread Richard Henderson
On 11/08/2013 02:43 AM, Joseph S. Myers wrote: > This patch cleans up various issues with the tests of atomics built-in > functions and libatomic functions, in preparation for adapting those > tests to add test coverage of stdatomic.h macros. The tests were > missing a return type for main (C11 do

_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 last

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 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 configured with > --with-cpu=pow

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. +{ + ba

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 discus

[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

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 wrote: >> Hello, >>there is a new version of the patch, I disabled the branch with >> profile-generate. Could you pleas

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 > > PR sanitizer/59106 > * asan/Makefile.am: Disable RTTI. * asan/Makefile.am (AM_CXXFLAGS): Add -fno-rtti. > * interception/Makefile.am: Likewise. * interception/Makefile.