RE: [PATCH, AArch64] Implement ffs standard pattern

2012-09-17 Thread Ian Bolton
OK for aarch64-4.7-branch as well? -Original Message- From: Richard Earnshaw Sent: 14 September 2012 18:31 To: Ian Bolton Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, AArch64] Implement ffs standard pattern On 14/09/12 16:26, Ian Bolton wrote: I've implemented the standard

Re: [PATCH, AArch64] Implement fnma, fms and fnms standard patterns

2012-09-17 Thread Richard Earnshaw
On 17/09/12 15:12, Ian Bolton wrote: OK for 4.7 as well? NO! But it is ok for the aarch64 branch of 4.7 :-) R. -Original Message- From: Richard Earnshaw Sent: 14 September 2012 18:18 To: Ian Bolton Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, AArch64] Implement fnma, fms

Re: [PATCH, AArch64] Implement ffs standard pattern

2012-09-17 Thread Richard Earnshaw
On 17/09/12 15:18, Ian Bolton wrote: OK for aarch64-4.7-branch as well? yes. R. -Original Message- From: Richard Earnshaw Sent: 14 September 2012 18:31 To: Ian Bolton Cc: gcc-patches@gcc.gnu.org Subject: Re: [PATCH, AArch64] Implement ffs standard pattern On 14/09/12 16:26,

Re: Decltype and opaque vectors

2012-09-17 Thread Jason Merrill
On 09/15/2012 09:01 AM, Marc Glisse wrote: + /* For vector types, pick a non-opaque variant. */ + if (TREE_CODE (type) == VECTOR_TYPE) + type = cp_build_qualified_type (TYPE_MAIN_VARIANT (type), + cp_type_quals (type)); I

Re: [Patch ARM] big-endian support for Neon vext tests

2012-09-17 Thread Christophe Lyon
On 17 September 2012 14:56, Richard Earnshaw rearn...@arm.com wrote: On 05/09/12 23:14, Christophe Lyon wrote: Hello, Although the recent optimization I have committed to use Neon vext instruction for suitable builtin_shuffle calls does not support big-endian yet, I have written a patch to

Re: [Patch ARM] big-endian support for Neon vext tests

2012-09-17 Thread Richard Earnshaw
On 17/09/12 16:13, Christophe Lyon wrote: On 17 September 2012 14:56, Richard Earnshaw rearn...@arm.com wrote: On 05/09/12 23:14, Christophe Lyon wrote: Hello, Although the recent optimization I have committed to use Neon vext instruction for suitable builtin_shuffle calls does not support

[PATCH] Add missing include file in Makefile

2012-09-17 Thread Florian Weimer
The attached patch is required so that plug-ins can include gimple.h. I tested that cfg-flags.def is actually installed after this change. Okay for trunk? -- Florian Weimer / Red Hat Product Security Team 2012-09-17 Florian Weimer fwei...@redhat.com * Makefile.in (BASIC_BLOCK_H): Add

[AArch64] Merge from upstream trunk r191370

2012-09-17 Thread Sofiane Naci
Hi, I have just merged upstream trunk on the aarch64-branch up to r191370. Thanks Sofiane

[Patch,avr,committed]: Tidy up CONST_FIXED_P usage.

2012-09-17 Thread Georg-Johann Lay
http://gcc.gnu.org/viewcvs?view=revisionrevision=191393 Committed as obvious. CONST_FIXED_P is promoted to rtl.h now. Johann * config/avr/avr.c (CONST_FIXED_P): Remove define. * config/avr/avr.md (addmode3): Use CONST_FIXED_P. Index: config/avr/avr.md

Re: [patch, mips] Patch for new mips triplet - mips-mti-elf

2012-09-17 Thread Steve Ellcey
On Sun, 2012-09-16 at 09:22 +0100, Richard Sandiford wrote: +/* This is idential to sde.h except for the ABI setting which defaults + to 32 instead of n32 on 32 bit architectures and the addition of + MIPS_ISA_SYNCI_SPEC. */ SDE defaults to o32 (-mabi=32) for 32-bit targets and n32

Re: [Patch ARM] big-endian support for Neon vext tests

2012-09-17 Thread Christophe Lyon
On 17 September 2012 17:21, Richard Earnshaw rearn...@arm.com wrote: On 17/09/12 16:13, Christophe Lyon wrote: On 17 September 2012 14:56, Richard Earnshaw rearn...@arm.com wrote: On 05/09/12 23:14, Christophe Lyon wrote: Hello, Although the recent optimization I have committed to use Neon

