Re: [Patch, i386] Avoid LCP stalls (issue5975045)

2012-04-05 Thread Uros Bizjak
Hello! > New patch to avoid LCP stalls based on feedback from earlier patch. I modified > H.J.'s old patch to perform the peephole2 to split immediate moves to HImode > memory. This is now enabled for Core2, Corei7 and Generic. > 2012-04-04 Teresa Johnson > > * config/i386/i386.h (ix86_

[PATCH] RX: Fix min instruction for unsigned variables

2012-04-05 Thread Naveen H. S
Hi, Please find attached the patch "smin.patch" which fixes the issue for unsigned values with "min" instruction. Currently, unsigned values greater than QI mode are converted into signed values and hence generates wrong value. The patch fixes the issue. Tested with rx-elf. No new regressions.

Re: [lra] spilling general class pseudos into SSE regs instead of memory (a target hooks driven implementation)

2012-04-05 Thread Ye Joey
On Mon, Apr 2, 2012 at 10:11 PM, Vladimir Makarov wrote: > On 03/31/2012 01:38 AM, Ye Joey wrote: > > Yes, if you mean spilling in LRA.  But IRA can assign union of general and > SSE classes to a pseudo if it is profitable. Any chance for IRA not to do so, saying by introducing an option? - Joey

Re: [PATCH 2/2] arm: add iwMMXt mmx-2.c test

2012-04-05 Thread Ramana Radhakrishnan
On 4 April 2012 19:35, Matt Turner wrote: > >  gcc/testsuite/gcc.target/arm/mmx-2.c |  158 > ++ >  1 files changed, 158 insertions(+), 0 deletions(-) >  create mode 100644 gcc/testsuite/gcc.target/arm/mmx-2.c > > diff --git a/gcc/testsuite/gcc.target/arm/mmx-2.c

Re: [patch, fortran] Fix PR 52668, bogus warning for only list

2012-04-05 Thread Tobias Burnus
Thomas Koenig wrote: > this fixes a 4.7/4.8 regression with a bogus warning for variables > which were excluded from being imported by "use, only" if they > were in a common block inside the module. > > Regression-tested. OK for trunk and for 4.7 after a few days? OK and thanks for the patch. >

Re: [C11-atomic] [patch] gimple atomic statements

2012-04-05 Thread Richard Guenther
On Wed, Apr 4, 2012 at 5:50 PM, Andrew MacLeod wrote: > On 04/04/2012 10:33 AM, Richard Guenther wrote: >> >> On Wed, Apr 4, 2012 at 3:28 PM, Andrew MacLeod >>  wrote: >> This is a WIP... that fntype fields is there for simplicity..   and no... >> you can do a 1 byte atomic operation on a full wor

[v3] Export std::_List_node_base for non-PIC code

2012-04-05 Thread Rainer Orth
As discussed in PR libstdc++/52689, no longer exporting std::_List_node_base for non-PIC code breaks linking go1 on Solaris with static libstdc++: Undefined first referenced symbol in file _ZNSt15_List_node_base4swapERS_S0_ /vol/gcc/lib/libppl_c

Re: [PATCH] Fix PR18589

2012-04-05 Thread Richard Guenther
On Wed, Apr 4, 2012 at 9:15 PM, William J. Schmidt wrote: > On Wed, 2012-04-04 at 15:08 +0200, Richard Guenther wrote: >> On Wed, Apr 4, 2012 at 2:35 PM, William J. Schmidt >> wrote: >> > On Wed, 2012-04-04 at 13:35 +0200, Richard Guenther wrote: >> >> On Tue, Apr 3, 2012 at 10:25 PM, William J.

Re: [PATCH] Fix PR52614

2012-04-05 Thread Richard Guenther
On Thu, Apr 5, 2012 at 6:22 AM, Mike Stump wrote: > On Apr 4, 2012, at 7:56 PM, William J. Schmidt wrote: >> There seems to be tacit agreement that the vector tests should use >> -fno-common on all targets to avoid the recent spate of failures (see >> discussion in 52571 and 52603). > >> OK for tr

[PATCH][1/3] Release SSA names properly

