Re: [PATCH v2] SPARC: Clarify -mapp-regs option

> I don't have commit access, so can someone please do this for me. Applied on all active branches. -- Eric Botcazou

[PATCH] Align gimple_statement_base.subcode

I just noticed we have a spare flag bit in gimple_statement_base which causes subcode to start at bit 15 with consequences for code-generation on the very frequent accesses to this field. Oops. This is a regression from 4.6 which has /* Padding to get subcode to 16 bit alignment. */ unsign

Re: PR 60604: CANNOT_CHANGE_MODE_CLASS being ignored

On Mon, Mar 31, 2014 at 6:42 PM, Richard Sandiford wrote: > PR 60604 shows a case where CANNOT_CHANGE_MODE_CLASS is being ignored > for a subreg of a floating-point register, causing it to be replaced > with an invalid (reg ...). > > There are various reasons why MIPS floating-point registers can'

Re: [PATCH PR60363]XFAIL case ssa-dom-thread-4.c for logical_op_short_circuit targets

On Tue, Apr 1, 2014 at 7:42 AM, bin.cheng wrote: > Hi, > Patch for PR60363 is sent at > http://gcc.gnu.org/ml/gcc-patches/2014-03/msg00857.html , but it needs to > wait for stage 1. In the meanwhile this patch is to xfail the case for > logical_op_short_circuit targets. Is it ok? Ok if you veri

RE: [PATCH PR60363]XFAIL case ssa-dom-thread-4.c for logical_op_short_circuit targets

> -Original Message- > From: Richard Biener [mailto:richard.guent...@gmail.com] > Sent: Tuesday, April 01, 2014 5:14 PM > To: Bin Cheng > Cc: GCC Patches > Subject: Re: [PATCH PR60363]XFAIL case ssa-dom-thread-4.c for > logical_op_short_circuit targets > > On Tue, Apr 1, 2014 at 7:42 AM,

[patch] Fix PR bootstrap/60620

Hi, the gnattools are now linked with the C++ compiler so, for the native case, it has a dependency on libstdc++-v3. Adding such a dependency for a host tool on a target library is debatable, but there is already a precedent with libada. Tested on x86_64-suse-linux, OK for the mainline? 2014

Re: [PATCH] Fix PR60505

On Fri, 28 Mar 2014, Cong Hou wrote: > Ping? Ok. Thanks, Richard. > > thanks, > Cong > > > On Wed, Mar 19, 2014 at 11:39 AM, Cong Hou wrote: > > On Tue, Mar 18, 2014 at 4:43 AM, Richard Biener wrote: > >> > >> On Mon, 17 Mar 2014, Cong Hou wrote: > >> > >> > On Mon, Mar 17, 2014 at 6:44 AM

RE: [patch] Shorten Windows path

Ian, thanks for your comments. Please find answers and new version below: > -Original Message- > From: Ian Lance Taylor [mailto:i...@google.com] > Sent: 25 March 2014 21:09 > To: Joey Ye > Cc: gcc-patches > Subject: Re: [patch] Shorten Windows path > > On Tue, Mar 25, 2014 at 1:58 AM, Joe

[committed, libjava] XFAIL sourcelocation (PR libgcj/55637) backported to 4.8.3

r...@cebitec.uni-bielefeld.de (Rainer Orth) wrote: > Sure, patch preapproved. Commited as r208983: 2014-04-01 Dominique d'Humieres Rainer Orth PR libgcj/55637 * testsuite/libjava.lang/sourcelocation.xfail: New file. Index: libjava/testsuite/libjava.la

Re: Lost __mips_o32 predefine on NetBSD

Martin Husemann writes: > On Mon, Mar 31, 2014 at 08:35:30PM +0100, Richard Sandiford wrote: >> Are you sure it was ever in FSF GCC? I went through every revision >> of netbsd.h in git and couldn't see it. > > No, I'm not - guess I never tried a pure gcc on mips. > >> I can go ahead and apply the

Re: [PATCH][LTO] Fix PR60720

