Re: [PATCH, committed] SH: Fix PR58314 (unsatisfied constraints)

2013-09-16 Thread Richard Sandiford
Hi Christian, Christian Bruel christian.br...@st.com writes: @@ -6893,11 +6894,14 @@ label: ;; reloading MAC subregs otherwise. For that probably special patterns ;; would be required. (define_insn *movmode_reg_reg - [(set (match_operand:QIHI 0 arith_reg_dest =r) -

Re: [PATCH, committed] * pt.c (make_auto_1): Use input_location rather than BUILTINS_LOCATION.

2013-09-16 Thread Eric Botcazou
Apologies if this was out of context. This was a change suggested and reviewed by Jason (http://gcc.gnu.org/ml/gcc-patches/2013-09/msg01114.html). Then don't repost it on gcc-patches, it is already available on gcc-cvs: http://gcc.gnu.org/ml/gcc-cvs/2013-09/msg00473.html -- Eric Botcazou

Re: [PATCH v3 12/18] convert the Go front end to automatic dependencies

2013-09-16 Thread Tom Tromey
Tom I can try a test build using a setting for CC that rejects -c -o. I did this and the build died pretty early on: /home/tromey/Space/Trunk/Git/bin/my-cc -c -DHAVE_CONFIG_H -g -g -I. -I../../gcc/libiberty/../include -W -Wall -Wwrite-strings -Wc++-compat -Wstrict-prototypes -pedantic

[PATCH, i386]: Update i386.o dependecies

2013-09-16 Thread Uros Bizjak
Hello! Attached patch synchronizes i386.o dependecies with i386.c. 2013-09-16 Uros Bizjak ubiz...@gmail.com * config/i386/t-i386 (i386.o): Update dependecies. Bootstrap on x86_64-pc-linux-gnu is in progress. Uros. Index: config/i386/t-i386

[gomp4, trunk] Two simd fixes

2013-09-16 Thread Jakub Jelinek
Hi! This patch fixes two issues I found on the pr58392.c testcase: 1) we weren't copying decl attributes, so e.g. inside #pragma omp parallel omp simd array temporary arrays lost their attribute and weren't adjusted because of that 2) DR_ALIGNED_TO wasn't reset after resetting DR_OFFSET on simd

Re: Dump framework newline cleanup

2013-09-16 Thread Xinliang David Li
I noticed there are a couple of dump_printf_loc instances in coverage.c not ended with newline. They should be fixed. David On Tue, Sep 10, 2013 at 6:32 AM, Teresa Johnson tejohn...@google.com wrote: On Mon, Sep 9, 2013 at 9:55 PM, Xinliang David Li davi...@google.com wrote: looks fine to me.

Re: Fwd: [PATCH] Scheduling result adjustment to enable macro-fusion

2013-09-16 Thread Wei Mi
Just notice another problem here: processor_type only contains PROCESSOR_COREI7, so I cannot differentiate Westmere and Sandybridge in x86-tune.def, which are different for TARGET_FUSE_ALU_AND_BRANCH. So do I have to separate m_SANDYBRIDGE out from m_COREI7? Yes, please. Thanks. --

Re: Dump framework newline cleanup

2013-09-16 Thread Teresa Johnson
On Mon, Sep 16, 2013 at 10:57 AM, Xinliang David Li davi...@google.com wrote: I noticed there are a couple of dump_printf_loc instances in coverage.c not ended with newline. They should be fixed. I committed this change this morning as r202628. I believe I fixed all the dump_printf_loc calls

Re: [PATCH v3 12/18] convert the Go front end to automatic dependencies

2013-09-16 Thread Paolo Bonzini
Il 16/09/2013 19:23, Tom Tromey ha scritto: Tom I can try a test build using a setting for CC that rejects -c -o. I did this and the build died pretty early on: /home/tromey/Space/Trunk/Git/bin/my-cc -c -DHAVE_CONFIG_H -g -g -I. -I../../gcc/libiberty/../include -W -Wall -Wwrite-strings

Re: [v3] More noexcept for vectors

2013-09-16 Thread Marc Glisse
New version that passed testing. 2013-09-16 Marc Glisse marc.gli...@inria.fr PR libstdc++/58338 * include/bits/stl_vector.h (vector::vector(), vector::vector(const allocator_type)): Merge. (_Vector_impl::_Vector_impl(_Tp_alloc_type const),

[PATCH] Remove empty node (gccint)Cond Exec Macros

2013-09-16 Thread Andreas Schwab
The description of the macro REVERSE_CONDEXEC_PREDICATES_P has been removed from (gccint)Cond Exec Macros in r188983, but the now effectively empty node has been left behind. Tested with make info and installed as obvious. Andreas. * doc/tm.texi.in (Cond Exec Macros): Remove node.

Re: Dump framework newline cleanup

2013-09-16 Thread Xinliang David Li
Looks like there is one missing spot: @@ -349,7 +349,7 @@ get_coverage_counts (unsigned counter, u (flag_guess_branch_prob ? file %s not found, execution counts estimated : file %s not found,

Re: [PATCH] Don't always instrument shifts (PR sanitizer/58413)

2013-09-16 Thread Jakub Jelinek
On Fri, Sep 13, 2013 at 08:01:36PM +0200, Marek Polacek wrote: 2013-09-13 Marek Polacek pola...@redhat.com PR sanitizer/58413 c-family/ * c-ubsan.c (ubsan_instrument_shift): Don't instrument an expression if we can prove it is correct. testsuite/ *

Re: Recent IPA regression with internal functions

2013-09-16 Thread Jan Hubicka
On Sun, Sep 15, 2013 at 09:08:00PM +0200, Jan Hubicka wrote: 2013-09-13 Jakub Jelinek ja...@redhat.com * ipa-prop.c (ipa_compute_jump_functions_for_edge): Return early for internal calls. That seems resonable. I wonder if we want to consider internal calls to form

Re: [PATCH, alpha]: Add alpha.o dependecies

2013-09-16 Thread Tom Tromey
Uros == Uros Bizjak ubiz...@gmail.com writes: Uros Attached patch decleres dependecies for config/alpha/alpha.o. Uros 2013-09-16 Uros Bizjak ubiz...@gmail.com Uros * config/alpha/t-alpha: New. Uros * config.gcc (alpha*-*-linux*): Add alpha/t-alpha to tmake_file. Uros

Re: New GCC options for loop vectorization

2013-09-16 Thread Xinliang David Li
I incorporated all the comments and committed the change (also fixed a test failure with --help=optimizers). thanks, David On Mon, Sep 16, 2013 at 3:07 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Sep 13, 2013 at 6:56 PM, Xinliang David Li davi...@google.com wrote: Updated

Re: RFA: Testsuite: Add exceptions for MSP430

2013-09-16 Thread Mike Stump
On Sep 16, 2013, at 5:10 AM, Nick Clifton ni...@redhat.com wrote: The new MSP430 target has a few limitations which show up as unexpected failures in the GCC testsuite. I would like to apply the patch below to update target-supports.exp with this information. Applying the patch results in

[Patch, Fortran] PR58436 - Fix a CLASS(*) finalization bug

2013-09-16 Thread Tobias Burnus
There were two related issues with the _vtab of CLASS(*): First, it was crashing in the generation of the finalization wrapper. Secondly, one needs to ensure that the _vtab contains a NULL pointer for _final. Build and regtested on x86-64-gnu-linux. OK for the trunk? Tobias

Re: [Patch, Fortran] PR58436 - Fix a CLASS(*) finalization bug

2013-09-16 Thread Tobias Burnus
I forgot, again, to attach the patch. However, to my surprise the email made it through. Contrary to Janus' second patch review of today which didn't went past GCC's mail server. (The same problem I had yesterday: Three emails didn't make it.) Tobias Burnus wrote: There were two related

Re: Dump framework newline cleanup

2013-09-16 Thread Teresa Johnson
Yep, looked too quickly every time and thought the newline after be zero was applying. Here is the patch with the fix. Ok for trunk pending regression testing? 2013-09-16 Teresa Johnson tejohn...@google.com * coverage.c (get_coverage_counts): Add missing newline. Index: coverage.c

Re: New GCC options for loop vectorization

2013-09-16 Thread Xinliang David Li
On Mon, Sep 16, 2013 at 3:13 AM, Richard Biener richard.guent...@gmail.com wrote: On Fri, Sep 13, 2013 at 5:16 PM, Xinliang David Li davi...@google.com wrote: On Fri, Sep 13, 2013 at 1:30 AM, Richard Biener richard.guent...@gmail.com wrote: On Thu, Sep 12, 2013 at 10:31 PM, Xinliang David Li

Re: [Patch, Fortran] PR58436 - Fix a CLASS(*) finalization bug

2013-09-16 Thread Janus Weil
Contrary to Janus' second patch review of today which didn't went past GCC's mail server. Huh, indeed. I hadn't noticed ... (The same problem I had yesterday: Three emails didn't make it.) Any idea what's the reason for this? Cheers, Janus

Re: [buildrobot] [PATCH] r202527: ssa / ssanames restructure broke alpha-linux

2013-09-16 Thread Uros Bizjak
On Mon, Sep 16, 2013 at 4:50 PM, Michael Matz m...@suse.de wrote: Why do you think so? The t-* frags are includes via ... tmake_file=...$(srcdir)/config/i386/t-i386 ... ifneq ($(tmake_file),) include $(tmake_file) endif ... It's just that there's no t-alpha frag yet. No,

[rl78] add RL78/G10 support

2013-09-16 Thread DJ Delorie
This patch adds support for the RL78/G10 variant, which doesn't have register banks like the other RL78 chips. Committed. * config/rl78/rl78.c (rl78_asm_file_start): Specify alternate vregs location for RL78/G10. (rl78_expand_prologue): Avoid SEL on G10.

[PATCH] Testcases for 58431 and 58419

2013-09-16 Thread Jeff Law
Just for completeness. Installed. diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index f606965..7150063 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2013-09-16 Jeff Law l...@redhat.com + + * gcc.c-torture/execute/pr58419.c: New

Re: [PATCH] Fix segfault with inlining

2013-09-16 Thread Jason Merrill
On 09/16/2013 05:28 AM, Richard Biener wrote: which exhibits exactly the behavior you quote - return x is considered throwing an exception. The C++ FE doesn't arrange for TREE_THIS_NOTRAP to be set here (maybe due to this issue you quote?). I haven't been aware of TREE_THIS_NOTRAP, but we

Re: [Patch, Fortran] PR58436 - Fix a CLASS(*) finalization bug

2013-09-16 Thread Tobias Burnus
Janus Weil wrote: (The same problem I had yesterday: Three emails didn't make it.) Any idea what's the reason for this? No idea. Usually, one gets a bounce message but this time one didn't. Thus, I have no possibility to debug this from my side. #overseers didn't see it as their business

Re: tree if convert pass control

2013-09-16 Thread Xinliang David Li
Ok -- abandon the patch. On Mon, Sep 16, 2013 at 2:04 AM, Richard Biener richard.guent...@gmail.com wrote: On Sat, Sep 14, 2013 at 8:10 AM, Xinliang David Li davi...@google.com wrote: tree if conversion is an enabler pass for vectorization, so by default, it is only turned on when

Re: [PATCH v3 05/18] convert the C front end to automatic dependencies

2013-09-16 Thread Tom Tromey
Paolo The series looks okay to me with that change. Two last questions while I'm testing my rebase -- First, do you mind if I resend the whole series? Otherwise I can try to pick out just the patches that have changed, plus the additional patches. Second, should I get a separate ok for the

[rl78] missed G10 bit

2013-09-16 Thread DJ Delorie
Forgot this bit, committed. * config/rl78/vregs.h: Add G10 register definitions. Index: config/rl78/vregs.h === --- config/rl78/vregs.h (revision 202637) +++ config/rl78/vregs.h (working copy) @@ -8,12 +8,35 @@ r_2 =

Re: operator new returns nonzero

2013-09-16 Thread Marc Glisse
Nobody has expressed concern for a week, so it may be worth doing an official review ;-) http://gcc.gnu.org/ml/gcc-patches/2013-09/msg00676.html On Mon, 9 Sep 2013, Marc Glisse wrote: I have now tested bootstrap+testsuite and there was no regression. 2013-09-07 Marc Glisse

Re: Using gen_int_mode instead of GEN_INT minor testsuite fallout on MIPS

2013-09-16 Thread Mike Stump
On Sep 16, 2013, at 2:09 AM, Richard Biener richard.guent...@gmail.com wrote: Here is the patch to add precision to partial int modes. I included switching over the rs6000 port to the new scheme so one can get a feel for that it would look like. Can you instead of adding PARTIAL_INT_MODE_N

Re: [PATCH, PR 57748] Check for out of bounds access

2013-09-16 Thread Martin Jambor
Hi, On Sun, Sep 15, 2013 at 06:55:17PM +0200, Bernd Edlinger wrote: Hello Richard, attached is my second attempt at fixing PR 57748. This time the movmisalign path is completely removed and a similar bug in the read handling of misaligned structures with a non-BLKmode is fixed too. There

Re: [PATCH, PowerPC] Fix PR57949 (ABI alignment issue)

2013-09-16 Thread Alan Modra
On Thu, Sep 12, 2013 at 09:33:05AM +0100, Andrew Haley wrote: On 09/12/2013 03:11 AM, Alan Modra wrote: We have precedent for compiling libffi based on gcc preprocessor defines, eg. __NO_FPRS__, so here's a way of making upstream libffi compatible with the various versions of gcc out there.

Fix PR middle-end/57393

2013-09-16 Thread Easwaran Raman
There are two separate root causes for the problem reported in PR 57393. This patch attempts to fix both. First is due to newly created stmts that have the default UID of 0 which are compared with statements with valid UIDs leading to broken dependences. As discussed in an earlier thread, I check

Re: [v3] More noexcept for vectors

2013-09-16 Thread Paolo Carlini
On 09/16/2013 07:32 PM, Marc Glisse wrote: New version that passed testing. Looks good to me, thanks! Paolo.

Re: [PATCH, committed] * pt.c (make_auto_1): Use input_location rather than BUILTINS_LOCATION.

2013-09-16 Thread Adam Butcher
On 16.09.2013 17:54, Eric Botcazou wrote: Apologies if this was out of context. This was a change suggested and reviewed by Jason (http://gcc.gnu.org/ml/gcc-patches/2013-09/msg01114.html). Then don't repost it on gcc-patches, it is already available on gcc-cvs:

[C++ Patch] PR 58435

2013-09-16 Thread Paolo Carlini
Hi, for this kind of code: template template typename class F, typename T struct apply { typedef FT type; }; template typename T using map = const T; static_assert(std::is_sameapplymap, int::type, const int::value, ); line #11542 of pt.c, in tsubst is reached: r =

Re: [PATCH, committed] PR 57589: Fix logic error in driver-rs6000.c

2013-09-16 Thread Alan Modra
On Tue, Jun 11, 2013 at 07:43:05PM -0400, Michael Meissner wrote: Ah, you are correct. I misread the code, thinking it was returning a string, and not a pointer to the string in memory. http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00640.html Patch reverted. -- Alan Modra Australia

RE: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions

2013-09-16 Thread bin.cheng
-Original Message- From: Richard Earnshaw Sent: Thursday, September 12, 2013 11:24 PM To: Bin Cheng Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH ARM]Extend thumb1_reorg to save more comparison instructions On 18/04/13 06:34, Bin Cheng wrote: Sorry for the delay, I've been

Re: Using gen_int_mode instead of GEN_INT minor testsuite fallout on MIPS

2013-09-16 Thread DJ Delorie
m32c's PSImode is 24-bits, why does it have 32 in the macro? /* 24-bit pointers, in 32-bit units */ -PARTIAL_INT_MODE (SI); +PARTIAL_INT_MODE_NAME (SI, 32, PSI);

RE: [PATCH GCC]Catch more MEM_REFs sharing common addressing part in gimple strength reduction

2013-09-16 Thread bin.cheng
-Original Message- From: gcc-patches-ow...@gcc.gnu.org [mailto:gcc-patches- ow...@gcc.gnu.org] On Behalf Of bin.cheng Sent: Thursday, September 12, 2013 2:13 PM To: 'Bill Schmidt'; Yufeng Zhang; Yufeng Zhang Cc: Richard Biener; GCC Patches Subject: RE: [PATCH GCC]Catch more

<    1   2