2012-04-05 Thread Richard Guenther
This releases SSA names properly in some places. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2012-02-10 Richard Guenther * tree-nrv.c (tree_nrv): Release VDEFs. * tree-sra.c (sra_modify_constructor_assign): Likewise. (sra_modify_as

[PATCH] Compact SSA version namespace when releasing the freelist

2012-04-05 Thread Richard Guenther
Currently we release the memory for released SSA names (which we keep around on a freelist, mainly to try to keep the SSA version namespace dense) after early optimizations. This has the negative impact (as documented) of permanently establishing holes in the SSA version namespace. The following

Re: [RS6000] Stack tie fix.

2012-04-05 Thread Olivier Hainque
Hello Alan, On Apr 4, 2012, at 03:22 , Alan Modra wrote: > I'll see whether my approach fixes pr30282 for gcc-4.4 which has known > deficiencies in alias analysis. Olivier, would you be kind enough to > backport and test against other versions of gcc that needed your > bigger hammer? Sure. I ca

Re: [PATCH] Make gsi_remove return whether EH cleanup is required

2012-04-05 Thread Rainer Orth
Richard Guenther writes: > Several passes needlessly cleanup EH after gsi_remove because they do > not know whether the stmt was removed from EH regions. The following > patch returns this information from gsi_remove and adjusts all users > I could find appropriately. I suspect this patch cause

Re: [PATCH] Make gsi_remove return whether EH cleanup is required

2012-04-05 Thread Richard Guenther
On Thu, Apr 5, 2012 at 1:20 PM, Rainer Orth wrote: > Richard Guenther writes: > >> Several passes needlessly cleanup EH after gsi_remove because they do >> not know whether the stmt was removed from EH regions.  The following >> patch returns this information from gsi_remove and adjusts all user

Re: [PATCH] Make gsi_remove return whether EH cleanup is required

2012-04-05 Thread Rainer Orth
Richard Guenther writes: > On Thu, Apr 5, 2012 at 1:20 PM, Rainer Orth > wrote: >> Richard Guenther writes: >> >>> Several passes needlessly cleanup EH after gsi_remove because they do >>> not know whether the stmt was removed from EH regions.  The following >>> patch returns this information

Re: [C11-atomic] [patch] gimple atomic statements

2012-04-05 Thread Andrew MacLeod
On 04/05/2012 05:14 AM, Richard Guenther wrote: Ok. I suppose having a GIMPLE_ATOMIC is fine then. Thanks for all the feedback, I haven't really had to play in gimple land much lately. Some of those routines I guessed at what to do in order to get through initial compilation and your comment

Re: [PATCH] Fix PR52614

2012-04-05 Thread William J. Schmidt
On Thu, 2012-04-05 at 11:30 +0200, Richard Guenther wrote: > On Thu, Apr 5, 2012 at 6:22 AM, Mike Stump wrote: > > On Apr 4, 2012, at 7:56 PM, William J. Schmidt wrote: > >> There seems to be tacit agreement that the vector tests should use > >> -fno-common on all targets to avoid the recent spate

Re: [branches/google/gcc-4_6] Backported r179661 and 179662 from mainline. (issue 5989043)

2012-04-05 Thread Andrew Stubbs
On 05/04/12 05:43, Michael Hope wrote: Andrew, could you check that the Google guys have the final version of your patch? Yes, it looks fine to me. Could you backport the fix to the 4.6 release branch if valid? Better than having the same patch in three places. Ok, I'll need permission fro

Re: [PATCH] Fix PR52614

2012-04-05 Thread Richard Guenther
On Thu, Apr 5, 2012 at 1:59 PM, William J. Schmidt wrote: > On Thu, 2012-04-05 at 11:30 +0200, Richard Guenther wrote: >> On Thu, Apr 5, 2012 at 6:22 AM, Mike Stump wrote: >> > On Apr 4, 2012, at 7:56 PM, William J. Schmidt wrote: >> >> There seems to be tacit agreement that the vector tests shou

Re: [PATCH] Make gsi_remove return whether EH cleanup is required

2012-04-05 Thread Richard Guenther
On Thu, 5 Apr 2012, Rainer Orth wrote: > Richard Guenther writes: > > > On Thu, Apr 5, 2012 at 1:20 PM, Rainer Orth > > wrote: > >> Richard Guenther writes: > >> > >>> Several passes needlessly cleanup EH after gsi_remove because they do > >>> not know whether the stmt was removed from EH reg

fix install-no-fixedincludes mishaps

2012-04-05 Thread Olivier Hainque
Hello, This is following up on a proposal first discussed at http://gcc.gnu.org/ml/gcc-patches/2010-10/msg00666.html The initial issue to resolve is still present today: install-no-fixedincludes remains a useful Makefile target and it misbehaves in two ways: 1) past the second invocation, it

Re: fix install-no-fixedincludes mishaps

2012-04-05 Thread Richard Guenther
On Thu, Apr 5, 2012 at 2:54 PM, Olivier Hainque wrote: > Hello, > > This is following up on a proposal first discussed at > http://gcc.gnu.org/ml/gcc-patches/2010-10/msg00666.html > > The initial issue to resolve is still present today: > install-no-fixedincludes remains a useful Makefile target >

Re: fix install-no-fixedincludes mishaps

2012-04-05 Thread Olivier Hainque
On Apr 5, 2012, at 15:07 , Richard Guenther wrote: > Nice. Thanks :) > I suppose this would enable a --disable-fixinclude? fixinclude can > take quite some time when a lot of headers are installed and if we are > not going to install them, why produce them ... That was another suggestion in