On Mon, 31 Mar 2014, Richard Biener wrote: > > The following patch makes sure we wrap symbol references in global > initializers the same as we do for IL references - inside a MEM_REF > that guards their context against type changes due to symbol merging > (and thus in this case against ICEs when

Re: Please revert the patches in bug #54040 and #59346 and special case x32

On Wed, 2014-03-26 at 00:25 +0100, Eric Botcazou wrote: > > Should I file a bug to get feedback on this issue? I know Ada is not the > > most prioritized language for gcc, but anyway. The current > > implementation is not POSIX-compliant. > > Can you post a complete patch doing the reversion? Bre

Re: std::quoted doesn't respect padding

[CCing gcc-patches] On 11/03/14 11:18 -0400, Ed Smith-Rowland wrote: On 02/14/2014 07:56 PM, Jonathan Wakely wrote: We need to implement this fix (probably after 4.9 is released though) http://cplusplus.github.io/LWG/lwg-active.html#2344 Here is a patch (Stage 1 obviously). A couple of thi

[PATCH, nios2] Misc. fixes

This contains a few small changes/fixes, committed to trunk. (1) a typo in nios2_function_profiler. (2) unneeded parameter in nios2_large_got_address(). (3) remove two no longer needed unspec enums. (4) Provide basic implementation of the delegitimize address hook, to silent a unrecognized UNSPECs

Re: [PATCH, PR 60640] When creating virtual clones, clone thunks too

Hi, On Fri, Mar 28, 2014 at 09:43:53PM +0100, Jan Hubicka wrote: > > Hi, > > > > this patch fixes PR 60640 by creating thunks to clones when that is > > necessary to properly redirect edges to them. I mostly does what > > cgraph_add_thunk does and what analyze_function does to thunks. It > > fi

[PATCH][LTO] Support -fno-use-linker-plugin with auto-loaded plugin

BFD supports plugin auto-loading for plugins placed in /usr/lib/bfd-plugins (if you make it work for GNU LTO objects by fixing PR14698). If it does so then we claim LTO IR objects via the plugin machinery even if -fno-use-linker-plugin is specified (in that case the gcc driver won't pass a -plug

[PATCH][LTO] Rework -flto-partition=, add =one case

This reworks the option to use the Enum support we have now and adds a =one case (to eventually get rid of one LTO operation mode, =none ...). I was tempted to support -flto-partition= and get rid of --param lto-partitions (thereby also supporting =1), but that param specifies the maximum number

PING [Patch] Silence test failures on darwin9

Patch submitted at http://gcc.gnu.org/ml/gcc-patches/2014-03/msg01477.html Results for powerpc-apple-darwin9 with the patches at http://gcc.gnu.org/ml/gcc-testresults/2014-03/msg02361.html and without them at http://gcc.gnu.org/ml/gcc-testresults/2014-02/msg01256.html OK for trunk? OK to bac

[PATCH] Fix PR60729, expand_abs and -ftrapv

It looks like expand_abs and clients use absv_optab even for FP modes which IMHO is wrong. The following restricts it to MODE_INT (not sure what modes the v variants should be present, but existing checks use MODE_INT and libgcc only provides SImode and DImode fallbacks). I've took the liberty t

[COMMITTED] Fix target/60704

Yet more fallout from my float->int changes. This time the failure is during sched1, where we try to constrain_operands to figure out what the sched parameters are for the insn. Except that since this is before register allocation, where we are going to force the operand to memory, there were no

Re: [COMMITTED] Fix target/60704

On Tue, Apr 1, 2014 at 7:48 AM, Richard Henderson wrote: > Yet more fallout from my float->int changes. > > This time the failure is during sched1, where we try to constrain_operands to > figure out what the sched parameters are for the insn. Except that since this > is before register allocation

Re: [PATCH][LTO] Rework -flto-partition=, add =one case

> > This reworks the option to use the Enum support we have now and > adds a =one case (to eventually get rid of one LTO operation mode, > =none ...). I was tempted to support -flto-partition= > and get rid of --param lto-partitions (thereby also supporting =1), Yep, I preffer to have one switch

Re: [PATCH][LTO] Support -fno-use-linker-plugin with auto-loaded plugin

> > BFD supports plugin auto-loading for plugins placed in > /usr/lib/bfd-plugins (if you make it work for GNU LTO objects by > fixing PR14698). If it does so then we claim LTO IR objects > via the plugin machinery even if -fno-use-linker-plugin is > specified (in that case the gcc driver won't

Re: [COMMITTED] Fix target/60704

On 04/01/2014 08:22 AM, H.J. Lu wrote: > On Tue, Apr 1, 2014 at 7:48 AM, Richard Henderson wrote: >> Yet more fallout from my float->int changes. >> >> This time the failure is during sched1, where we try to constrain_operands to >> figure out what the sched parameters are for the insn. Except th

Fwd: [PATCH] pedantic warning behavior when casting void* to ptr-to-func, 4.8 and 4.9

I just realized I posted the patch in the wrong list. -- Forwarded message -- From: Daniel Gutson Date: Tue, Apr 1, 2014 at 10:43 AM Subject: [PATCH] pedantic warning behavior when casting void* to ptr-to-func, 4.8 and 4.9 To: gcc Mailing List Hi, I observed two different

Re: [Fortran][PATCH][gomp4]: Transform OpenACC loop directive

Hi Tobias! Thanks a lot for your review! On 26.03.2014 03:16, Tobias Burnus wrote: * And do concurrent also supports masks: This is doable: generate mask conditions inside of the deepest for loop (see applied patch). That will work in the most common cases but not in general. At least

Re: C++ PATCH for c++/60642 (abi_tag and templates)

On 03/31/2014 02:49 PM, Jason Merrill wrote: This bug report pointed out that we weren't properly decorating template mangled names with the appropriate abi_tags. This patch fixes this by making sure that the mangler only looks at the abi_tag from the template, and the parser warns that we'll ig

[PATCH] powerpc, document vec_vgbbd

It was pointed out to me that I neglated to document the powerpc ISA 2.07 builtin function vec_vbggd. After making sure the documentation built, I checked in the following patch as being obvious: 2014-04-01 Michael Meissner * doc/extend.texi (PowerPC AltiVec/VSX Built-in Functions):

C++ PATCH for c++/60713 (ICE with PMF in init-list)

The gimplifier was getting confused because we didn't set TREE_SIDE_EFFECTS on the CONSTRUCTOR. Tested x86_64-pc-linux-gnu, applying to trunk. commit 00cbdd8b8a83dd7b8e386fc713bb2b1dab173d81 Author: Jason Merrill Date: Tue Apr 1 14:06:46 2014 -0400 PR c++/60713 * typeck2.c (PICFLAG

C++ PATCH for c++/60708 (ICE with array temporary)

We need to make sure that the type we're converting to has its TYPE_SIZE set appropriately. Tested x86_64-pc-linux-gnu, applying to trunk. commit 4d056b7b21032f78270f6ae398c15508b5fae8b2 Author: Jason Merrill Date: Tue Apr 1 14:31:35 2014 -0400 PR c++/60708 * call.c (build_array_co

Re: [PATCH] Fix PR c++/44859

Applied, thanks. Jason

C++ PATCH for c++/60374 (ICE with missing typename and variadics)

For a normal template argument, we notice missing typename and adjust the argument appropriately, but we weren't doing that for a pack expansion, leading to confusion later on when we tried to treat a SCOPE_REF as a type. But since the standard requires that any variadic template must be well-

[patch] Fix broken links in libstdc++ manual

Fix some broken links in the libstdc++ docs. Committed to trunk commit 201c0a880621618055a572fb1dbb4a7cc39896f4 Author: Jonathan Wakely Date: Tue Apr 1 20:45:54 2014 +0100 * doc/xml/manual/backwards_compatibility.xml (backwards.third.headers): Update link. * doc/xml/

[PATCH] aarch64 suuport for libitm

Comments? If approved, should this go in for 4.9, or wait for stage1? Certainly it's self-contained... r~ * configure.tgt: Handle aarch64 as target. * config/aarch64/sjlj.S: New file. * config/aarch64/target.h: New file. diff --git a/libitm/config/aarch64/sjlj.S b/libit

Fix various x86 tests for --with-arch=bdver3 --with-cpu=bdver3

When I fixed various tests in for failures with --with-arch=bdver3, I missed that a so-configured compiler still defaults to -mtune=generic. If you override that as well with --with-cpu=bdver3, further failures appear, and this patch fixes

Re: [PATCH] aarch64 suuport for libitm

On Tue, Apr 1, 2014 at 3:24 PM, Richard Henderson wrote: > Comments? If approved, should this go in for 4.9, or wait for stage1? > Certainly it's self-contained... On Cavium's thunder processor the cache line size is going to be bigger than 64 bytes, what is your solution to improve performance

[PATCH] PowerPC, PR60735: _Decimal64 moves broken on -mspe

In backporting the power8 changes to the 4.8 branch, one of the testers of these patches noticed that libgcc cannot be built on a linux SPE target. The reason was the _Decimal64 type did not have a proper move insn in the SPE environment. This patch fixes that issue. In looking at the patch, I d

Re: [PATCH] Guard special installs in install-driver

On Mar 31, 2014, at 4:50 AM, Richard Biener wrote: > -$(INSTALL_PROGRAM) xgcc$(exeext) > $(DESTDIR)$(bindir)/$(GCC_INSTALL_NAME)$(exeext) > ! -rm -f > $(DESTDIR)$(bindir)/$(target_noncanonical)-gcc-$(version)$(exeext) > ! -( cd $(DESTDIR)$(bindir) && \ > !$(LN) $(GCC_INSTAL

Re: [Patch] Silence test failures on darwin9

On Mar 26, 2014, at 3:33 PM, Dominique Dhumieres wrote: > OK for trunk? Ok.

Re: PING [Patch] Silence test failures on darwin9

On Apr 1, 2014, at 6:38 AM, Dominique Dhumieres wrote: > Patch submitted at > > http://gcc.gnu.org/ml/gcc-patches/2014-03/msg01477.html > OK to backport the suitable patches for pr54083 and > pr54407 to 4.8 and 4.7 branches? Sure, as long as you let it bake a little on trunk. :-).

[PATCH][1/3] Fix PR54733 Optimize endian independent load/store

I took the lack of answer for this patch as an indication that the patch is too big. This is the first patch in a series of three. Its purpose is to create some new effective target for architecture having byte swap instructions and make use of them in the existing byte swap tests. One effective t

[PATCH][2/3] Fix PR54733 Optimize endian independent load/store

Here is the second patch in the series. Its purpose is to extend the current bswap optimization to handle bitwise operations on memory sources (array and structures). This patch extends the concept of symbolic number used in the bswap pass to memorize some extra informations for bitwise operations

[PATCH][3/3] Fix PR54733 Optimize endian independent load/store

Here is the third and last patch in the series. Its purpose is to add support for bitwise operation from a source operand that are equivalent to a load in the target's endianness. The ChangeLog are as follows: *** gcc/ChangeLog *** 2014-04-01 Thomas Preud'homme PR tree-optimization/5

Re: [PATCH] PowerPC, PR60735: _Decimal64 moves broken on -mspe

On Tue, Apr 1, 2014 at 7:55 PM, Michael Meissner wrote: > In backporting the power8 changes to the 4.8 branch, one of the testers of > these patches noticed that libgcc cannot be built on a linux SPE target. The > reason was the _Decimal64 type did not have a proper move insn in the SPE > environ

Re: [PATCH][2/3] Fix PR54733 Optimize endian independent load/store

On Wed, 2 Apr 2014, Thomas Preud'homme wrote: Note that as it stands the patch does not work for arrays indexed with variable (such a tab[a] || (tab[a+1] << 8)) because fold_const does not fold (a + 1) - a. Uh? It does fold a+1-a for me. What it doesn't do is look through the definition of b

Re: Fix various x86 tests for --with-arch=bdver3 --with-cpu=bdver3

On Wed, Apr 2, 2014 at 12:27 AM, Joseph S. Myers wrote: > When I fixed various tests in > for failures > with --with-arch=bdver3, I missed that a so-configured compiler still > defaults to -mtune=generic. If you override that as well with