Re: [PATCH,GRAPHITE] Fix for P1 bug 58028

2014-02-27 Thread Richard Biener
On Wed, Feb 26, 2014 at 10:13 PM, Tobias Grosser tob...@grosser.es wrote: On 02/26/2014 10:09 PM, Mircea Namolaru wrote: This patch fixes the libgomp problems: http://gcc.gnu.org/bugzilla/show_bug.cgi?id=58028 2014-02-26 Tobias Grosser tob...@grosser.es Mircea Namolaru mnamo...@inria.fr

Re: PR60147: fix ICE with DECL_NAMELIST and tree-pretty-printer.c

2014-02-27 Thread Richard Biener
On Wed, Feb 26, 2014 at 11:19 PM, Tobias Burnus bur...@net-b.de wrote: Dear all, as suggested by Richard, it now only prints the namelist name and no longer the variables of the namelist. Bootstrapped on x86-64-gnu-linux and currently regtesting. OK for the trunk when it succeeds? Works

Re: [PR debug/57232] don't record non-fixed reg as CFA base value

2014-02-27 Thread Richard Biener
On Thu, Feb 27, 2014 at 6:28 AM, Alexandre Oliva aol...@redhat.com wrote: Some ports were failing an assertion check that was supposed to make sure some RTX created a new VALUE, rather than reuse an existing one in the cselib tables. The reason the value was already there was that we'd