Re: PATCH: Define TRY_EMPTY_VM_SPACE for Linux/x32

2012-04-05 Thread H.J. Lu
On Wed, Apr 4, 2012 at 11:56 AM, H.J. Lu wrote: > On Wed, Apr 4, 2012 at 11:53 AM, Andrew Pinski wrote: >> On Wed, Apr 4, 2012 at 11:50 AM, Uros Bizjak wrote: >>> Looking at how other targets implement this check, I don't think that >>> this is a problem at all. This issue only shows on a non-bo

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Konstantinos Margaritis
On Wed, 4 Apr 2012 07:09:46 -0500 Dennis Gilmore wrote: > Fedora does use /lib64 on x86_64 I would personally prefer /libhfp but > wouldn't object to /libhf though today we have f17 about to go beta > and all of rawhide built using /lib Hi Dennis, One potential problem that is born from the

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Dennis Gilmore
El Wed, 4 Apr 2012 08:54:12 +0200 Jakub Jelinek escribió: > On Wed, Apr 04, 2012 at 01:34:30PM +1200, Michael Hope wrote: > > >  I did two ports of Mandriva to armv7. One of my choice to use > > > softfp, and another hardfp port to be compatible with other > > > distros. But other than a previous

[PATCH] Move predicates that only make sense during gimplification

2012-04-05 Thread Richard Guenther
This continues what I've started some time ago - move predicates that do not deeply inspect trees to gimplify.c and make them private. Bootstrapped on x86_64-unknown-linux-gnu, applied. Richard. 2012-04-05 Richard Guenther * gimple.c (walk_gimple_op): Compute val_only for the LHS

Re: [PATCH] Fix PR18589

2012-04-05 Thread William J. Schmidt
On Thu, 2012-04-05 at 11:23 +0200, Richard Guenther wrote: > On Wed, Apr 4, 2012 at 9:15 PM, William J. Schmidt > wrote: > > > > Unfortunately this seems to be necessary if I name the two passes > > "reassoc1" and "reassoc2". If I try to name both of them "reassoc" I > > get failures in other te

Re: [Patch, i386] Avoid LCP stalls (issue5975045)

2012-04-05 Thread Teresa Johnson
Thanks, I will do both and update the comment as suggested by David, retest and then commit. Teresa On Thu, Apr 5, 2012 at 12:41 AM, Uros Bizjak wrote: > Hello! > >> New patch to avoid LCP stalls based on feedback from earlier patch. I >> modified >> H.J.'s old patch to perform the peephole2 to

[PATCH][1/n] Cleanup global pass execution flow

2012-04-05 Thread Richard Guenther
I'm trying to get an idea where and why we execute what part of the pass pipeline (yeah, a two-line patch doing -Og should be possible after this series ...). This patch removes an odd caller of execute_pass_list (pass_early_local_passes.pass.sub), named tree_lowering_passes, which is even undocu

Re: [RS6000] Stack tie fix.

