Re: [PATCH,x86] Fix combine for condditional instructions.

2012-12-14 Thread Yuri Rumyantsev
Hi Uros, With your new fix that add if-then-else splitting for memory operand I got expected performance speed-up - +6.7% for Atom and +8.4% for SNB. We need to do all testing this weekend and I will get you our final feedback on Monday. Thanks ahead for all your help. Yuri. 2012/12/13 Uros

RE: [PATCH, AArch64] Make zero_extends explicit for common SImode patterns

2012-12-14 Thread Ian Bolton
Hi Richard, + add\\t%w0, %w2, %w, suxtSHORT:size ^^^ %w1 Got spot. I guess that pattern hasn't fired yet then! I'll fix it. This patch significantly reduces the number of redundant uxtw instructions seen in a variety of programs. (There are further patterns

[PATCH] Fix PR55684

2012-12-14 Thread Richard Biener
Honza left an ICE when we cannot compute number_of_iterations_exit in remove_redundant_iv_tests, but that can for sure happen by design (see bugzilla). The following patch fixes it, bootstrap and regtest pending on x86_64-unknown-linux-gnu. Richard. 2012-12-14 Richard Biener

RE: [PATCH, AArch64] Make zero_extends explicit for common SImode patterns

2012-12-14 Thread Ian Bolton
Hi Richard, + add\\t%w0, %w2, %w, suxtSHORT:size ^^^ %w1 Got spot. I guess that pattern hasn't fired yet then! I'll fix it. Now fixed in v3. I should have said that I am indeed running with REE enabled. It has some impact (about 70 further UXTW removed from

[PATCH] Fix PR55687

2012-12-14 Thread Richard Biener
This fixes PR55687 - there is a latent issue in SCEV analysis. For (int) (short) {(unsigned short) i_2, +, 1 }_4 no_evolution_in_loop_p returns false when asked whether that CHREC has an evolution in loop 4 (which is obviously wrong). The reason is that the machinery is not good at filtering

Re: [PATCH, PR 55355, trunk, 4.6, 4.7] One more host_integerp check in SRA

2012-12-14 Thread Richard Biener
On Fri, Dec 14, 2012 at 1:04 PM, Martin Jambor mjam...@suse.cz wrote: Hi, below are two variants for PR 55355, one for trunk and for 4.7, the second one, without dumping, for 4.6. On 4.6 they fix an ICE for a too-large integer on i686, which perhaps can also happen on 4.7 and trunk even

Re: [PATCH] Make some asan builtins tm_pure (PR sanitizer/55508)

2012-12-14 Thread Torvald Riegel
On Thu, 2012-12-13 at 10:02 +0100, Jakub Jelinek wrote: On Thu, Dec 13, 2012 at 10:38:13AM +0400, Dmitry Vyukov wrote: On Wed, Dec 12, 2012 at 11:50 PM, Jakub Jelinek ja...@redhat.com wrote: Various TM tests ICE when built with -fgnu-tm -fsanitizer=address. The problem is that asan.c pass

Re: [PATCH] Make some asan builtins tm_pure (PR sanitizer/55508)

2012-12-14 Thread Torvald Riegel
On Fri, 2012-12-14 at 13:44 +0400, Dmitry Vyukov wrote: On Thu, Dec 13, 2012 at 1:02 PM, Jakub Jelinek ja...@redhat.com wrote: Various TM tests ICE when built with -fgnu-tm -fsanitizer=address. The problem is that asan.c pass adds calls to builtins that weren't there before and TM is

Re: [PATCH 4/7] s390: Add mode attribute for mode bitsize

2012-12-14 Thread Andreas Krebbel
On Thu, Aug 09, 2012 at 07:31:58PM -0700, Richard Henderson wrote: Constant fold, and less typing than, GET_MODE_BITSIZE with another mode substitution. I did just a minor change in order to avoid a warning in the last hunk. Tested on s390 and s390x with different arch flags. Please apply to

Re: [PATCH 5/7] s390: Implement extzv for z10

2012-12-14 Thread Andreas Krebbel
I've changed the s390_extzv_shift_ok function and have added extzv patterns for zEC12. I also tried to implement an extzvsi expander but ran into some issues which need to be fixed first. Tested on s390 and s390x with z196 and zEC12. Please apply to mainline. Thanks! Bye, -Andreas-

Re: [PATCH 7/7] s390: Generate rnsbg

2012-12-14 Thread Andreas Krebbel
I did the change suggested by Uli in the *insv_rnsbg_srl pattern. Tested on s390 and s390x with z196 and zEC12. Please apply to mainline. Thanks! Bye, -Andreas- gcc/config/s390/s390.md | 55 1 file changed, 55 insertions(+) Index:

Re: [fortran, patch] Allow displaying backtraces from user code

2012-12-14 Thread Janus Weil
Hi all, here is another attempt to make gfortran support user-requested backtraces. A patch in this direction was already proposed by FX in March, but did not make it in so far. It was last discussed in June, cf. http://gcc.gnu.org/ml/fortran/2012-06/msg00131.html and follow-ups, where the

Go patch committed: Better error message for invalid shifts

2012-12-14 Thread Ian Lance Taylor
My understanding of shifts of constants in Go has improved. This patch more accurately reflects the language spec when it comes to reporting errors for invalid shifts. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 9be532343312

Re: [PATCH][Revised] PR55679: skip invalid tests from r194458 on darwin

2012-12-14 Thread Jakub Jelinek
On Fri, Dec 14, 2012 at 10:13:47AM -0500, Jack Howarth wrote: 2012-12-14 Jack Howarth howa...@bromo.med.uc.edu PR sanitizer/55679 * g++.dg/asan/interception-test-1.C: Skip on darwin. * lib/target-supports.exp (check_effective_target_swapcontext): Use

[COMMITTED] Tweak to patch for PR55492

2012-12-14 Thread Richard Henderson
Yvan noticed that the set of memory models has already been validated by expand_builtin_atomic_store, so the test here in expand_atomic_store is fully redundant. Committed. r~ * optabs.c (expand_atomic_store): Elide redundant model test. diff --git a/gcc/optabs.c b/gcc/optabs.c index

Re: [cxx-conversion] Iterator for hash_table, and associated conversions

2012-12-14 Thread Diego Novillo
On 2012-12-13 23:08 , Lawrence Crowl wrote: + class iterator + { + public: +inline iterator (); +inline iterator (value_type **, value_type **); +inline value_type operator * (); +void slide (); +inline iterator operator ++ (); +inline bool operator != (const iterator

C++ PATCH for c++/42315 (ICE after error with invalid array init)

2012-12-14 Thread Jason Merrill
In this bug, when we see the invalid initializer we changed the type of x to be error_mark_node. But we had already built an ARRAY_REF to it using the incomplete type. Then the back end gets confused by this invalid expression wrapped in a valid one. Fixed by not messing with the type of x.

Re: [ARM] Turning off 64bits ops in Neon and gfortran/modulo-scheduling problem

2012-12-14 Thread Christophe Lyon
Ping^2? On 7 December 2012 09:34, Christophe Lyon christophe.l...@linaro.org wrote: Ping? http://gcc.gnu.org/ml/gcc-patches/2012-11/msg02558.html Thanks, Christophe.

Re: [PATCH][Cilkplus] Fix pragma simd info being lost

2012-12-14 Thread Jakub Jelinek
On Fri, Dec 14, 2012 at 04:59:02AM +, Iyer, Balaji V wrote: --- tree-vect-loop.c (revision 194483) +++ tree-vect-loop.c (working copy) @@ -234,8 +234,8 @@ if (flag_enable_cilk pragma_simd_assert_requested_p (loop-pragma_simd_index))

RE: [PATCH][Cilkplus] Fix pragma simd info being lost

2012-12-14 Thread Iyer, Balaji V
Hello Jakub, Please see my responses below. Thanks, Balaji V. Iyer. -Original Message- From: Jakub Jelinek [mailto:ja...@redhat.com] Sent: Friday, December 14, 2012 11:59 AM To: Iyer, Balaji V Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH][Cilkplus] Fix pragma simd info

[Patch, AArch64] Add basic recognition for cpu cortex-a53 and cortex-a57

2012-12-14 Thread Yufeng Zhang
Hi, This patch adds the basic recognition for cpu cortex-a53 and cortex-a57. OK to commit? Thanks, Yufeng gcc/ 2012-12-14 Yufeng Zhang yufeng.zh...@arm.com * config/aarch64/aarch64-cores.def: Add entries for cortex-a53 and cortex-a57. *

Re: [PATCH, libgcc] Make possible to disable JCR in crtstuff.c

2012-12-14 Thread H.J. Lu
On Thu, Dec 13, 2012 at 10:56 PM, Joey Ye joey...@arm.com wrote: -Original Message- From: H.J. Lu [mailto:hjl.to...@gmail.com] Sent: Friday, December 14, 2012 11:55 To: Joey Ye Cc: gcc-patches@gcc.gnu.org; Joseph Prostko Subject: Re: [PATCH, libgcc] Make possible to disable JCR in

[patch, libstdc++ testsuite] Shrink more tests for simulator

2012-12-14 Thread Steve Ellcey
Here are four more C++ tests that fail for me when run under the GNU simulator. I would like to shrink them to use less memory in the same way as the other tests that I modified earlier. OK to checkin? Steve Ellcey sell...@mips.com 2012-12-14 Steve Ellcey sell...@mips.com *

RE: [PATCH][Cilkplus] Fix pragma simd info being lost

2012-12-14 Thread Iyer, Balaji V
-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of Jakub Jelinek Sent: Friday, December 14, 2012 12:09 PM To: Iyer, Balaji V Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH][Cilkplus] Fix pragma simd info being lost On

Re: [patch, libstdc++ testsuite] Shrink more tests for simulator

2012-12-14 Thread Jonathan Wakely
On 14 December 2012 18:10, Steve Ellcey wrote: Here are four more C++ tests that fail for me when run under the GNU simulator. I would like to shrink them to use less memory in the same way as the other tests that I modified earlier. OK to checkin? OK, thanks.

[patch, testsuite, committed] Obvious fix for targets with no profiling

2012-12-14 Thread Steve Ellcey
I am going to check in this patch as the obvious fix for PR 55688. I tested it on mips-mti-elf. Steve Ellcey sell...@mips.com 2012-12-14 Steve Ellcey sell...@mips.com PR regression/55688 * g++.dg/other/pr55650.C: Add dg-require-profiling. diff --git

[google][4.7] Allow function reordering linker plugin to separate hot and cold code into different ELF segments

2012-12-14 Thread Sriraman Tallam
Hi Rong, Please review this code. This code allows the function reordering plugin to separate hot and cold code into different ELF segments. This would allow optimizations like mapping the hot code alone to huge pages. With this patch, by default, the plugin maps .text.unlikely sections

[PATCH] Compute and emit working set information from gcov-dump (issue6940061)

2012-12-14 Thread Teresa Johnson
This patch enables the gcov-dump tool to optionally compute and dump the working set information from the counter histogram, via a new -w option. This is useful to help understand and tune how the compiler will use the counter histogram, since it first computes the working set and selects

Re: [PATCH] Use new dump scheme for loop unroll passes

2012-12-14 Thread Sharad Singhai
Teresa, Yes, I didn't take enhancements in google branches into account while porting this patch. In light of these comments, I withdraw this patch and will wait for your patch. Once your patch is in, I will update this patch for regular dumps. To answer your other question, yes, the new dump