[DOC] Update -feliminate-unused-debug-types description

2012-09-17 Thread Paul_Koning
Currently the description of -feliminate-unused-debug-types says that it is off by default. In fact, it is on by default. The attached patch corrects the documentation to reflect that. Ok to commit? paul 2012-09-17 Paul Koning n...@arrl.net * doc/invoke.text

Re: [PATCH] PR other/54411: libiberty: objalloc_alloc integer overflows (CVE-2012-3509)

2012-09-17 Thread Ian Lance Taylor
On Mon, Sep 17, 2012 at 2:49 AM, Florian Weimer fwei...@redhat.com wrote: On 09/05/2012 07:31 AM, Ian Lance Taylor wrote: On Wed, Aug 29, 2012 at 10:32 AM, Florian Weimer f...@deneb.enyo.de wrote: This patches fixes an integer overflow in libiberty, which leads to crashes in binutils. The

C++ PATCH for c++/53661 (wrong narrowing error with enum)

2012-09-17 Thread Jason Merrill
When determining whether a conversion from an unscoped enum is a narrowing conversion, we check whether the values of the enum fit into the target type. [dcl.enum] defines the values of an enum with unspecified underlying type to be the values of the smallest bit-field that could hold it, not

[Patch,lto,committed]: Fix PR lto/54598

2012-09-17 Thread Georg-Johann Lay
http://gcc.gnu.org/viewcvs?view=revisionrevision=191394 Committed as preapproved in http://gcc.gnu.org/PR54598#c1 Johann PR lto/54598 * tree-streamer-in.c (unpack_ts_real_cst_value_fields): Use ggc'ed FIXED_VALUE_TYPE instead of struct fixed_value. Index:

Re: Backtrace library [1/3]

2012-09-17 Thread Ian Lance Taylor
On Thu, Sep 13, 2012 at 1:00 PM, Diego Novillo dnovi...@google.com wrote: On 2012-09-11 18:53 , Ian Lance Taylor wrote: 2012-09-11 Ian Lance Taylor i...@google.com * Initial implementation. OK. Thanks for all the reviews. I have committed the libbacktrace library to trunk. I

Re: [PATCH] Rs6000 infrastructure cleanup (switches)

2012-09-17 Thread Michael Meissner
On Sat, Sep 15, 2012 at 06:49:19PM +0200, Andreas Tobler wrote: Tested on powerpc64-freebsd10.0, results here: http://gcc.gnu.org/ml/gcc-testresults/2012-09/msg01403.html I ran the tests with and w/o patch. No diffs seen. Great, thanks. -- Michael Meissner, IBM 5 Technology Place Drive,

Re: [PATCH, ARM] Prefer vld1.64/vst1.64 over vldm/vstm

2012-09-17 Thread Ulrich Weigand
Richard Earnshaw wrote: On 14/09/12 19:02, Ulrich Weigand wrote: * config/arm/arm.c (output_move_neon): Update comment. Use vld1.64/vst1.64 instead of vldm/vstm where possible. (neon_vector_mem_operand): Support double-word modes. * config/arm/neon.md (*neon_mov VD): Call

Re: [PATCH] Rs6000 infrastructure cleanup (switches)

2012-09-17 Thread Michael Meissner
On Mon, Sep 17, 2012 at 09:21:09AM +0100, Iain Sandoe wrote: Hello Michael, On 12 Sep 2012, at 23:43, Michael Meissner wrote: It would be nice to know if this doesn't break the other ppc environments (AIX, Darwin) before I commit it. Are there any problems with this patch? For

PATCH RFA: Print backtrace on ICE

