[PATCH] Committed: fix typo in comment

2011-03-21 Thread Mingjie Xing
Hi, I checked the patch in trunk directly since it is obvious. Committed revision 171206. Thanks, Mingjie 2011-03-21 Mingjie Xing mingjie.x...@gmail.com * tree-dfa.c (add_referenced_var): Fix typo in comment. Index: tree-dfa.c

Re: [patch, fortran] Function call optimization

2011-03-21 Thread Thomas Koenig
Hi Mikael, Here is the new version of the patch. Regression-tested. OK for trunk? OK this time. Thank you. Sende fortran/ChangeLog Sende fortran/dependency.c Sende fortran/frontend-passes.c Sende fortran/gfortran.h Sende fortran/invoke.texi

Re: [patch][4.7] Enhance XOR handling in simplify-rtx.c

2011-03-21 Thread Chung-Lin Tang
On 2011/3/18 12:18 AM, Richard Henderson wrote: On 03/16/2011 06:55 PM, Chung-Lin Tang wrote: You have to use DeMorgan's Law to distribute the ~ operator: Duh. Not sure where my head was yesterday. Let's enhance the comment for someone else having an off day. Perhaps /* Given

Re: [objc++] RFH: PR 48167 gengtype failure (issue4291054)

2011-03-21 Thread Laurynas Biveinis
2011/3/18 Nicola Pero nicola.p...@meta-innovation.com: Index: gcc/ChangeLog === --- gcc/ChangeLog       (revision 171155) +++ gcc/ChangeLog       (working copy) @@ -1,3 +1,7 @@ +2011-03-18  Nicola Pero  

Re: [patch i386,c,c++]: PR/12171 - calling convention omitted in error message

2011-03-21 Thread Kai Tietz
2011/3/18 Jason Merrill ja...@redhat.com: OK. Jason Applied first part at rev. 171209. Second part at rev. 171210. Kai

Re: [PATCH] Fix s390 delegitimize_address ICE (PR target/48213)

2011-03-21 Thread Andreas Krebbel
On 03/20/2011 08:48 PM, Jakub Jelinek wrote: Ok for trunk/4.6? 2011-03-20 Jakub Jelinek ja...@redhat.com PR target/48213 * config/s390/s390.c (s390_delegitimize_address): Don't call lowpart_subreg if orig_x has BLKmode. * gcc.dg/pr48213.c: New test. Ok.

Re: [PATCH] libstdc++ ABI issues with __int128

2011-03-21 Thread Jakub Jelinek
On Sun, Mar 20, 2011 at 01:58:24AM +0100, Benjamin Kosnik wrote: 2011-03-19 Jakub Jelinek ja...@redhat.com * config/abi/pre/gnu.ver (CXXABI_1.3): Don't export _ZT[IS][PK]*[no]. (CXXABI_1.3.5): Export _ZTI[PK]*[no]. This looks good to me, nice catch. I'll figure out what

Re: [patch i386,c,c++]: PR/12171 - calling convention omitted in error message

2011-03-21 Thread Gabriel Dos Reis
On Mon, Mar 21, 2011 at 3:33 AM, Kai Tietz ktiet...@googlemail.com wrote: 2011/3/18 Jason Merrill ja...@redhat.com: OK. Jason Applied first part at rev. 171209. Second part at rev. 171210. thanks.

[PATCH] Update some baseline_symbols.txt

2011-03-21 Thread Jakub Jelinek
Hi! This patch updates baseline_symbols.txt for a bunch of targets. This is generated with the http://gcc.gnu.org/ml/gcc-patches/2011-03/msg01175.html patch applied. 2011-03-21 Jakub Jelinek ja...@redhat.com * config/abi/post/i386-linux-gnu/baseline_symbols.txt: Update. *

Re: [PATCH] recog.c: Fix RTX unsharing in change groups

2011-03-21 Thread Eric Botcazou
I think with the patch validate_(unshare)_change becomes easier to use. You can still decide locally if you want unsharing or not. Without taking into account in which contexts your function gets called. I consider this a big advantage. You're supposed to know what you're doing though and

[PATCH] dwarf2out: Print the name of the unspec as part of the warning

2011-03-21 Thread Andreas Krebbel
Hi, with the attached patch dwarf2out tries to print the name of the unspec together with the numeric value in case the back end uses define_c_enum for the unspec constants. Ok for mainline? Bye, -Andreas- 2011-03-21 Andreas Krebbel andreas.kreb...@de.ibm.com * dwarf2out.c

Re: [build, lto] Only accept -fuse-linker-plugin if linker supports -plugin (PR lto/46944)

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: Richard Guenther rguent...@suse.de writes: On Fri, 18 Mar 2011, Rainer Orth wrote: Richard Guenther richard.guent...@gmail.com writes: It seemed to have disabled linker-plugin support for old binutils with

Re: [patch libcpp]: Improve handling of DOS-filenames and -paths

2011-03-21 Thread Kai Tietz
PING 2011/3/14 Kai Tietz ktiet...@googlemail.com: Hello, this patch fixes some DOS-filename comparision and directory-separator checks. Changelog libcpp 2011-03-14  Kai Tietz        * files.c (file_hash_eq): Use filename_cmp        instead of strcmp.        (nonexistent_file_hash_eq):

[doc] Improve Perl requirements documentation (PR bootstrap/48135)

2011-03-21 Thread Rainer Orth
While reviewing PR bootstrap/48135, it came up that the Perl requirements in install.texi are inaccurate: * On some platforms (Darwin, Solaris), Perl is a build-time requirement, but documented with the development requirements only. * On Solaris with symbol versioning enabled, it is always

Re: [doc] Update install.texi for Solaris 2, IRIX, Tru64 UNIX

2011-03-21 Thread Gerald Pfeifer
On Thu, 17 Mar 2011, Rainer Orth wrote: I've just reviewed install.texi for my platforms and found it mostly accurate. A few minor changes are desirable for the 4.6 release, but also apply to mainline since the file is identical there: * Update recommendations to binutils 2.21 since this is

[patch] Couple of tweaks to the gimplifier

2011-03-21 Thread Eric Botcazou
Hi, the attached patch makes a couple of tweaks to the gimplifier in order to help Ada, but I think that they are of general usefulness: 1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls. This is correct since stack memory isn't considered as trapping in the IL. 2)

[build, doc] Avoid File::Glob in in make_sunver.pl (PR bootstrap/48135)

2011-03-21 Thread Rainer Orth
Prompted by PR bootstrap/48135, I had a closer look at make_sunver.pl and noticed that it doesn't use File::Glob at all, but turns globs in the input version script into a regular expression for matching. Therefore, the requirement of a Perl version newer than what is bundled in Solaris 8 should

Re: [doc] Improve Perl requirements documentation (PR bootstrap/48135)

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: While reviewing PR bootstrap/48135, it came up that the Perl requirements in install.texi are inaccurate: * On some platforms (Darwin, Solaris), Perl is a build-time requirement, but documented with the development requirements only. * On Solaris

[build] Link with libstdc++ for libpwl test (PR bootstrap/48120)

2011-03-21 Thread Rainer Orth
As reported in the PR, the toplevel configure test for libpwl doesn't work since it fails to link with libstdc++. The following patch corrects this. There were a couple of issues: * One cannot use LDFLAGS here since that is passed to $CC before -lpwl, which of course fails with a static

Re: [Patch][AVR]: Support tail calls

2011-03-21 Thread Georg-Johann Lay
Denis Chertykov schrieb: 2011/3/18 Georg-Johann Lay a...@gjlay.de: Is it tested for regressions ? Denis. I ran tests against svn 170942 (latest 4.7.0 snapshot). Besides timestamps, the diff looks like this: 1435a1436,1437 XPASS: gcc.dg/sibcall-3.c execution test XPASS:

Re: [build] Link with libstdc++ for libpwl test (PR bootstrap/48120)

2011-03-21 Thread Paolo Bonzini
On 03/21/2011 12:48 PM, Rainer Orth wrote: As reported in the PR, the toplevel configure test for libpwl doesn't work since it fails to link with libstdc++. The following patch corrects this. There were a couple of issues: * One cannot use LDFLAGS here since that is passed to $CC before

[build, doc] Remove --enable-threads=solaris support

2011-03-21 Thread Rainer Orth
We still have support for --enable-threads=solaris (the old Unix International thread library) in GCC. This is pretty pointless IMO: * Both libthread and libpthread are only two different interfaces to the same implementation. * Most of our runtime libraries don't support libthread at all. *

Re: [doc] Update install.texi for Solaris 2, IRIX, Tru64 UNIX

2011-03-21 Thread Rainer Orth
Gerald Pfeifer ger...@pfeifer.com writes: On Thu, 17 Mar 2011, Rainer Orth wrote: I've just reviewed install.texi for my platforms and found it mostly accurate. A few minor changes are desirable for the 4.6 release, but also apply to mainline since the file is identical there: * Update

[Ada] Avoid duplicating branches in loop conditions

2011-03-21 Thread Eric Botcazou
This avoids duplicating branches in the generated code at -O0 for complex loop conditions involving boolean expressions. Tested on i586-suse-linux, applied on the mainline. 2011-03-21 Eric Botcazou ebotca...@adacore.com * gcc-interface/decl.c (elaborate_expression_1): When

Re: [patch] Couple of tweaks to the gimplifier

2011-03-21 Thread Richard Guenther
On Mon, Mar 21, 2011 at 12:19 PM, Eric Botcazou ebotca...@adacore.com wrote: Hi, the attached patch makes a couple of tweaks to the gimplifier in order to help Ada, but I think that they are of general usefulness:  1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls.  This is    

Re: [patch] Fix PR48183, NEON ICE in emit-rtl.c:immed_double_const() under -g

2011-03-21 Thread Chung-Lin Tang
On 2011/3/20 08:11 PM, Richard Guenther wrote: On Sun, Mar 20, 2011 at 12:01 PM, Chung-Lin Tang clt...@codesourcery.com wrote: Hi, PR48183 is a case where ARM NEON instrinsics, under -O -g, produce debug insns that tries to expand OImode (32-byte integer) zero constants, much too large to

Re: [build] Link with libstdc++ for libpwl test (PR bootstrap/48120)

2011-03-21 Thread Paolo Bonzini
On 03/21/2011 01:40 PM, Rainer Orth wrote: Paolo Bonzinibonz...@gnu.org writes: On 03/21/2011 12:48 PM, Rainer Orth wrote: As reported in the PR, the toplevel configure test for libpwl doesn't work since it fails to link with libstdc++. The following patch corrects this. There were a

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Rainer Orth
Mike Stump mikest...@comcast.net writes: +catch { exec $libtool --mode=clean rm -f $basename.lo } If libtool runs on the host, this doesn't do that, try: [remote_exec host ... ] see gcc.exp for examples. +catch { exec $libtool --mode=clean rm -f [file rootname [file tail

[PATCH] Fix PR 48143

2011-03-21 Thread Andrey Belevantsev
Hello, As noted in the PR audit trail, the scheduler crashes for this bug because we see an SSE insn without proper reservation. Uros has pointed several such insns to me. The following patch adding athlon_decode attributes is bootstrapped and tested on x86_64. Uros, you have preapproved

Re: [build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Rainer Orth
Richard Guenther richard.guent...@gmail.com writes: Ok for 4.7, RMs should decide about 4.6.1. Applied, thanks.  It would be nice to get into 4.6 (even 4.6.1) since otherwise install.texi which claims --disable-symvers as a workaround lies :-) Ok. Just to make no mistake: ok for 4.6.0 or

[testsuite] Don't xfail 22_locale/num_put/put/char/14220.cc on 64-bit Solaris 10/SPARC

2011-03-21 Thread Rainer Orth
22_locale/num_put/put/char/14220.cc XPASSes on 64-bit Solaris 10/SPARC for quite some time, so I'm removing the corresponding xfail. Tested with the appropriate runtest invocations, installed on 4.5 branch and mainline. Ok for 4.6, too? Thanks. Rainer 2011-03-12 Rainer Orth

Re: [build] Honor --disable-symvers (PR bootstrap/48135)

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: Richard Guenther richard.guent...@gmail.com writes: Ok for 4.7, RMs should decide about 4.6.1. Applied, thanks.  It would be nice to get into 4.6 (even 4.6.1) since otherwise install.texi which claims --disable-symvers as a workaround lies :-)

[Patch] Disable -mfdpic for bfin-uclinux target

2011-03-21 Thread Henderson, Stuart
-mfdpic is not supported for the bfin-uclinux target. this patch defines a macro in the uclinux header and then adds a check to bfin.c if the macro is defined. 2011-03-21 Stuart Henderson stuart.hender...@analog.com Originally from Bernd Schmidt * config/bfin/uclinux.h

[PATCH] Fix PR48210

2011-03-21 Thread Richard Guenther
Pretty obvious, require at least 1 partition. Richard. 2011-03-31 Richard Guenther rguent...@suse.de PR lto/48210 * params.def (lto-partitions): Require at least 1 partition. Index: gcc/params.def === ---

Re: [PATCH] Fix PR48210

2011-03-21 Thread Rainer Orth
Richard Guenther rguent...@suse.de writes: Index: gcc/params.def === --- gcc/params.def(revision 171229) +++ gcc/params.def(working copy) @@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_TYPE_LIST_SIZE, DEFPARAM

[PATCH] Weaken PR47661

2011-03-21 Thread Richard Guenther
We force all maybe-trapping conditions into separate stmts, but this isn't necessary w/o -fno-call-exceptions. It also confuses our branch prediction. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. The bug remains for -fnon-call-exceptions. Richard. 2011-03-21

Re: [PATCH 02/18] enforce TREE_CHAIN and TREE_TYPE accesses

2011-03-21 Thread Nathan Froyd
On Sat, Mar 12, 2011 at 01:08:23PM +0100, Eric Botcazou wrote: gcc/ada/ * gcc-interface/ada-tree.h (union lang_tree_node): Check for TS_COMMON before calling TREE_CHAIN. * gcc-interface/misc.c (gnat_init_ts): New function. (LANG_HOOKS_INIT_TS): Define. I presume that

Re: [build, doc] Obsolete Solaris 8 support

2011-03-21 Thread Joseph S. Myers
On Mon, 21 Mar 2011, Rainer Orth wrote: Therefore I'll soon commit this patch to obsolete Solaris 8 support in 4.7. It will be removed in 4.8 then. When obsoleting targets or other features, please update the release notes (gcc-4.7/changes.html). -- Joseph S. Myers jos...@codesourcery.com

invoke.texi: Fix minor typos

2011-03-21 Thread Nick Clifton
Hi Gerald, Hi Jospeh, The patch below fixes a bunch of minor typos in the invoke.texi file. I suspect that it might qualify as obvious, but I though that I ought to run it past you first. OK to apply ? Cheers Nick gcc/ChangeLog 2011-03-21 Nick Clifton ni...@redhat.com *

[patch commited] rs6000/t-freebsd 4.6 branch

2011-03-21 Thread Andreas Tobler
All, I committed the obvious patch below after asking Jakub. Andreas 2011-03-21 Andreas Tobler andre...@fgznet.ch * config/rs6000/t-freebsd: Remove duplication from file. Index: t-freebsd === --- t-freebsd (revision

Re: [PATCH] Fix PR48210

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Joseph S. Myers wrote: On Mon, 21 Mar 2011, Rainer Orth wrote: Richard Guenther rguent...@suse.de writes: Index: gcc/params.def === --- gcc/params.def(revision 171229) +++

Re: [PATCH] Fix PR48210

2011-03-21 Thread Rainer Orth
Richard Guenther rguent...@suse.de writes: DEFPARAM (MIN_PARTITION_SIZE, lto-min-partition, - Size of minimal paritition for WHOPR (in estimated instructions), + Minimal size of a paritition for LTO (in estimated instructions), Another typo :-)^

Re: [PATCH] Fix PR48210

2011-03-21 Thread Richard Guenther
On Mon, 21 Mar 2011, Rainer Orth wrote: Richard Guenther rguent...@suse.de writes: DEFPARAM (MIN_PARTITION_SIZE, lto-min-partition, - Size of minimal paritition for WHOPR (in estimated instructions), + Minimal size of a paritition for LTO (in estimated instructions),

Re: PATCH: copy_addr_to_reg: Convert to Pmode if needed

2011-03-21 Thread Richard Sandiford
H.J. Lu hongjiu...@intel.com writes: diff --git a/gcc/explow.c b/gcc/explow.c index 460af1f..9b3c082 100644 --- a/gcc/explow.c +++ b/gcc/explow.c @@ -611,6 +611,8 @@ copy_to_reg (rtx x) rtx copy_addr_to_reg (rtx x) { + if (GET_MODE (x) != VOIDmode GET_MODE (x) != Pmode) +x =

[PATCH] More ternary RHS, less single

2011-03-21 Thread Richard Guenther
This removes odd things from being a valid RHS and transitions the two easy cases from single to ternary RHS. There are now two codes left that should IMHO move over to ternary, COND_EXPR and VEC_COND_EXPR. I'll tackle them separately. Bootstrapped and tested on x86_64-unknown-linux-gnu,

Re: [PATCH] Fix shorten_compare (PR c/48197)

2011-03-21 Thread Joseph S. Myers
On Sun, 20 Mar 2011, Jakub Jelinek wrote: 2011-03-20 Jakub Jelinek ja...@redhat.com PR c/48197 * c-common.c (shorten_compare): If primopN is first sign-extended to opN and then zero-extended to result type, set primopN to opN. * gcc.c-torture/execute/pr48197.c:

[cxx-mem-model] test invalid bitfield twiddling

2011-03-21 Thread Aldy Hernandez
This test is inspired by PR 48124. [Jakub, let me know if you find any additional variations on this theme.] In the C++ memory model, non contiguous bitfields are distinct memory locations and cannot be accessed by and/or magic if we introduce data races. In the test below, we can trigger

Re: [build, doc] Obsolete Solaris 8 support

2011-03-21 Thread Rainer Orth
Joseph S. Myers jos...@codesourcery.com writes: When obsoleting targets or other features, please update the release notes (gcc-4.7/changes.html). Like this? Or should I link to the UI threads removal patch, too? I don't think such a link is needed. Ok, I've checked in the patch.

Re: [PATCH 02/18] enforce TREE_CHAIN and TREE_TYPE accesses

2011-03-21 Thread Eric Botcazou
You're right, though; they should probably be marked as such for completeness. OK to commit with those additions and reordering as above sight unseen, or would you like to see the patch? Note that I didn't ask for that, only for a comment explaining why they were set aside. So it's up to

Re: [Ada] Implement switches to reorder record components in gigi

2011-03-21 Thread Eric Botcazou
Hi Eric, this looks like a nice improvement. I noticed this mysterious undef though - what is it for? +#undef MOVE_FROM_FIELD_LIST_TO Not clear what kind of answer you're expecting. To cancel the previous define? -- Eric Botcazou

Re: [patch] Couple of tweaks to the gimplifier

2011-03-21 Thread Eric Botcazou
 1) Set TREE_THIS_NOTRAP on the INDIRECT_REF built for VLA decls.  This is correct since stack memory isn't considered as trapping in the IL. This is ok. Thanks.  2) Improve gimplification of complex conditions in COND_EXPR.  They are     naturally generated by the Ada compiler and the

Re: [PATCH] Fix shorten_compare (PR c/48197)

2011-03-21 Thread Jakub Jelinek
On Mon, Mar 21, 2011 at 04:31:31PM +, Joseph S. Myers wrote: On Sun, 20 Mar 2011, Jakub Jelinek wrote: 2011-03-20 Jakub Jelinek ja...@redhat.com PR c/48197 * c-common.c (shorten_compare): If primopN is first sign-extended to opN and then zero-extended to result

Re: [Ada] Avoid duplicating branches in loop conditions

2011-03-21 Thread Eric Botcazou
2011-03-21 Eric Botcazou ebotca...@adacore.com * gcc-interface/decl.c (elaborate_expression_1): When optimization is disabled, use the variable for bounds of loop iteration scheme. This quite mysteriously triggers a bogus -Wuninitialized warning on IRIX. Fixed thusly, tested on

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Mike Stump
On Mar 21, 2011, at 5:59 AM, Rainer Orth wrote: I've applied those changes, and also managed to fix the problems I had in Tru64 UNIX testing. Here's the patch I'd like to commit to 4.7. +# Build shared library from LIB. +proc boehm-gc-build-shlib { lib flags extra-flags } { +# FIXME:

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Rainer Orth
Mike Stump mikest...@comcast.net writes: On Mar 21, 2011, at 5:59 AM, Rainer Orth wrote: I've applied those changes, and also managed to fix the problems I had in Tru64 UNIX testing. Here's the patch I'd like to commit to 4.7. +# Build shared library from LIB. +proc boehm-gc-build-shlib {

PING: PATCH: PR target/46770: Use .init_array/.fini_array sections

2011-03-21 Thread H.J. Lu
On Mon, Mar 14, 2011 at 12:28 PM, H.J. Lu hjl.to...@gmail.com wrote: On Thu, Jan 27, 2011 at 2:40 AM, Richard Guenther richard.guent...@gmail.com wrote: On Thu, Jan 27, 2011 at 12:12 AM, H.J. Lu hongjiu...@intel.com wrote: On Tue, Dec 14, 2010 at 05:20:48PM -0800, H.J. Lu wrote: This patch

[PATCH, alpha]: Fix PR bootstrap/48214: ICE in dwarf2out_var_location

2011-03-21 Thread Uros Bizjak
Hello! Attached patch fixes recent bootstrap failure on alpha-pc-linux-gnu in in dwarf2out_var_location, at dwarf2out.c:21983. 2011-03-21 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop between a call and its CALL_ARG_LOCATION

Re: [testsuite, build] Convert boehm-gc testsuite to DejaGnu (PR boehm-gc/11412)

2011-03-21 Thread Mike Stump
On Mar 21, 2011, at 11:33 AM, Paolo Bonzini wrote: So, this still seems unportable to me? These are libtool options. Ah, that's the part I was missing... Thanks.

Re: [PATCH] libstdc++ ABI issues with __int128

2011-03-21 Thread Jakub Jelinek
On Mon, Mar 21, 2011 at 07:32:14PM +0100, Benjamin Kosnik wrote: Well, such stricter checking wouldn't be completely painless, e.g. currently both current baseline_symbols.txt and the ones I've posted don't include the two TLS: lines, because of the fear that not everyone has TLS enabled.

Re: [PATCH] prune warn_compact_unwind warnings on darwin

2011-03-21 Thread Mike Stump
On Mar 12, 2011, at 1:01 PM, Jack Howarth wrote: 2011-03-12 Jack Howarth howa...@bromo.med.uc.edu libstdc++-v3/ * testsuite/lib/prune.exp: Prune could not create compact unwind for warnings. gcc/ * testsuite/lib/prune.exp: Ditto. Checked in r171263. I modified the

Re: Cleaning up expand optabs code

2011-03-21 Thread Richard Henderson
On 03/19/2011 12:52 PM, Richard Sandiford wrote: Given the mode stuff above, I've tried to be quite draconian as far as caller-provided modes go. I think the caller really should know what mode they're dealing with. The one case where I had to hold back a bit was create_convert_operand_from,

Re: [PATCH, alpha]: Fix PR bootstrap/48214: ICE in dwarf2out_var_location

2011-03-21 Thread Richard Henderson
On 03/21/2011 11:40 AM, Uros Bizjak wrote: Hello! Attached patch fixes recent bootstrap failure on alpha-pc-linux-gnu in in dwarf2out_var_location, at dwarf2out.c:21983. 2011-03-21 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop

Re: [PATCH] Fix PR 48143

2011-03-21 Thread Andrey Belevantsev
On 21.03.2011 18:12, Uros Bizjak wrote: 2011/3/21 Andrey Belevantseva...@ispras.ru: Hello, As noted in the PR audit trail, the scheduler crashes for this bug because we see an SSE insn without proper reservation. Uros has pointed several such insns to me. The following patch adding

[PATCH, alpha]: Committed: use define_c_enum for the unspec/unspecv constants

2011-03-21 Thread Uros Bizjak
Hello! 2011-03-21 Uros Bizjak ubiz...@gmail.com * config/alpha/alpha.md (unspec): New define_c_enum. (unspecv): Ditto. Tested on alphaev68-pc-linux-gnu, committed to mainline SVN. Uros. Index: alpha.md === ---

Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Wellander, Pat - Code Sourcery
This is my first patch submittal so constructive criticism is appreciated. This patch causes PowerPC AltiVec local stack arrays 16 bytes to be aligned at 16 bytes. 2011-01-27 Pat Wellander pwell...@codesourcery.com * config/rs6000/rs6000.h (LOCAL_ALIGNMENT): AltiVec local arrays = 16

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Andrew Pinski
On Mon, Mar 21, 2011 at 2:54 PM, Wellander, Pat - Code Sourcery pwell...@codesourcery.com wrote: This is my first patch submittal so constructive criticism is appreciated. This patch causes PowerPC AltiVec local stack arrays 16 bytes to be aligned at 16 bytes. Wait I thought the alignment of

Re: [libgo] Support GOTESTFLAGS

2011-03-21 Thread Ian Lance Taylor
Rainer Orth r...@cebitec.uni-bielefeld.de writes: 2011-03-20 Rainer Orth r...@cebitec.uni-bielefeld.de * Makefile.am (CHECK): Use $(GOTESTFLAGS). * Makefile.in: Regenerate. Thanks. Committed. Ian

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Andrew Pinski
On Mon, Mar 21, 2011 at 3:21 PM, Wellander, Pat - Code Sourcery pwell...@codesourcery.com wrote: The stack is aligned at 16 bytes with AltiVec (if not, this patch would not make sense). Also, all char arrays are aligned at 16 bytes. This patch causes other local stack arrays = 16 bytes to be

Re: Patch to align local PowerPC AltiVec arrays

2011-03-21 Thread Wellander, Pat - Code Sourcery
On 3/21/2011 3:26 PM, Andrew Pinski wrote: On Mon, Mar 21, 2011 at 3:25 PM, Andrew Pinskipins...@gmail.com wrote: On Mon, Mar 21, 2011 at 3:21 PM, Wellander, Pat - Code Sourcery pwell...@codesourcery.com wrote: The stack is aligned at 16 bytes with AltiVec (if not, this patch would not make

Re: [PATCH][RFC] Add gimple_fold

2011-03-21 Thread Diego Novillo
On 03/18/2011 10:11 AM, Richard Guenther wrote: This tries to extend the previously posted CCP folding patch by introducing a generic interface for non-tree-building, GIMPLE SSA aware folding. The low-level interface for folding regular operations is /* Fold the expression composed by *CODEP,

[PATCH, 4.7] Fix PR 48226, Allow Iterator::vector vector on powerpc with VSX

2011-03-21 Thread Michael Meissner
This patch fixes a problem discovered by compiling the boost libraries with VSX enabled. In the middle of a template is the code: namespace boost { // ... template typename Iterator struct apply { // ... typedef

[PATCH, 4.7] Fix failure of two tests on 32-bit powerpc

2011-03-21 Thread Michael Meissner
I noticed that dg.gcc/torture/vector-1.c and dg.gcc/torture/vector-2.c were failing on 32-bit PowerPC Linux systems. The failure was due to the compiler complaining that by default the ABI on 32-bit does not allow passing vector items. I fixed the tests so that they pass -mabi=altivec on powerpc

Re: invoke.texi: Fix minor typos

2011-03-21 Thread Gerald Pfeifer
On Mon, 21 Mar 2011, Nick Clifton wrote: 2011-03-21 Nick Clifton ni...@redhat.com * doc/invoke.texi (Overall Options): Move closing brace to end of options list. (Optimization Options): Add missing @gol. (Directory Options): Likewise. (i386 and x86-64

Re: [patch, libgfortran] Bug 48030 - Implement read_x using fbuf_getc

2011-03-21 Thread Jerry DeLisle
Ping On 03/19/2011 09:46 PM, Jerry DeLisle wrote: Hi, The attached patch re-implements read_x using fbuf_getc. Self explanatory. Regression tested on x86-64 linux. No new test case needed. Ok for trunk? Regards, Jerry 2011-03-19 Jerry DeLisle jvdeli...@gcc.gnu.org PR libgfortran/48030 *