Re: [PR debug/59992 #1/2] avoid quadratic behavior for the removal of useless values

2014-02-27 Thread Richard Biener
On Thu, Feb 27, 2014 at 6:54 AM, Alexandre Oliva aol...@redhat.com wrote: We indirectly call remove_useless_values quite often during vt_initialize; at least once per extended basic block. On functions with thousands of small basic blocks, each adding permanent and temporary entries to the

Re: [PATCH, GOMP4] Fix OpenACC async clause

2014-02-27 Thread Thomas Schwinge
Hi! On Wed, 26 Feb 2014 19:32:35 +0400, Ilmir Usmanov i.usma...@samsung.com wrote: This patch is pretty obvious. Currently ASYNC clause cannot have integer-expression-list. Patch fixes this. OK for gomp4 branch? From df76a29ebf869687209d7a606e243624cc136dbc Mon Sep 17 00:00:00 2001 From:

Re: [PATCH][AArch64] Use Cortex-A57 rtx costs for the generic CPU

2014-02-27 Thread Richard Earnshaw
On 26/02/14 14:48, Kyrill Tkachov wrote: Hi all, The generic rtx cost table was written with AArch32 cores in mind. It would be a better idea to use the Cortex-A57 costs for the generic CPU in aarch64. That way we schedule for the Cortex-A53 and do instruction selection for the Cortex-A57.

Re: PR60147: fix ICE with DECL_NAMELIST and tree-pretty-printer.c

2014-02-27 Thread Tobias Burnus
Hi Richard, hi all, I wrote: @@ -1713,21 +1733,24 @@ dump_generic_node (pretty_printer *buffer, tree node, int spc, int flags, case BIND_EXPR: ... for (op0 = BIND_EXPR_VARS (node); op0; op0 = DECL_CHAIN (op0)) { -print_declaration (buffer, op0,

Re: [PING^3][PATCH] Add a couple of dialect and warning options regarding Objective-C instance variable scope

2014-02-27 Thread Dimitris Papavasiliou
Ping! On 02/20/2014 12:11 PM, Dimitris Papavasiliou wrote: Hello all, Pinging this patch review request again. See previous messages quoted below for details. Regards, Dimitris On 02/13/2014 04:22 PM, Dimitris Papavasiliou wrote: Hello, Pinging this patch review request. Can someone

Re: [PATCH,GRAPHITE] Fix for P1 bug 58028

2014-02-27 Thread Tobias Burnus
Hi Tobias and Mircea, On 02/26/2014 10:09 PM, Mircea Namolaru wrote: + /* Do not remove scalar dimensions. Cloog be default removes scalar + dimensions very early from the input schedule. However, they are Small nit: Instead of 'be default' it should be 'by default' Tobias

Re: PR60147: fix ICE with DECL_NAMELIST and tree-pretty-printer.c

2014-02-27 Thread Richard Biener
On Thu, Feb 27, 2014 at 10:37 AM, Tobias Burnus tobias.bur...@physik.fu-berlin.de wrote: Hi Richard, hi all, I wrote: @@ -1713,21 +1733,24 @@ dump_generic_node (pretty_printer *buffer, tree node, int spc, int flags, case BIND_EXPR: ... for (op0 = BIND_EXPR_VARS (node);

Re: [PATCH, GOMP4] Fix OpenACC async clause

2014-02-27 Thread Ilmir Usmanov
Hi Thomas! Fixed patch and ChangeLog in attachment. On 27.02.2014 12:59, Thomas Schwinge wrote: GANG/ASYNC/WAIT, for consistency? Fixed. For consistency, also move OMP_CLAUSE_ASYNC_EXPR in gcc/tree.h, and also adapt OMP_CLAUSE_ASYNC in gcc/tree-pretty-print.c:dump_omp_clause? It is already

Re: [PATCH,GRAPHITE] Fix for P1 bug 58028

2014-02-27 Thread Rainer Orth
Tobias Burnus tobias.bur...@physik.fu-berlin.de writes: On 02/26/2014 10:09 PM, Mircea Namolaru wrote: + /* Do not remove scalar dimensions. Cloog be default removes scalar + dimensions very early from the input schedule. However, they are Small nit: Instead of 'be default' it should be 'by

Re: [PATCH, GOMP4] Fix OpenACC async clause

2014-02-27 Thread Thomas Schwinge
Hi Ilmir! On Thu, 27 Feb 2014 13:55:36 +0400, Ilmir Usmanov i.usma...@samsung.com wrote: Fixed patch and ChangeLog in attachment. By the way, what I usually do is to put a change's ChangeLog snippet into that Git commit's commit message, and then once I get to the point where I push something

Re: PR60147: fix ICE with DECL_NAMELIST and tree-pretty-printer.c

2014-02-27 Thread Tobias Burnus
On Thu, Feb 27, 2014 at 10:43:33AM +0100, Richard Biener wrote: hence, one ends up in the code I quote above. That code calls print_declaration() ... Hence, TREE_TYPE() is void_type_node and dereferencing that one leads to an ICE. Sorry, I misremembered. In any case, the following

Re: PR60147: fix ICE with DECL_NAMELIST and tree-pretty-printer.c

2014-02-27 Thread Richard Biener
On Thu, Feb 27, 2014 at 11:39 AM, Tobias Burnus tobias.bur...@physik.fu-berlin.de wrote: On Thu, Feb 27, 2014 at 10:43:33AM +0100, Richard Biener wrote: hence, one ends up in the code I quote above. That code calls print_declaration() ... Hence, TREE_TYPE() is void_type_node and

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Andreas Schwab
r15 = [r14], 8 ;; mov b6 = r15 ld8 r1 = [r14] br.call.sptk.many b0 = b6 ;; mov r1 = r34 .L1: mov ar.pfs = r33 mov b0 = r32 br.ret.sptk.many b0 ;; .endp _Z1fv# .ident GCC: (GNU) 4.9.0 20140227 (experimental

Re: [PATCH, GOMP4] Fix OpenACC async clause

2014-02-27 Thread Ilmir Usmanov
Committed as r208197. -- Ilmir.

Re: [PATCH] Fix PR 60268

2014-02-27 Thread Andreas Schwab
Thanks, this is what I have installed. Andreas. * config/m68k/m68k.c (m68k_option_override): Disable -flive-range-shrinkage for classic m68k. (m68k_override_options_after_change): Likewise. diff --git a/gcc/config/m68k/m68k.c b/gcc/config/m68k/m68k.c index

Re: [PATCH][AARCH64]PR60034

2014-02-27 Thread Marcus Shawcroft
On 21 February 2014 04:24, Kugan kugan.vivekanandara...@linaro.org wrote: Compiling inline asm results in ICE (PR60034). Alignment calculation in aarch64_classify_address for (symbol_ref:DI (*.LANCHOR4) [flags 0x182])) seems wrong here. Hi Kugan, + else if (SYMBOL_REF_FLAGS (sym)) +

[Patch ARM] Fix failing gcc.dg/atomics/atomic-exec-2.c - movmisalignmode expanders.

2014-02-27 Thread Ramana Radhakrishnan
Hi, This is a case where we end up with an ICE in movmisalignmode where mode is DImode because the address so generated is aligned to 32 bits as a result of V_C_E on an _Atomic Complex float temporary to a DImode temporary. The problem also shows up only when you have -mfpu=neon turned on

Re: [PATCH, AArch64] Define __ARM_NEON by default

2014-02-27 Thread Marcus Shawcroft
On 24 February 2014 10:01, Ian Bolton ian.bol...@arm.com wrote: Hi, This is needed for when people are porting their aarch32 code to aarch64. They will have #ifdef __ARM_NEON (as specified in ACLE) and their intrinsics currently won't get used on aarch64 because it's not defined there by

Re: [PING][PATCH][AARCH64]Resolves testsuite/gcc.target/aarch64/aapcs64/ret-func-1.c regression

2014-02-27 Thread Marcus Shawcroft
On 24 February 2014 09:49, Renlin Li renlin...@arm.com wrote: gcc/testsuite/ChangeLog: 2014-02-03 Renlin Li renlin...@arm.com * gcc.target/aarch64/aapcs64/validate_memory.h: Move f32in64 and i32in128 cases outside special big-endian processing block. This is is a fix for a

Re: [PATCH][AARCH64]Adjust address with offset assembler format

2014-02-27 Thread Marcus Shawcroft
On 12 February 2014 16:10, Renlin Li renlin...@arm.com wrote: gcc/ChangeLog: 2014-02-12 Renlin Li renlin...@arm.com * config/aarch64/aarch64.c (aarch64_print_operand_address): Adjust the output asm format by adding a space between base register and offset. OK for stage-1.

Re: copyright dates in binutils (and includes/)

2014-02-27 Thread Joel Brobecker
Hi Alan, My two cents... --- a/bfd/elf32-sparc.c +++ b/bfd/elf32-sparc.c @@ -1,7 +1,5 @@ /* SPARC-specific support for 32-bit ELF - Copyright 1993, 1994, 1995, 1996, 1997, 1998, 1999, 2000, 2001, 2002, - 2003, 2004, 2005, 2006, 2007, 2010, 2011 - Free Software Foundation, Inc. +

Re: [AArch64] Fix possible wrong code generation when comparing DImode values.

2014-02-27 Thread Marcus Shawcroft
On 24 February 2014 18:17, James Greenhalgh james.greenha...@arm.com wrote: gcc/ 2014-02-24 James Greenhalgh james.greenha...@arm.com * config/aarch64/aarch64-simd.md (aarch64_cmoptabdi): Always split. (*aarch64_cmoptabdi): New. (aarch64_cmtstdi): Always

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Jason Merrill
Hmm, I wonder why we aren't devirtualizing that call on ia64. Does it work with -O3? Jason

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Andreas Schwab
Jason Merrill ja...@redhat.com writes: Hmm, I wonder why we aren't devirtualizing that call on ia64. Does it work with -O3? That doesn't change anything fundamentally. Andreas. -- Andreas Schwab, SUSE Labs, sch...@suse.de GPG Key fingerprint = 0196 BAD8 1CE9 1970 F4BE 1748 E4D4 88E3 0EEA

[Patch ARM] Allow any register for DImode values in Thumb2.

2014-02-27 Thread Ramana Radhakrishnan
Hi I noticed that for T32 we don't allow any old register for DImode values. The restriction of an even register is true only for ARM state because the ISA doesn't allow any old register in this place. In a few large .i files that I had knocking about, noticed a nice drop in stack usage and

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Richard Biener
On Thu, Feb 27, 2014 at 2:53 PM, Andreas Schwab sch...@suse.de wrote: Jason Merrill ja...@redhat.com writes: Hmm, I wonder why we aren't devirtualizing that call on ia64. Does it work with -O3? That doesn't change anything fundamentally. I think the vtable lookup sequence is different and

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Jason Merrill
On 02/27/2014 09:03 AM, Richard Biener wrote: Jason Merrill ja...@redhat.com writes: Hmm, I wonder why we aren't devirtualizing that call on ia64. I think the vtable lookup sequence is different and nobody cared to adjust the gimple matcher to also match the ia64 sequence. Ah. So xfail on

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Richard Biener
On Thu, Feb 27, 2014 at 3:51 PM, Jason Merrill ja...@redhat.com wrote: On 02/27/2014 09:03 AM, Richard Biener wrote: Jason Merrill ja...@redhat.com writes: Hmm, I wonder why we aren't devirtualizing that call on ia64. I think the vtable lookup sequence is different and nobody cared to

Re: [PATCHv2/AARCH64 2/3] Fix TLS for ILP32.

2014-02-27 Thread Yufeng Zhang
Hi, On 02/26/14 02:25, Andrew Pinski wrote: Hi, With ILP32, some simple usage of TLS variables causes an unrecognizable instruction due to needing to use SImode for loading pointers from memory. This fixes the three (tlsie_small, tlsle_small, tlsdesc_small) patterns to support SImode for

Re: [PATCHv2/AARCH64 3/3] Support ILP32 multi-lib

2014-02-27 Thread Yufeng Zhang
On 02/26/14 02:25, Andrew Pinski wrote: Hi, This is the final patch which adds support for the dynamic linker and multi-lib directories for ILP32. I did not change multi-arch support as I did not know what it should be changed to and internally here at Cavium, we don't use multi-arch.

[Ping] Re: [C++ Patch] PR 60253

2014-02-27 Thread Paolo Carlini
On 02/21/2014 12:14 PM, Paolo Carlini wrote: Hi, unless we have reasons to believe that the diagnostic quality could regress in some circumstances, we can easily resolve this ICE on invalid regression by always returning error_mark_node after error (thus outside SFINAE too). Pinging this

Fwd: [PATCH, ARM] Improve 64 bit division performance

2014-02-27 Thread Charles Baylis
[resending as text/plain] Hi These patches optimise 64 bit division by removing the use of the __gnu_[u]ldivmod_helper functions and hence avoiding the redundant calculation of the remainder in those functions. Bootstrapped, tested and checked for arm-unknown-linux-gnueabihf. Benchmarked on

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Martin Jambor
On Thu, Feb 27, 2014 at 04:00:22PM +0100, Richard Biener wrote: On Thu, Feb 27, 2014 at 3:51 PM, Jason Merrill ja...@redhat.com wrote: On 02/27/2014 09:03 AM, Richard Biener wrote: Jason Merrill ja...@redhat.com writes: Hmm, I wonder why we aren't devirtualizing that call on ia64. I

C++ PATCH for c++/60353 (bogus error with unnamed class)

2014-02-27 Thread Jason Merrill
The code I factored out of expand_or_defer_fn_1 into tentative_decl_linkage is intended for use with functions that are defined; using it on functions that have not been defined causes problems. Tested x86_64-pc-linux-gnu, applying to trunk. commit 2e63bb7138e59e7e04e2649fb3d8086fffc238a7

Re: [PATCHv2/AARCH64 2/3] Fix TLS for ILP32.

2014-02-27 Thread Marcus Shawcroft
On 26 February 2014 02:25, Andrew Pinski apin...@cavium.com wrote: Hi, With ILP32, some simple usage of TLS variables causes an unrecognizable instruction due to needing to use SImode for loading pointers from memory. This fixes the three (tlsie_small, tlsle_small, tlsdesc_small) patterns to

patch to fix PR59222

2014-02-27 Thread Vladimir Makarov
The following patch fixes http://gcc.gnu.org/bugzilla/show_bug.cgi?id=59222 The patch was successfully bootstrapped and tested on x86/x86-64. Committed as rev. 208201. 2014-02-27 Vladimir Makarov vmaka...@redhat.com PR target/59222 * lra.c (lra_emit_add): Check SUBREG too.

Re: [C++ Patch] PR 60253

2014-02-27 Thread Jason Merrill
OK. Jason

Re: [C++ patch] for C++/52369

2014-02-27 Thread Jason Merrill
On 02/23/2014 02:36 PM, Fabien ChĂȘne wrote: * cp/method.c (walk_field_subobs): improve the diagnostic locations for both REFERENCE_TYPEs and non-static const members. It's important to have the error location be the place where the actual problem is, namely the constructor

Re: copyright dates in binutils (and includes/)

2014-02-27 Thread Joseph S. Myers
On Thu, 27 Feb 2014, Joel Brobecker wrote: I should mention, however, that for us to use ranges like this, the FSF asked us to add a note explaining that the copyright years could be abbreviated into a range. See gdb/README (at the end). I suspect that you'll need the same note for binutils.

Re: [PATCH/AARCH64 1/3] Add AARCH64 ILP32 PCH support

2014-02-27 Thread Kyle McMartin
On Tue, Feb 25, 2014 at 06:25:11PM -0800, Andrew Pinski wrote: -#elif defined(__aarch64__) +#elif defined(__aarch64__) defined(__LP64__) # define TRY_EMPTY_VM_SPACE 0x10 +#elif defined(__aarch64__) +# define TRY_EMPTY_VM_SPACE 0x6000 awesome. thanks andrew! --kyle

[PATCH] [rtl-optimization/49847] Fix cse to handle cc0 setter and cc0 user in different blocks

2014-02-27 Thread Jeff Law
As discussed in 49847, a few years ago GCC was changed to add EH edges for exceptions that might arise from a floating point comparison. That change made it possible for a cc0-setter and cc0-user to end up in different blocks, separated by a NOTE. After discussing reverting the change,

Re: C++ PATCH for lto/53808 (devirtualization of defaulted virtual dtor)

2014-02-27 Thread Jan Hubicka
On Thu, Feb 27, 2014 at 2:53 PM, Andreas Schwab sch...@suse.de wrote: Jason Merrill ja...@redhat.com writes: Hmm, I wonder why we aren't devirtualizing that call on ia64. Does it work with -O3? That doesn't change anything fundamentally. I think the vtable lookup sequence is

Re: [C++ Patch] PR 60314 (ICE with decltype(auto))

2014-02-27 Thread Jason Merrill
On 02/25/2014 05:03 AM, Paolo Carlini wrote: here we ICE exactly as we did in c++/53756: the only difference is the use of decltype(auto) instead of auto. Now, if we compare is_cxx_auto to is_auto (the front-end helper), evidently there is an inconsistency about the handling of decltype(auto)

Re: [RFA][rtl-optimization/52714] Do not allow combine to create invalid RTL

2014-02-27 Thread Jeff Law
On 02/17/14 02:28, Eric Botcazou wrote: Although it's probably time to concede defeat in this particular case, I don't think we should use the NONJUMP_INSN_P big hammer because we want to eliminate branches if possible. So I'd just add the minimal test to the two existing conditions so as to

Re: [PATCH] [rtl-optimization/49847] Fix cse to handle cc0 setter and cc0 user in different blocks

2014-02-27 Thread H.J. Lu
On Thu, Feb 27, 2014 at 11:02 AM, Jeff Law l...@redhat.com wrote: As discussed in 49847, a few years ago GCC was changed to add EH edges for exceptions that might arise from a floating point comparison. That change made it possible for a cc0-setter and cc0-user to end up in different blocks,

Re: [PATCH] [rtl-optimization/49847] Fix cse to handle cc0 setter and cc0 user in different blocks

2014-02-27 Thread H.J. Lu
On Thu, Feb 27, 2014 at 11:50 AM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Feb 27, 2014 at 11:02 AM, Jeff Law l...@redhat.com wrote: As discussed in 49847, a few years ago GCC was changed to add EH edges for exceptions that might arise from a floating point comparison. That change made it

[gomp4] Gimplification: Merge gimplify_oacc_parallel into gimplify_omp_workshare. (was: [gomp4 9/9] OpenACC: Basic support for #pragma acc parallel.)

2014-02-27 Thread Thomas Schwinge
Hi! On Wed, 6 Nov 2013 20:53:00 +0100, I wrote: gcc/ * gimplify.c [...] (gimplify_oacc_parallel): New function. To get rid of code duplication, I have applied the following in r208206: commit 9ffb216dd43bda84f56ce7fe68ae15cc08110924 Author: tschwinge

Re: [PR debug/59992 #2/2] preserve permanent values in a separate table

2014-02-27 Thread Richard Henderson
On 02/26/2014 10:09 PM, Alexandre Oliva wrote: Regstrapped on x86_64-linux-gnu and i686-linux-gnu, along with the other patch for PR debug/59992. Ok to install? for gcc/ChangeLog PR debug/59992 * cselib.c (cselib_hasher::equal): Special-case VALUE lookup.

[C++ Patch] PR 58610

2014-02-27 Thread Paolo Carlini
Hi, in this regression, we ICE in explain_non_literal_class in: if (DECL_DELETED_FN (fn)) maybe_explain_implicit_delete (fn); because fn is a TEMPLATE_DECL. Now, if I read the comment which provides rationale for LANG_DECL_FN_CHECK: /* We want to be able to check

Re: [PATCH,GRAPHITE] Fix for P1 bug 58028

2014-02-27 Thread Mircea Namolaru
Thanks for comments - updated the patch (fixed my e-mail address too :-)). 2014-02-26 Tobias Grosser tob...@grosser.es Mircea Namolaru mircea.namol...@inria.fr Fix for bug 58028 * graphite-clast-to-gimple.c (set_cloog_options): Don't remove scalar

Re: [PATCH] [rtl-optimization/49847] Fix cse to handle cc0 setter and cc0 user in different blocks

2014-02-27 Thread Jeff Law
On 02/27/14 12:50, H.J. Lu wrote: On Thu, Feb 27, 2014 at 11:02 AM, Jeff Law l...@redhat.com wrote: As discussed in 49847, a few years ago GCC was changed to add EH edges for exceptions that might arise from a floating point comparison. That change made it possible for a cc0-setter and

Re: [PATCH,GRAPHITE] Fix for P1 bug 58028

2014-02-27 Thread Rainer Orth
Hi Mircea, two last nits: 2014-02-26 Tobias Grosser tob...@grosser.es Mircea Namolaru mircea.namol...@inria.fr Fix for bug 58028 Please write this as PR tree-optimization/58028 instead. This way, the commit triggers an update to the bugzilla bug.

Re: [jit] Major API change: blocks rather than labels

2014-02-27 Thread David Malcolm
On Thu, 2014-02-27 at 17:11 -0500, David Malcolm wrote: [...] With this commit, the API changes to using basic blocks instead: blocks are created within functions, and statements are added to blocks, rather than to functions. [...] I've also ported the jittest example to the new API, as of

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread H.J. Lu
On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng bin.ch...@arm.com wrote: Hi, This patch is to fix regression reported in PR60280 by removing forward loop headers/latches in cfg cleanup if possible. Several tests are broken by this change since cfg cleanup is shared by all optimizers. Some tests

Re: [Patch, testsuite]: Allow MicroBlaze .weakext pattern in regex match

2014-02-27 Thread Michael Eager
On 02/26/14 16:29, David Holsgrove wrote: Hi Mike S., Michael E., -Original Message- From: Mike Stump [mailto:mikest...@comcast.net] Sent: Friday, 21 February 2014 6:17 am To: David Holsgrove Cc: gcc-patches@gcc.gnu.org; Michael Eager (ea...@eagerm.com); Vidhumouli Hunsigida; Nagaraju

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread Hans-Peter Nilsson
On Tue, 25 Feb 2014, bin.cheng wrote: Hi, This patch is to fix regression reported in PR60280 by removing forward loop headers/latches in cfg cleanup if possible. Several tests are broken by this change since cfg cleanup is shared by all optimizers. Some tests has already been fixed by

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread Bin.Cheng
Thanks for reporting this, I will look into it. Thanks, bin On Fri, Feb 28, 2014 at 8:52 AM, H.J. Lu hjl.to...@gmail.com wrote: On Mon, Feb 24, 2014 at 9:12 PM, bin.cheng bin.ch...@arm.com wrote: Hi, This patch is to fix regression reported in PR60280 by removing forward loop headers/latches

Re: [PATCH GCC]Allow cfgcleanup to remove forwarder loop preheaders and latches

2014-02-27 Thread Bin.Cheng
Sorry, I didn't test it against logical_op_short_circuit target. I will look into this PR. Thanks, bin On Fri, Feb 28, 2014 at 9:34 AM, Hans-Peter Nilsson h...@bitrange.com wrote: On Tue, 25 Feb 2014, bin.cheng wrote: Hi, This patch is to fix regression reported in PR60280 by removing

RE: [patch] [arm] Fix PR60169 - thumb1 far jump

2014-02-27 Thread Joey Ye
Ping. OK for trunk and 4.8? -Original Message- From: Joey Ye [mailto:joey...@arm.com] Sent: 21 February 2014 19:32 To: gcc-patches@gcc.gnu.org Subject: [patch] [arm] Fix PR60169 - thumb1 far jump Patch http://gcc.gnu.org/ml/gcc-patches/2012-12/msg01229.html introduced this ICE:

[PATCH] [libgcc,arm] Fix PR 60166 - NAN fraction bits

2014-02-27 Thread Joey Ye
This patch is a mirror copy from approved patch in glibc: http://sourceware.org/ml/libc-alpha/2014-02/msg00741.html OK to trunk, 4.8 and 4.7? ChangeLog.libgcc: * config/arm/sfp-machine.h (_FP_NANFRAC_H, _FP_NANFRAC_S, _FP_NANFRAC_D, _FP_NANFRAC_Q): Set to zero. diff --git

RE: [PATCH v4] PR middle-end/60281

2014-02-27 Thread Bernd Edlinger
Hi, I see the problem too. But I think it is not necessary to change the stack alignment to solve the problem. It appears to me that the code in asan_emit_stack_protection is just wrong. It uses SImode when the memory is not aligned enough for that mode. This would not happen if that code is

Re: [PATCH v4] PR middle-end/60281

2014-02-27 Thread lin zuojian
Hi Bernd, I agree you with the mode problem. And I have not change the stack alignment.What I change is the virtual register base's alignment. Realignment must be make in !STRICT_ALIGNMENT machine,or emitting the efficient code is impossible. For example 4 set mem:QI X,REG:QI Y will not combine