2012-09-17 Thread Ian Lance Taylor
This patch to the diagnostic code uses the new backtrace library to print a backtrace on an ICE. For example, here is the output of a test case I took from a C++ PR: /home/iant/foo2.cc:6:6: internal compiler error: in cp_lexer_new_from_tokens, at cp/parser.c:638 0xec549f internal_error(char

Re: [PATCH, ARM] Use vld1/vst1 to implement vec_set/vec_extract

2012-09-17 Thread Ulrich Weigand
Richard Earnshaw wrote: You confused me for a bit with the reference to line numbers, but I think you must mean lane numbers. Indeed, sorry for the confusion. FTR, I see no reason why GCC would have problems with 64-bit vectors in big-endian mode, it's only 128-bit (and larger) vectors that

Re: [DOC] Update -feliminate-unused-debug-types description

2012-09-17 Thread Ian Lance Taylor
On Mon, Sep 17, 2012 at 8:59 AM, paul_kon...@dell.com wrote: 2012-09-17 Paul Koning n...@arrl.net * doc/invoke.text (-feliminate-unused-debug-types): Update to reflect that this is enabled by default. This is OK. Thanks. Ian

Re: [PATCH] Add missing include file in Makefile

2012-09-17 Thread Ian Lance Taylor
On Mon, Sep 17, 2012 at 8:34 AM, Florian Weimer fwei...@redhat.com wrote: The attached patch is required so that plug-ins can include gimple.h. I tested that cfg-flags.def is actually installed after this change. -BASIC_BLOCK_H = basic-block.h $(PREDICT_H) $(VEC_H) $(FUNCTION_H) cfghooks.h

Re: [patch] Fix PR rtl-optimization/54290

2012-09-17 Thread Ulrich Weigand
Eric Botcazou wrote: Reload 1 of insn #85 inherits the reload reg from reload 1 of insn #84, the bug being that the same reload reg is also used for reload 0 of insn #85. This is supposed to work like so: the inheritance code in choose_reload_regs calls free_for_value_p with

[Patch, Fortran] PR54608 - fix SCAN/VERIFY simplification

2012-09-17 Thread Tobias Burnus
gfortran shouldn't simplify SCAN/VERIFY if the BACK= argument is present but not a constant. Build and regtested on x86-64-linux. OK for the trunk? Tobias 2012-09-17 Tobias Burnus bur...@net-b.de PR fortran/54608 * simplify.c (gfc_simplify_scan, gfc_simplify_verify): Fix handling of

PATCH COMMITTED: Fix libbacktrace bootstrap on Darwin

2012-09-17 Thread Ian Lance Taylor
It didn't take long to find a build problem with libbacktrace. I forgot to back and retest the unknown cases after changing the way backtrace state was handled. This patch should fix bootstrap. Tested on x86_64-unknown-linux-gnu, which proves little since the file is not used on that platform.

Re: [Patch, Fortran] PR54608 - fix SCAN/VERIFY simplification

2012-09-17 Thread Steve Kargl
On Mon, Sep 17, 2012 at 07:34:12PM +0200, Tobias Burnus wrote: gfortran shouldn't simplify SCAN/VERIFY if the BACK= argument is present but not a constant. Build and regtested on x86-64-linux. OK for the trunk? OK. -- Steve

Re: [Patch ARM] big-endian support for Neon vext tests

2012-09-17 Thread Richard Earnshaw
On 17/09/12 16:50, Christophe Lyon wrote: On 17 September 2012 17:21, Richard Earnshaw rearn...@arm.com wrote: On 17/09/12 16:13, Christophe Lyon wrote: On 17 September 2012 14:56, Richard Earnshaw rearn...@arm.com wrote: On 05/09/12 23:14, Christophe Lyon wrote: Hello, Although the recent

Re: [PATCH] Prevent cselib substitution of FP, SP, SFP

2012-09-17 Thread Ahmad Sharif
On Fri, Sep 14, 2012 at 9:39 AM, Carrot Wei car...@google.com wrote: Hi Jakub I have run it on 4.6, it passes the following testing: x86-64 bootstrap x86-64 regression test regression test on arm qemu Is it OK for gcc4.6? Ahmad, is it OK for google/gcc-4_6/ and google/gcc-4_6-mobile ?

Re: [PATCH] PR 53528 c++/ C++11 Generalized Attribute support

2012-09-17 Thread Jason Merrill
On 09/17/2012 11:35 AM, Dodji Seketeli wrote: And I wonder if we want to offer this as an optional warning for GNU attribute syntax. What option would be used to control this optional feature? Would you accept this a separate patch? Let's not worry about this for now. +

Re: [Patch, fortran] PR46897 - [OOP] type-bound defined ASSIGNMENT(=) not used for derived type component in intrinsic assign

2012-09-17 Thread Mikael Morin
Hello, On 10/09/2012 20:58, Paul Richard Thomas wrote: Dear All, Please find attached a new attempt at the patch for PR46897. It now uses temporaries to overcome the side effects that Mikael pointed out. And here comes the next round of comments. Index: gcc/fortran/resolve.c

Re: [Patch ARM] big-endian support for Neon vext tests

2012-09-17 Thread Christophe Lyon
On 17 September 2012 20:04, Richard Earnshaw rearn...@arm.com wrote: On 17/09/12 16:50, Christophe Lyon wrote: On 17 September 2012 17:21, Richard Earnshaw rearn...@arm.com wrote: On 17/09/12 16:13, Christophe Lyon wrote: On 17 September 2012 14:56, Richard Earnshaw rearn...@arm.com wrote: On

Re: [DOC] Update -feliminate-unused-debug-types description

2012-09-17 Thread Paul_Koning
On Sep 17, 2012, at 1:20 PM, Ian Lance Taylor wrote: On Mon, Sep 17, 2012 at 8:59 AM, paul_kon...@dell.com wrote: 2012-09-17 Paul Koning n...@arrl.net * doc/invoke.text (-feliminate-unused-debug-types): Update to reflect that this is enabled by default. This is OK.

Re: [PATCHv3] rs6000: Add 2 built-ins to read the Time Base Register on PowerPC

2012-09-17 Thread David Edelsohn
On Mon, Sep 17, 2012 at 8:53 AM, Tulio Magno Quites Machado Filho tul...@linux.vnet.ibm.com wrote: Add __builtin_ppc_get_timebase and __builtin_ppc_mftb to read the Time Base Register on PowerPC. They are required by applications that measure time at high frequencies with high precision that

Re: [patch, mips] Patch for new mips triplet - mips-mti-elf

2012-09-17 Thread Richard Sandiford
Steve Ellcey sell...@mips.com writes: On Sun, 2012-09-16 at 09:22 +0100, Richard Sandiford wrote: +/* This is idential to sde.h except for the ABI setting which defaults + to 32 instead of n32 on 32 bit architectures and the addition of + MIPS_ISA_SYNCI_SPEC. */ SDE defaults to o32

Re: Backtrace library [1/3]

2012-09-17 Thread Gerald Pfeifer
On Mon, 17 Sep 2012, Ian Lance Taylor wrote: Please let me know about any build problems. i386-unknown-freebsd10.0 now fails as follows. Others presumably have not run into this since This source file is compiled if the unwind library is not available.

Re: Backtrace library [1/3]

2012-09-17 Thread Ian Lance Taylor
On Mon, Sep 17, 2012 at 2:00 PM, Gerald Pfeifer ger...@pfeifer.com wrote: On Mon, 17 Sep 2012, Ian Lance Taylor wrote: Please let me know about any build problems. i386-unknown-freebsd10.0 now fails as follows. Others presumably have not run into this since This source file is compiled if

Re: Backtrace library [1/3]

2012-09-17 Thread Ian Lance Taylor
On Mon, Sep 17, 2012 at 2:00 PM, Gerald Pfeifer ger...@pfeifer.com wrote: On Mon, 17 Sep 2012, Ian Lance Taylor wrote: Please let me know about any build problems. i386-unknown-freebsd10.0 now fails as follows. Others presumably have not run into this since This source file is compiled if

[patch] pr/54508: fix incomplete debug information for class

2012-09-17 Thread Paul_Koning
If the only reference in a source file is to a static method of a class, then GCC would output debug information for the class name but not any of its members or base classes. The attached patch fixes this by having prune_unused_types_mark mark all of the parent's children if the parent DIE

Re: [patch, mips] Patch for new mips triplet - mips-mti-elf

2012-09-17 Thread Steve Ellcey
On Mon, 2012-09-17 at 21:36 +0100, Richard Sandiford wrote: It's a hosted vs. embedded thing. Hosted targets like *-linux-gnu have dynamic ABI requirements and so are keyed off an ABI rather than an architecture. The only effect of -march= and --with-arch= should be to extend the choice of

C++ PATCH for c++/54575 (ICE with alias template)

2012-09-17 Thread Jason Merrill
The change to make SCOPE_REFs value-dependent revealed a bug in our handling of alias-template instantiations; we weren't pushing into the context of the alias before substituting the type, and then we were remembering the previously instantiated one when instantiating in a different context.

Re: C++ PATCH for c++/54575 (ICE with alias template)

2012-09-17 Thread Gabriel Dos Reis
On Mon, Sep 17, 2012 at 10:39 PM, Jason Merrill ja...@redhat.com wrote: The change to make SCOPE_REFs value-dependent revealed a bug in our handling of alias-template instantiations; we weren't pushing into the context of the alias before substituting the type, and then we were remembering the

<    1   2