Re: libsanitizer merge from upstream r175042

2013-02-18 Thread Jakub Jelinek
On Fri, Feb 15, 2013 at 07:39:28AM -0800, Ian Lance Taylor wrote: > On Thu, Feb 14, 2013 at 11:45 PM, Konstantin Serebryany > wrote: > > > > Unfortunately, the test does not work if gold is the system linker. > > Any suggestion on how to make the test work with either linker? > > I don't know of

Re: Go testsuite patch committed: Load timeout.exp

2013-02-18 Thread Matthias Klose
Am 09.02.2013 19:24, schrieb Ian Lance Taylor: > On Sat, Feb 9, 2013 at 3:18 AM, Andreas Schwab wrote: >> Ian Lance Taylor writes: >> >>> 2013-02-08 Ian Lance Taylor >>> >>> * lib/go.exp: Load timeout.exp. >> >> ERROR: Couldn't find library file timeout.exp. > > Sorry, I was told on IRC

Re: libsanitizer merge from upstream r175042

2013-02-18 Thread Konstantin Serebryany
On Mon, Feb 18, 2013 at 12:20 PM, Jakub Jelinek wrote: > On Fri, Feb 15, 2013 at 07:39:28AM -0800, Ian Lance Taylor wrote: >> On Thu, Feb 14, 2013 at 11:45 PM, Konstantin Serebryany >> wrote: >> > >> > Unfortunately, the test does not work if gold is the system linker. >> > Any suggestion on how

Re: [cxx-conversion] Add Record Builder Class

2013-02-18 Thread Nathan Sidwell
On 02/12/13 19:47, Lawrence Crowl wrote: Add class record_builder to ease construction of records and unions. Use it in some appropriate places. Nathan please review the vxworks changes. config/vxworks.c Replace vxworks_emutls_var_fields() with vxworks_emutls_object_type(). U

Re: [PATCH] Add new debug_bb_range debugging function