2012-04-05 Thread Alan Modra
Hi Olivier, On Thu, Apr 05, 2012 at 12:36:01PM +0200, Olivier Hainque wrote: > On Apr 4, 2012, at 03:22 , Alan Modra wrote: > > I'll see whether my approach fixes pr30282 for gcc-4.4 which has known > > deficiencies in alias analysis. Olivier, would you be kind enough to > > backport and test aga

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Niels de Vos
On 04/05/2012 03:30 PM, Konstantinos Margaritis wrote: On Wed, 4 Apr 2012 07:09:46 -0500 Dennis Gilmore wrote: Fedora does use /lib64 on x86_64 I would personally prefer /libhfp but wouldn't object to /libhf though today we have f17 about to go beta and all of rawhide built using /lib Hi Den

[PATCH] Remove definition of LANG_HOOKS_CALLGRAPH_EXPAND_FUNCTION

2012-04-05 Thread Richard Guenther
It's gone since 2007. Committed as obvious. Richard. 2012-04-05 Richard Guenther * lto-lang.c (LANG_HOOKS_CALLGRAPH_EXPAND_FUNCTION): Remove definition. Index: lto-lang.c === --- lto-lang.c (revision 186165) +

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Steve McIntyre
On Wed, Apr 04, 2012 at 01:11:33AM +0200, Jakub Jelinek wrote: >On Wed, Apr 04, 2012 at 09:18:59AM +1200, Michael Hope wrote: >> >> The subdirectories could be called fred and jim and it would still work. >> >>  The only thing required is that this part of the naming scheme be >> >> agreed amongst

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Steve McIntyre
On Wed, Apr 04, 2012 at 01:34:30PM +1200, Michael Hope wrote: >2012/4/4 Paulo César Pereira de Andrade >> >>  I did two ports of Mandriva to armv7. One of my choice to use softfp, >> and another hardfp port to be compatible with other distros. But other >> than a previous armv5 port, there is not m

Re: [PATCH][ARM] NEON DImode not

2012-04-05 Thread Andrew Stubbs
On 28/03/12 14:37, Ramana Radhakrishnan wrote: OK if no regressions. There was a small problem with VMVN being not actually predicable. I've made the obvious change and committed the attached. Andrew 2012-04-05 Andrew Stubbs gcc/ * config/arm/arm.md (arch): Add neon_onlya8 and neon_not

Re: [PATCH][1/n] Cleanup global pass execution flow

2012-04-05 Thread Jan Hubicka
> > I'm trying to get an idea where and why we execute what part of the > pass pipeline (yeah, a two-line patch doing -Og should be possible > after this series ...). This patch removes an odd caller of > execute_pass_list (pass_early_local_passes.pass.sub), named > tree_lowering_passes, which is

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Steve McIntyre
On Tue, Apr 03, 2012 at 10:56:18PM +, Joseph S. Myers wrote: > >(c) Please include libc-ports on future submissions and provide both the >GCC patch and the glibc ports patch that have been tested to work together >to build and install the library in the given path; a patch to one >component

[Libiberty - V2]: Handle VMS as a LLP64 platform in splay-tree.h

2012-04-05 Thread Tristan Gingold
On Apr 4, 2012, at 5:07 PM, Ian Lance Taylor wrote: > Tristan Gingold writes: > >> Would something like that be acceptable ? >> I have just checked that I can still build gcc with that patch. If you like >> this approach I will properly submit a patch. > > Thanks. > > You should also test t

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Mike Frysinger
On Thursday 05 April 2012 09:30:23 Konstantinos Margaritis wrote: > On Wed, 4 Apr 2012 07:09:46 -0500 Dennis Gilmore wrote: > > Fedora does use /lib64 on x86_64 I would personally prefer /libhfp but > > wouldn't object to /libhf though today we have f17 about to go beta > > and all of rawhide buil

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Mike Frysinger
On Thursday 05 April 2012 10:38:07 Steve McIntyre wrote: > On Wed, Apr 04, 2012 at 01:34:30PM +1200, Michael Hope wrote: > >2012/4/4 Paulo César Pereira de Andrade > > > >> I did two ports of Mandriva to armv7. One of my choice to use softfp, > >> and another hardfp port to be compatible with othe

Re: [PATCH][1/n] Cleanup global pass execution flow

