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 konstantin.s.serebry...@gmail.com wrote: Unfortunately, the test does not work if gold is the system linker. Any suggestion on how to make the test work with either

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 sch...@linux-m68k.org wrote: Ian Lance Taylor i...@google.com writes: 2013-02-08 Ian Lance Taylor i...@google.com * lib/go.exp: Load timeout.exp. ERROR: Couldn't find library file

Re: libsanitizer merge from upstream r175042

2013-02-18 Thread Konstantin Serebryany
On Mon, Feb 18, 2013 at 12:20 PM, Jakub Jelinek ja...@redhat.com 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 konstantin.s.serebry...@gmail.com wrote: Unfortunately, the test does not work if gold is the

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().

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 ja...@redhat.com 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 about

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 pBefore reporting that GCC compiles your code incorrectly, compile it with codegcc -Wall -Wextra/code 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: +pGCC 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 imposed

[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: vselcond.f32 s0, s1, s2 means s0 := cond ? s1 : s2 This can be useful,

[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 rguent...@suse.de PR tree-optimization/56366 *

Re: [PATCH] Fix PR56321

2013-02-18 Thread Richard Biener
On Fri, Feb 15, 2013 at 6:53 PM, Bill Schmidt wschm...@linux.vnet.ibm.com 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

[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 rguent...@suse.de PR tree-optimization/56321 * tree-ssa-reassoc.c (propagate_op_to_single_use): Properly order

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 wschm...@linux.vnet.ibm.com 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

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

[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

[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

[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

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 richard.guent...@gmail.com wrote: On Tue, Feb 12, 2013 at 1:30 PM, Matthias Klose d...@ubuntu.com wrote: The libffi library, headers and documentation are still installed, although libffi provides

[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

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 -gfoo options, unless foo is

[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

[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 greta.yo...@arm.com * config/arm/arm.md (arm_subdi3): Convert define_insn into define_insn_and_split.

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

[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 igor.zamya...@intel.com * config/i386/i386.c

[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 greta.yo...@arm.com * config/arm/arm.md (arm_smax_insn): Convert

[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 greta.yo...@arm.com * 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 ---

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

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 d...@ubuntu.com 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

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 izamya...@gmail.com 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

[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

[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

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

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 and

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

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 is

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

[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 ja...@redhat.com PR tree-optimization/56381 * tree-ssa-pre.c (create_expression_by_pieces): Fix up last

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 the

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 8db4c698a78df8c150ea3353aaed254448620fba

Re: [v3] fix ADL bugs in functional

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

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

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 ja...@redhat.com PR tree-optimization/56381 * tree-ssa-pre.c