2013-02-18 Thread Richard Biener
On Sat, 16 Feb 2013, Dodji Seketeli wrote: > Jakub Jelinek writes: > > > On Sat, Feb 16, 2013 at 10:40:43AM +0100, Dodji Seketeli wrote: > >> --- a/gcc/cfg.c > >> +++ b/gcc/cfg.c > >> @@ -663,6 +663,21 @@ debug_bb_n (int n) > >>return bb; > >> } > >> > >> +/* Dumps cfg related information

Re: [PATCH][www] Add loop case to non-bugs in bugs.html

2013-02-18 Thread Richard Biener
On Sun, 17 Feb 2013, Gerald Pfeifer wrote: > On Thu, 7 Feb 2013, Richard Biener wrote: > > As promised, a bugs.html entry. > > Looks good to me. > > Now, at the top of the page we have > > Before reporting that GCC compiles your code incorrectly, compile it > with gcc -Wall -Wextra and see

Re: [PATCH][RFC] Add -fno-aggressive-loop-optimizations

2013-02-18 Thread Richard Biener
On Sun, 17 Feb 2013, Gerald Pfeifer wrote: > On Thu, 31 Jan 2013, Richard Biener wrote: > > +GCC now uses more a aggressive analysis to derive an upper bound for > > I think "a " should be omitted here. Yes, that was fixed before commit. > > +the number of iterations of loops using constraints

[arm/embedded-4_7-branch] Merge with gcc-4_7-branch r196107

2013-02-18 Thread Joey Ye
Committed as r196116.

[PATCH] Fix PR56350

2013-02-18 Thread Richard Biener
The issue in PR56350 is two-fold. First, fix_loop_structure mishandles latch edge computation (ugh). Second, RTL if-conversion, when merging a latch block into sth else ends up generating stale latch block references. The following two patches cure that, adding some basic latch block checking.

Re: [PATCH][ARM] use vsel instruction for floating point conditional moves in ARMv8

2013-02-18 Thread Ramana Radhakrishnan
On 01/30/13 09:24, Kyrylo Tkachov wrote: Hi all, This patch uses the new ARMv8 AArch32 vsel instruction to implement conditional moves of floating point numbers. For example, an instruction of the form: vsel.f32 s0, s1, s2 means s0 := cond ? s1 : s2 This can be useful, among oth

[PATCH] Fix PR56366

2013-02-18 Thread Richard Biener
This fixes another case where missed sign-conversions during vectorization need to be handled (outer-loop vectorization induction). Bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2013-02-18 Richard Biener PR tree-optimization/56366 * tree-vect-loop.c (ge

Re: [PATCH] Fix PR56321

2013-02-18 Thread Richard Biener
On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt wrote: > When we remove __builtin_pow statements as part of reassociation, we > have to unlink the associated VDEF. We've always done this when we > directly remove the statement. However, in reassociation the statements > are sometimes modified in p

[PATCH] Fix PR56321

2013-02-18 Thread Richard Biener
This fixes PR56321 - the unlink_stmt_vdef was a no-op as it was called after release_defs. Oops. Committed as obvoious. Richard. 2013-02-18 Richard Biener PR tree-optimization/56321 * tree-ssa-reassoc.c (propagate_op_to_single_use): Properly order SSA name release a

Re: [PATCH] Fix PR56321

2013-02-18 Thread Bill Schmidt
On Mon, 2013-02-18 at 13:05 +0100, Richard Biener wrote: > On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt > wrote: > > When we remove __builtin_pow statements as part of reassociation, we > > have to unlink the associated VDEF. We've always done this when we > > directly remove the statement. How

Re: [PATCH] Disable PCH for -g other than dwarf[234] (PR pch/54117, take 2)

2013-02-18 Thread John David Anglin
The patch successfully disables stabs+PCH on hppa2.0w-hp-hpux11.11 and fixes the testsuite regressions. Also tested on hppa64-hp-hpux11.11 with no regressions. On 2013-02-15 9:08 AM, Jakub Jelinek wrote: On Thu, Feb 14, 2013 at 05:48:57AM -0700, Jeff Law wrote: I think two tests should be su

[committed] Fix PR target/56214

2013-02-18 Thread John David Anglin
The attached change fixes PR target/56214. We need to reject everything but QImode and HImode when reload is in progress and strict is specified. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. Committed to trunk. Dave -- J. David Anglin dave.ang...@nr

[C++ Patch] PR 56373

2013-02-18 Thread Paolo Carlini
Hi, I think submitter is right that with -Wzero-as-null-pointer-constant we want to warn also for zero converted to nullptr_t, not just pointer types. In that case the below is the simplest fix I could figure out, passes testing on x86_64-linux. Otherwise I guess we should simply close the PR

[committed] Work around bug in powf function on HP-UX

2013-02-18 Thread John David Anglin
The powf function on HP-UX incorrectly clobbers floating point register %fr12. It is supposed to be saved and restored by callee. The patch marks the register(s) as call used on HP-UX. We still save and restore the register is the prologue/epilogue, so the runtime isn't changed. Tested on hppa2

Re: [patch] [libffi] do not install libffi library, headers and documentation

2013-02-18 Thread Matthias Klose
Am 12.02.2013 13:45, schrieb Richard Biener: > On Tue, Feb 12, 2013 at 1:44 PM, Richard Biener > wrote: >> On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose wrote: >>> The libffi library, headers and documentation are still installed, although >>> libffi provides separate releases for a long time.

[PATCH] Remove broken powf hack

2013-02-18 Thread John David Anglin
This patch removes the broken powf hack. This problem is now fixed in the PA backend. Tested on hppa2.0w-hp-hpux11.11 and hppa64-hp-hpux11.11. OK for trunk? Dave -- J. David Anglin dave.ang...@nrc-cnrc.gc.ca National Research Council of Canada (613

Re: [PATCH] Disable PCH for -g other than dwarf[234] (PR pch/54117, take 2)

2013-02-18 Thread Jeff Law
On 02/15/13 08:08, Jakub Jelinek wrote: On Thu, Feb 14, 2013 at 05:48:57AM -0700, Jeff Law wrote: I think two tests should be sufficient. First, compile a simple program with -g and verify it generates dwarf2 debug records. Second verify that there aren't any -g options, unless is dwarf2. I'm

[PATCH, ARM][0/n] Split patterns that output multiple assembly instruction

2013-02-18 Thread Greta Yorsh
This sequence of patches aims at cleaning up patterns that output multiple assembly instructions. The first few patches handle some of the patterns in arm.md. [1/n] Add new patterns for subtract with carry. [2/n] Split subdi patterns. [3/n] Split patterns andsi_iorsi3_notsi, abs, cmpdi, and negd

[PATCH,ARM][1/n] New patterns for subtract with carry

2013-02-18 Thread Greta Yorsh
Add patterns to handle various subtract with carry operations. These patterns match RTL insns emitted by splitters for DImode operations such as subdi, negdi, and cmpdi. gcc/ 2013-02-14 Greta Yorsh * config/arm/arm.md (subsi3_carryin, subsi3_carryin_const): New patterns. (su

[PATCH,ARM][2/n] Split subdi patterns

2013-02-18 Thread Greta Yorsh
Convert define_insn into define_insn_and_split for various subdi patterns that output multiple assembly instructions. 2013-02-14 Greta Yorsh * config/arm/arm.md (arm_subdi3): Convert define_insn into define_insn_and_split. (subdi_di_zesidi,subdi_di_sesidi): Likewi

[PATCH,ARM][3/n] Split various patterns

2013-02-18 Thread Greta Yorsh
Convert define_insn into define_insn_and_split for various patterns that output multiple assembly instructions. It appears that preparation statements in define_insn_and_split sometimes are called with which_alternative set to -1 even after reload. Therefore, preparation statements use conditions

[PATCH,ARM][4/n] Add negdi_extend patterns

2013-02-18 Thread Greta Yorsh
This patch adds patterns to handle negation of an extended 32-bit value more efficiently. For example, (set (reg:DI r0) (neg:DI (sign_extend:DI (reg:SI r0))) The compiler currently generates mov r1, r0, asr #31 rsbsr0, r0, #0 rsc r1, r1, #0 and after the patch

[PATCH,ARM][5/n] Split shift di patterns

2013-02-18 Thread Greta Yorsh
Convert define_insn into define_insn_and_split for various DImode shift operations that output multiple assembly instructions. This patch also adds a new pattern for RRX using a new UNSPEC. This pattern matches RTL insns emitted by arm_ashrdi3_1bit and arm_lshrdi3_1bit splitters. This patch also a

[PATCH, x86, AVX2] FP reassociation enabling for AVX2 targets

2013-02-18 Thread Igor Zamyatin
Hi! This small change enables FP reassociation for AVX2 processors. This gives ~+1.5% in performance geomean for spec2006FP tests. Bootstrapped/regtested on x86_64-linux, ok for trunk? 2013-02-14 Igor Zamyatin * config/i386/i386.c (initial_ix86_tune_features): Turn on fp rea

[PATCH,ARM][6/n] Split min and max patterns

2013-02-18 Thread Greta Yorsh
Convert define_insn into define_insn_and_split for various min and max patterns that output multiple assembly instructions. Use movsicc to emit RTL. A separate patch will split movsicc. gcc/ 2013-02-14 Greta Yorsh * config/arm/arm.md (arm_smax_insn): Convert define_insn into def

[PATCH, ARM][7/n] Comment on splitting THUMB1 patterns

2013-02-18 Thread Greta Yorsh
This patch adds a comment explaining why it is difficult to split Thumb1 patterns. gcc/ 2013-02-12 Greta Yorsh * config/arm/arm.md: Comment on splitting Thumb1 patterns.diff --git a/gcc/config/arm/arm.md b/gcc/config/arm/arm.md index 64888f9..ce98013 100644 --- a/gcc/config/arm/arm.md +++

Re: PR target/52555: attribute optimize is overriding command line options

2013-02-18 Thread Aldy Hernandez
On 02/16/13 05:19, Richard Sandiford wrote: Looks good to me otherwise, thanks. Implemented all your suggestions. Sorry that SWITCHABLE_TARGETS has been so much hassle. TBH, like Jakub says in the PR, I was hoping things like the optimize attribute could use the target_globals stuff too.

[PATCH] Fix vectorizable_reduction (PR tree-optimization/56350)

2013-02-18 Thread Jakub Jelinek
Hi! On the following testcase, a = a / 8; looks like reasonable reduction statement, but we pattern recognize it as patt_1 = a < 0 ? 7 : 0; patt_2 = a + patt_1; a = patt2 >> 3; and in the first pattern stmt the reduction operand is in the condition of COND_EXPR, not operands as vectorizable_reduct

Re: Go testsuite patch committed: Load timeout.exp

2013-02-18 Thread Ian Lance Taylor
On Mon, Feb 18, 2013 at 12:23 AM, Matthias Klose wrote: > >> As I mentioned on the bug, can you figure out why it fails? The file >> gcc/testsuite/lib/go.exp does a load_lib of several files from that >> directory. Why would they all work but loading timeout.exp fails? > > but as I see now, it s

Re: [PATCH, x86, AVX2] FP reassociation enabling for AVX2 targets

2013-02-18 Thread Uros Bizjak
On Mon, Feb 18, 2013 at 7:44 PM, Igor Zamyatin wrote: > This small change enables FP reassociation for AVX2 processors. This > gives ~+1.5% in performance geomean for spec2006FP tests. > > Bootstrapped/regtested on x86_64-linux, ok for trunk? > > > 2013-02-14 Igor Zamyatin > > * config

[PATCH] Fix a thinko in dumping in ipa-cp.c

2013-02-18 Thread Martin Jambor
Hi, in ipa-cp there is a dumping condition that was intended to trigger for non-aliases and non-thunks but I have lost the negations somewhere when introducing the code. This probably shows this particular dumping is not very crucial but I think it summarizes the initialization well and decided t

[PATCH, PR 55334] Disable IPA-CP on restrict pointers to arrays

2013-02-18 Thread Martin Jambor
Hi, after much pondering about PR 55334 I came to conclusion that no nice fix to the regression could be introduced in stage4. So for the sake of the SPEC 200 benchmark I decided to cripple IPA-CP on restrict pointers to arrays so that the restrict-ness of the memory references which need to be v

Re: [lra] branch merged with trunk

2013-02-18 Thread Steven Bosscher
On Sat, Feb 16, 2013 at 1:55 AM, Vladimir Makarov wrote: > The branch was merged with trunk @ r196091. > > Successfully bootstrapped on x86-64. > > Committed as rev. 196099. Hello Vlad, It looks like this broke bootstrap on ia64 and /home/stevenb/lra_powerpc64_autotester/lra-branch/libgfortran/g

Re: [lra] branch merged with trunk

2013-02-18 Thread Steven Bosscher
On Mon, Feb 18, 2013 at 9:21 PM, Steven Bosscher wrote: > On Sat, Feb 16, 2013 at 1:55 AM, Vladimir Makarov wrote: >> The branch was merged with trunk @ r196091. >> >> Successfully bootstrapped on x86-64. >> >> Committed as rev. 196099. > > Hello Vlad, > > It looks like this broke bootstrap on ia64

RFC: [PATCH,ARM] Fix 56110

2013-02-18 Thread Tilman Sauerbeck
Hi, adding the instruction pattern below fixes my testcase for PR 56110; however I'm not sure if adding a new pattern is the correct way to go. I duplicated the andsi3_compare0_scratch pattern, and lifted the requirement that the 2nd operand be an arm_not_operand. I didn't copy over the clobber pa

Re: RFC: [PATCH,ARM] Fix 56110

2013-02-18 Thread Tilman Sauerbeck
Tilman Sauerbeck [2013-02-18 22:47]: > [...] > + "TARGET_32BIT" > + "@ > + tst%?\\t%0, %1" > [...] I managed to post the wrong diff -- line 2 in the citation should be omitted of course. Sorry. Regards, Tilman -- A: Because it messes up the order in which people normally read text. Q: Why

Re: [lra] branch merged with trunk

2013-02-18 Thread Steven Bosscher
On Mon, Feb 18, 2013 at 10:29 PM, Steven Bosscher wrote: >> /home/stevenb/lra_powerpc64_autotester/lra-branch/libgfortran/generated/pow_i8_i4.c: >> In function 'pow_i8_i4': >> /home/stevenb/lra_powerpc64_autotester/lra-branch/libgfortran/generated/pow_i8_i4.c:75:1: >> internal compiler error: in c

[PATCH] Fix create_expression_by_pieces (PR tree-optimization/56381)

2013-02-18 Thread Jakub Jelinek
Hi! This accesses one past the last initialized argument. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2013-02-18 Jakub Jelinek PR tree-optimization/56381 * tree-ssa-pre.c (create_expression_by_pieces): Fix up last argument to fold

Re: [patch] Fix libstdc++/55043 - issue with nesting unordered_map containing unique_ptr into vector

2013-02-18 Thread Jonathan Wakely
On 16 January 2013 23:57, Jonathan Wakely wrote: > Here's another attempt to fix this regression, I hope this time it > doesn't cause more problems than it solves. > > Instead of specializing is_copy_constructible for the unordered > containers this causes their copy constructors to be deleted if t

Re: [patch] improve comments for libstdc++ hash tables

2013-02-18 Thread Jonathan Wakely
On 19 November 2012 21:35, Jonathan Wakely wrote: > * include/bits/hashtable.h: Improve comments. > * include/bits/hashtable_policy.h: Likewise. > > Tested x86_64-linux, committed to trunk. Attached version committed to the 4.7 branch. commit 8db4c698a78df8c150ea3353aaed254448620fb

Re: [v3] fix ADL bugs in

2013-02-18 Thread Jonathan Wakely
On 27 March 2012 21:25, Jonathan Wakely wrote: >* include/std/functional (mem_fn): Qualify to prevent ADL. >* testsuite/20_util/function_objects/mem_fn/adl.cc: New. > > Tested x86_64-linux, committed to trunk. Not a regression but should > be safe for all branches. Attached patch

Re: PR target/52555: attribute optimize is overriding command line options

2013-02-18 Thread Jakub Jelinek
On Mon, Feb 18, 2013 at 12:50:59PM -0600, Aldy Hernandez wrote: > OK pending tests? > PR target/52555 > * genopinit.c (raw_optab_handler): Use this_fn_optabs. > (swap_optab_enable): Same. > (init_all_optabs): Use argument instead of global. > * tree.h (struct tree_opt

Re: [PATCH] Temporarily revert Steven's PCH changes for 4.8 (PR pch/54117)

2013-02-18 Thread Joel Brobecker
> > >> AFAICT, for gcc+gas it should already work with binutils that include > > >> the AdaCore patch for DWARF support. But this has apparently not been > > >> tested with AIX ld, and there are AdaCore local patches pending. > > >> http://sourceware.org/ml/binutils/2011-04/msg00250.html > > >> htt

Re: [PATCH] Fix create_expression_by_pieces (PR tree-optimization/56381)

2013-02-18 Thread Jeff Law
On 02/18/13 15:32, Jakub Jelinek wrote: Hi! This accesses one past the last initialized argument. Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2013-02-18 Jakub Jelinek PR tree-optimization/56381 * tree-ssa-pre.c (create_expression_by_pie