2012-04-05 Thread Diego Novillo
On 4/5/12 9:54 AM, Richard Guenther wrote: Honza, does this look ok? I am going to continue this way until I can re-architect the toplevel pass structure in a way that we hand control more to the pass manager. Well, hopefully ;) Related to this, have you thought about the handoff points betw

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Konstantinos Margaritis
On Thu, 5 Apr 2012 11:08:56 -0400 Mike Frysinger wrote: > i don't think that's true. on an x86_64 system, the 64bit libs are in > /lib64/. some distros tried to (pointlessly imo) resist and force 64bits > into > /lib/ when the native ABI was x86_64 (Gentoo included), but those are legacy > i

Re: [PATCH] Compact SSA version namespace when releasing the freelist

2012-04-05 Thread Diego Novillo
On 4/5/12 6:13 AM, Richard Guenther wrote: Currently we release the memory for released SSA names (which we keep around on a freelist, mainly to try to keep the SSA version namespace dense) after early optimizations. This has the negative impact (as documented) of permanently establishing holes

latch mem to reg before multi-access in convert_move

2012-04-05 Thread Olivier Hainque
Hello, At some point in convert_move we have: <

Re: [PATCH][1/n] Cleanup global pass execution flow

2012-04-05 Thread Jan Hubicka
> On 4/5/12 9:54 AM, Richard Guenther wrote: > > >Honza, does this look ok? I am going to continue this way > >until I can re-architect the toplevel pass structure in a way > >that we hand control more to the pass manager. Well, hopefully ;) > > Related to this, have you thought about the hando

Re: PATCH: Define TRY_EMPTY_VM_SPACE for Linux/x32

2012-04-05 Thread Uros Bizjak
On Thu, Apr 5, 2012 at 3:28 PM, H.J. Lu wrote: Looking at how other targets implement this check, I don't think that this is a problem at all. This issue only shows on a non-bootstrapped build. A full bootstrap will use correct address. >>> >>> The other place where it shows up is

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Paulo César Pereira de Andrade
Em 5 de abril de 2012 12:09, Mike Frysinger escreveu: > On Thursday 05 April 2012 10:38:07 Steve McIntyre wrote: >> On Wed, Apr 04, 2012 at 01:34:30PM +1200, Michael Hope wrote: >> >2012/4/4 Paulo César Pereira de Andrade >> > >> >>  I did two ports of Mandriva to armv7. One of my choice to use so

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Mike Frysinger
On Thursday 05 April 2012 11:24:15 Konstantinos Margaritis wrote: > On Thu, 5 Apr 2012 11:08:56 -0400 Mike Frysinger wrote: > > i don't think that's true. on an x86_64 system, the 64bit libs are in > > /lib64/. some distros tried to (pointlessly imo) resist and force 64bits > > into /lib/ when th

Re: PATCH: Define TRY_EMPTY_VM_SPACE for Linux/x32

2012-04-05 Thread H.J. Lu
On Thu, Apr 5, 2012 at 8:36 AM, Uros Bizjak wrote: > On Thu, Apr 5, 2012 at 3:28 PM, H.J. Lu wrote: > > Looking at how other targets implement this check, I don't think that > this is a problem at all. This issue only shows on a non-bootstrapped > build. A full bootstrap will use corr

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Steve McIntyre
On Thu, Apr 05, 2012 at 11:32:39AM +1200, Michael Hope wrote: > >So: > * Big endian: undefined, defaults to /lib/ld-linux.so.3 > * Little endian, soft float: /lib/ld-linux.so.3 > * Little endian, hard float: /libhf/ld-linux.so.3 > >> Standard upstream practice supports having multiple variants that

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Steve McIntyre
On Thu, Apr 05, 2012 at 01:16:27PM +1200, Michael Hope wrote: >On 5 April 2012 12:07, Joseph S. Myers wrote: >> >> No.  A system is either purely hard-float or purely soft-float, and the >> same paths are used for both so they can't coexist.  (Mismatches at >> *static* link time are detected throu

[patch optimization]: Add some basic folding for ==/!= comparison

2012-04-05 Thread Kai Tietz
Hello, this patch adds some basic folding capabilities to fold-const for equal and none-equal comparisons on integer typed argument. ChangeLog 2012-04-05 Kai Tietz * fold-const.c (fold_comparison_1): New function. (fold_comparison): Use fold_comparison_1. 2012-04-05

Re: [branches/google/gcc-4_6] Backported r179661 and 179662 from mainline. (issue 5989043)

2012-04-05 Thread Xinliang David Li
ok for google branches David On Wed, Apr 4, 2012 at 8:56 PM, wrote: > Reviewers: Diego Novillo, jingyu, davidxl, > > Message: > Please take a look at this patch and tell me if it's OK for > branches/google/gcc-4_6. > > Description: > Backported the following patch from trunk: > > 2011-10-07  An

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Steve McIntyre
On Thu, Apr 05, 2012 at 11:08:56AM -0400, Mike Frysinger wrote: >On Thursday 05 April 2012 09:30:23 Konstantinos Margaritis wrote: >> On Wed, 4 Apr 2012 07:09:46 -0500 Dennis Gilmore wrote: >> > Fedora does use /lib64 on x86_64 I would personally prefer /libhfp but >> > wouldn't object to /libhf t

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Konstantinos Margaritis
On Thu, 5 Apr 2012 11:55:14 -0400 Mike Frysinger wrote: > note: i don't care about /lib/ld-linux-hf.so.3 or /lib/ld-linux.so.4 or > /libhf/ld-linux.so.[34]. /lib// is really the only one i > don't > think doesn't belong. and I'm just saying that I dislike /libhf, I also think that just raisi

Re: RFA: RL78: Add -fstack-usage support

2012-04-05 Thread DJ Delorie
> OK for mainline/4.7 branch ? Ok with me

Re: [RS6000] Stack tie fix.

2012-04-05 Thread Olivier Hainque
On Apr 5, 2012, at 16:03 , Alan Modra wrote: > Well it turns out that gcc-4.4 still gets this testcase from pr30282 > wrong. Hmph :-( >> There are lots of places in the emit_epilogue code that assign >> frame_reg_rtx with different possible values, (hard_fp, sp, r11) > > r12 too Right ...

[google/gcc-4_6] Added dejagnu base line for x86_64-cros-linux-gnu. (issue 5990044)

2012-04-05 Thread shenhan
Reviewers: asharif1, jingyu, Message: Hi Jing and Ahmad, This adds a new test base line for x86_64-cros-linux-gnu. Please take a look. Thanks, Han Description: The new test base line for x86_64-cros-linux-gnu. Please review this at http://codereview.appspot.com/5990044/ Affected files: A c

Re: [PATCH] Fix PR52614

2012-04-05 Thread Mike Stump
On Apr 5, 2012, at 2:30 AM, Richard Guenther wrote: > Why can the link editor not promote the definitions alignment > when merging with a common with bigger alignment? The problem is that when a common symbol is upped in alignment, but then not chosen by ld (or worse, by the dynamic linker), but

Re: [PATCH] ARM: Use different linker path for hardfloat ABI

2012-04-05 Thread Mike Frysinger
On Thursday 05 April 2012 12:15:41 Steve McIntyre wrote: > On Thu, Apr 05, 2012 at 11:08:56AM -0400, Mike Frysinger wrote: > >On Thursday 05 April 2012 09:30:23 Konstantinos Margaritis wrote: > >> Loic suggested a -IMHO- better solution: to change the dynamic linker > >> filename, not the dir, i.e.

Re: [branches/google/gcc-4_6] Backported r179661 and 179662 from mainline. (issue 5989043)

2012-04-05 Thread asharif
Committed as r186168. http://codereview.appspot.com/5989043/

[branches/google/gcc-4_6_2-mobile] Backported r186168 from branches/google/gcc-4_6. (issue 5991052)

2012-04-05 Thread asharif
Reviewers: jingyu, bjanakiraman_google.com, dougkwan, Message: PTAL. Description: 2012-04-04 Ahmad Sharif Backport from mainline r179661, r179662. 2011-10-07 Andrew Stubbs gcc/ * config/arm/predicates.md (shift_amount_operand): Remove constant ra

[SH] Simplify cstore patterns

2012-04-05 Thread Oleg Endo
Hi, The attached patch simplifies the cstore patterns by removing GET_CODE checks that are already done in sh_expand_t_scc. Tested against rev 185893 with... make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a-single/-mb, -m4-single/-ml,-m4-single/-mb, -m4a-single/-ml,-m4a-si

[SH] Fold big/little endian word code with MSW and LSW

2012-04-05 Thread Oleg Endo
Hi, The attached patch folds some of the code that needs to access high/low subwords depending on the target endianess by utilizing the existing MSW and LSW macros. Tested against rev 185893 with... make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a-single/-mb, -m4-single/-m

[SH] Replace hi_const with satisfies_constraint_I16

2012-04-05 Thread Oleg Endo
Hi, The attached patch removes the hi_const function and replaces its use with satisfies_constraint_I16. Tested against rev 185893 with... make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a-single/-mb, -m4-single/-ml,-m4-single/-mb, -m4a-single/-ml,-m4a-single/-mb}" ...and

RFC: PR rtl-optimization/52876: Sign extend 32 to 64bit then clear upper 32bits fails O1 or higher

2012-04-05 Thread H.J. Lu
Hi, CSE turns (reg:DI 64) (insn 6 3 7 2 (set (reg:DI 64) (sign_extend:DI (subreg/u:SI (reg/v/f:DI 63 [ addr ]) 0))) x.i:6 122 {*extendsidi2_rex64} (nil)) into (reg/f:DI 64 [ addr ]). But nonzero_bits1 in rtlanal.c has #if defined(POINTERS_EXTEND_UNSIGNED) && !defined(HAVE_ptr_extend)

Added better handling of outdated profile data. (issue 5989046)

2012-04-05 Thread asharif
Reviewers: jh_suse.cz, davidxl, bjanakiraman_google.com, Message: Jan, please take a look and provide some feedback. Thanks, Description: Added better handling of outdated profiles. gcc dumps profile information in .gcda files that contain the function id and the corresponding counter informat

Re: [libjava] Restore HAVE_INET6 tests (PR libgcj/52645)

2012-04-05 Thread Tom Tromey
> "Rainer" == Rainer Orth writes: Rainer> 2012-03-21 Rainer Orth Rainer> PR libgcj/52645 Rainer> * gnu/java/net/natPlainDatagramSocketImplPosix.cc (setOption): Rainer> Restore HAVE_INET6 check. Rainer> * gnu/java/net/natPlainDatagramSocketImplWin32.cc (setOp

Re: RFA: consolidate DWARF strings into libiberty

2012-04-05 Thread Tom Tromey
> "Tom" == Tom Tromey writes: Tom> Here is a new patch for gcc. Tom> I still haven't updated the src side, but there's little to do there Tom> that isn't already done in this patch. Tom> Ok? Ping. Tom

[SH] Use bool return types where possible

2012-04-05 Thread Oleg Endo
Hi, The attached patch changes the return types from int to bool for functions where it seems appropriate and removes the forward declaration of the no longer exiting function sh_pass_in_reg_p. Tested by doing 'make all'. Will run the testsuite on it just in case. OK? Cheers, Oleg ChangeLog:

[PATCH, i386]: FIX PR 52882, ICE in memory_address_length, at config/i386/i386.c:23373

2012-04-05 Thread Uros Bizjak
Hello! We can allow AND zero-extended CONST_INT operands in ix86_decompose_address. These are sometimes generated by IRA for certain corner cases, as in attached testcase. 2012-04-05 Uros Bizjak PR target/52882 * config/i386/i386.c (ix86_decompose_address): Allow VOIDmode

Re: [branches/google/gcc-4_6_2-mobile] Backported r186168 from branches/google/gcc-4_6. (issue 5991052)

2012-04-05 Thread asharif
Added shenhan to the list of reviewers. http://codereview.appspot.com/5991052/

Re: [PATCH] Fix PR52614

2012-04-05 Thread Ian Lance Taylor
Richard Guenther writes: > On Thu, Apr 5, 2012 at 6:22 AM, Mike Stump wrote: >> On Apr 4, 2012, at 7:56 PM, William J. Schmidt wrote: >>> There seems to be tacit agreement that the vector tests should use >>> -fno-common on all targets to avoid the recent spate of failures (see >>> discussion in

Re: [Libiberty - V2]: Handle VMS as a LLP64 platform in splay-tree.h

2012-04-05 Thread Ian Lance Taylor
Tristan Gingold writes: > gcc/ > 2012-04-05 Tristan Gingold > > * gengtype.c (main): Make uintptr_t a known type. > > include/ > 2012-04-05 Tristan Gingold > > * splay-tree.h: Conditionnaly includes stdint.h and inttypes.h > (libi_uhostptr_t, libi_shostptr_t): Remove, repl

[PATCH][Cilkplus] Elemental function cloning with vector params.

2012-04-05 Thread Iyer, Balaji V
Hello Everyone, This patch is for Cilkplus branch affecting mainly C and C++ compilers. It will clone a function that is marked as elemental function and pass the parameters and return values into the appropriate vector registers. Thanking You, Yours Sincerely, Balaji V. Iyer. diff --git

Re: [SH] Simplify cstore patterns

2012-04-05 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch simplifies the cstore patterns by removing GET_CODE > checks that are already done in sh_expand_t_scc. > > Tested against rev 185893 with... > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a-single/-mb, > -m4-single/-ml,-m4-single/-

Re: [SH] Fold big/little endian word code with MSW and LSW

2012-04-05 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch folds some of the code that needs to access high/low > subwords depending on the target endianess by utilizing the existing MSW > and LSW macros. [snip] > * config/sh/sh.c (MSW, LSW): Move macros to... > * config/sh/sh.h: ...here. > * config/

Re: [SH] Replace hi_const with satisfies_constraint_I16

2012-04-05 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch removes the hi_const function and replaces its use > with satisfies_constraint_I16. OK. Regards, kaz

Re: [SH] Use bool return types where possible

2012-04-05 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch changes the return types from int to bool for > functions where it seems appropriate and removes the forward declaration > of the no longer exiting function sh_pass_in_reg_p. > > Tested by doing 'make all'. > Will run the testsuite on it just in case. > > OK

Re: [RS6000] Stack tie fix.

2012-04-05 Thread Alan Modra
On Thu, Apr 05, 2012 at 07:23:20PM +0200, Olivier Hainque wrote: > IIUC (please correct me if I'm wrong), every time we need to emit a tie, we > must ensure that it references all the base regs that were used before to > restore regs from the frame (to prevent the sp adjustment past the tie from

Re: [google/gcc-4_6] Added dejagnu base line for x86_64-cros-linux-gnu. (issue 5990044)

2012-04-05 Thread asharif
On 2012/04/05 17:25:44, shenhan wrote: Hi Jing and Ahmad, This adds a new test base line for x86_64-cros-linux-gnu. Please take a look. Thanks, Han lgtm. http://codereview.appspot.com/5990044/

Re: [SH] Fold big/little endian word code with MSW and LSW

2012-04-05 Thread Oleg Endo
On Fri, 2012-04-06 at 07:38 +0900, Kaz Kojima wrote: > Oleg Endo wrote: > > The attached patch folds some of the code that needs to access high/low > > subwords depending on the target endianess by utilizing the existing MSW > > and LSW macros. > [snip] > > * config/sh/sh.c (MSW, LSW): Move ma

Re: [branches/google/gcc-4_6_2-mobile] Backported r186168 from branches/google/gcc-4_6. (issue 5991052)

2012-04-05 Thread asharif
On 2012/04/05 21:56:19, asharif wrote: Added shenhan to the list of reviewers. I'm abandoning this CL in favor of a new one which will be a bulk integrate from branches/google/gcc-4_6 to branches/google/gcc-4_6_2-mobile. Sorry about the noise. http://codereview.appspot.com/5991052/

Re: [SH] Fold big/little endian word code with MSW and LSW

2012-04-05 Thread Kaz Kojima
From: Oleg Endo >> Exposing three-letter macro MSW and LSW globally looks not >> a good idea to me. > > Would 'HIGH_WORD' and 'LOW_WORD' be OK as an alternative? Sounds better but still a bit too generic. SH_{HIGH,LOW}_WORD would be Ok, though not so cool. Regards, kaz

[Patch, ARM] Enable GCC to support new ARM core cortex-m0+

2012-04-05 Thread Terry Guo
Hello, This simple patch is to enable GCC to accept new option -mcpu=cortex-m0plus in order to support newly released Cortex-M0+. Is it OK to trunk, 4.7 branch and 4.6 branch? BR, Terry 2012-04-06 Terry Guo * config/arm/arm-cores.def: Added core cortex-m0plus. * config/arm/ar