Re: [patch, fortran] Function call optimization

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 Sende

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

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 > > /*

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

2011/3/18 Nicola Pero : > Index: gcc/ChangeLog > === > --- gcc/ChangeLog       (revision 171155) > +++ gcc/ChangeLog       (working copy) > @@ -1,3 +1,7 @@ > +2011-03-18  Nicola Pero   > + > +       * gengtype.c (files_rules): Added ru

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

2011/3/18 Jason Merrill : > OK. > > Jason > Applied first part at rev. 171209. Second part at rev. 171210. Kai

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

On 03/18/2011 09:11 PM, Eric Botcazou wrote: >> The attached patch prevents that by doing RTX unsharing for all >> subsequent changes after finding a single change where unsharing has >> been explicitly requested. > > This looks like a big hammer. Why not doing it in canonicalize_change_group, >

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

On 03/20/2011 08:48 PM, Jakub Jelinek wrote: > Ok for trunk/4.6? > > 2011-03-20 Jakub Jelinek > > 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. Thanks! -And

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

On Sun, Mar 20, 2011 at 01:58:24AM +0100, Benjamin Kosnik wrote: > 2011-03-19 Jakub Jelinek > > * 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 happened with >

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

On Mon, Mar 21, 2011 at 3:33 AM, Kai Tietz wrote: > 2011/3/18 Jason Merrill : >> OK. >> >> Jason >> > > Applied first part at rev. 171209. Second part at rev. 171210. > thanks.

[PATCH] Update some baseline_symbols.txt

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 * config/abi/post/i386-linux-gnu/baseline_symbols.txt: Update. * config/abi/post/i486-

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

> 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

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

Richard Guenther writes: > On Fri, 18 Mar 2011, Rainer Orth wrote: > >> Richard Guenther writes: >> >> > It seemed to have disabled linker-plugin support for old binutils >> > with --with-plugin-ld=/usr/local/bin/gold, explicit -fuse-linker-plugin >> > says >> > it is not supported. The syste

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

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 * dwarf2out.c (const_ok_for_output_1): Print the

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

On Mon, 21 Mar 2011, Rainer Orth wrote: > Richard Guenther writes: > > > On Fri, 18 Mar 2011, Rainer Orth wrote: > > > >> Richard Guenther writes: > >> > >> > It seemed to have disabled linker-plugin support for old binutils > >> > with --with-plugin-ld=/usr/local/bin/gold, explicit -fuse-link

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

On 03/21/2011 11:18 AM, Rainer Orth wrote: Richard Guenther writes: On Fri, 18 Mar 2011, Rainer Orth wrote: Richard Guenther writes: It seemed to have disabled linker-plugin support for old binutils with --with-plugin-ld=/usr/local/bin/gold, explicit -fuse-linker-plugin says it is not sup

Prune empty cases from toplevel configure.ac

This patch, relative to a tree with my deprecated targets removal patch applied, further cleans up the toplevel configure.ac by removing empty cases from a case statement. The original patch still needs review by someone able to approve

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

PING 2011/3/14 Kai Tietz : > 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): Likewise. >  

Re: [patch gcc c++ c-family java fortan lto]: Fix DOS-filesystem issues

PING 2011/3/15 Kai Tietz : > 2011/3/15 Pedro Alves : >> Thanks. >> >> This looks like lbasename-able as well, getting rid of >> that HAVE_DOS...-ish foo: > > Thanks. Yes, that one I missed. > > Attached updated version of patch. > > Kai > -- |  (\_/) This is Bunny. Copy and paste | (='.'=) Bun

Re: Prune empty cases from toplevel configure.ac

On 03/21/2011 11:41 AM, Joseph S. Myers wrote: This patch, relative to a tree with my deprecated targets removal patch applied, further cleans up the toplevel configure.ac by removing empty cases from a case statement. The original patch

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

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 nec

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

While investigating PR bootstrap/48135, it came up that the submitter had run into trouble bootstrapping with mpc 0.9.0. Unfortunately, install.texi when describing support library requirements always lists `(or later)' with the individual libraries. Users who don't have the minimal required libr

Re: C-family stack check for threads

* Florian Weimer: * Thomas Klein: e.g. if using "-fstack-check=indirect -fstack-limit-symbol=stack_limit_var" Have you looked at -fsplit-stack? It emits quite similar code. Yes I have seen this. But the switches -fstack-check and -fsplit-stack are for different cases. The "stack check" (wit

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

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 thi

[patch] Couple of tweaks to the gimplifier

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) Impro

[build, doc] Obsolete Solaris 8 support

Continuing to support Solaris 8 is starting to cause lots of trouble: * Regular support has ended in March 2009, and even extended support with special arrangements will end in March 2012. * Access to Solaris 8 patches is restricted to customers with such arrangements, which I'm not. * While

extend.texi: Fix grammatical problems with builtin machine function descriptions

Hi Gerald, Hi Joseph, I have a attached a mostly mechanical patch to fix a couple of very minor grammatical issues with the built-in machine function descriptions in the extend.texi file. The first issue is with this sentence: They all generate the machine instruction that is part of

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

Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors --disable-symvers. The following patch corrects this, but also notices again the mess we've created by having at least 3 different sets of checks for symbol versioning, each with its own variations ;-( But this is not the time

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

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 go

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

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 S

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

On 03/21/2011 12:34 PM, Rainer Orth wrote: Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors --disable-symvers. The following patch corrects this, but also notices again the mess we've created by having at least 3 different sets of checks for symbol versioning, each with its

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

On Mon, 21 Mar 2011, Rainer Orth wrote: > While investigating PR bootstrap/48135, it came up that the submitter > had run into trouble bootstrapping with mpc 0.9.0. Unfortunately, > install.texi when describing support library requirements always lists > `(or later)' with the individual libraries

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

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 libpwl

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

Denis Chertykov schrieb: > 2011/3/18 Georg-Johann Lay : > >>> 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: gcc.d

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

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 -lpwl,

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

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. *

[libgo] Support GOTESTFLAGS

The following patch came in handy when I need to repeat an individual libgo test and investigate the resulting binary: $ make GOTESTFLAGS=--keep /check does the trick and is far neater than modifying the generated Makefile. Rainer 2011-03-20 Rainer Orth * Makefile.am (CHECK

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

Gerald Pfeifer 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 recommenda

[Ada] Avoid duplicating branches in loop conditions

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 * gcc-interface/decl.c (elaborate_expression_1): When optimization is disable

PING^2: Reducing the overhead of dwarf2 location tracking

Second ping for this patch (4.5+): http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00255.html It fixes a source of quadratic memory use while generating dwarf2 var-tracking information. Thanks, Richard

Re: [patch] Couple of tweaks to the gimplifier

On Mon, Mar 21, 2011 at 12:19 PM, Eric Botcazou 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 >     correct since

[Ada] Implement switches to reorder record components in gigi

This is the gigi part of the implementation of the -gnatd.r/-gnatd.v switches that allow record components to be reordered in certain cases. The algorithm is documented in the code. Tested on i586-suse-linux, applied on the mainline. 2011-03-21 Eric Botcazou * gcc-interface/decl.c

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

Paolo Bonzini writes: > On 03/21/2011 12:34 PM, Rainer Orth wrote: >> Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors >> --disable-symvers. The following patch corrects this, but also notices >> again the mess we've created by having at least 3 different sets of >> checks f

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

Paolo Bonzini 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 couple of issues: >> >> * One cannot use LDFLAGS her

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

On 2011/3/20 08:11 PM, Richard Guenther wrote: > On Sun, Mar 20, 2011 at 12:01 PM, Chung-Lin Tang > 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 represent as two

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

On 03/21/2011 01:40 PM, Rainer Orth wrote: Paolo Bonzini 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 couple of issues: *

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

Mike Stump 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 >> $testcase]] $

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

Hi Eric, this looks like a nice improvement. I noticed this mysterious undef though - what is it for? +#undef MOVE_FROM_FIELD_LIST_TO Ciao, Duncan.

[PATCH] Fix PR 48143

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 t

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

On Mon, Mar 21, 2011 at 1:39 PM, Rainer Orth wrote: > Paolo Bonzini writes: > >> On 03/21/2011 12:34 PM, Rainer Orth wrote: >>> Prompted by PR bootstrap/48135, I found that only libstdc++-v3 honors >>> --disable-symvers.  The following patch corrects this, but also notices >>> again the mess we'v

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

Richard Guenther 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 ok for 4.6.1?

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

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)

On Mon, 21 Mar 2011, Rainer Orth wrote: > Richard Guenther 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 :-) > > > > O

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

On Mon, 21 Mar 2011, Rainer Orth wrote: > 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? Ok for 4

[Patch] Disable -mfdpic for bfin-uclinux target

-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 Originally from Bernd Schmidt * config/bfin/uclinux.h (SUBTARGET_FDPIC_NOT_SUPPORTED): New macro.

[PATCH] Fix PR48210

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

Re: [PATCH] Fix PR48210

Richard Guenther 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 (PARAM_LTO_PARTITIONS, >

[PATCH] Weaken PR47661

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 Richard

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

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 pre

Re: [PATCH] Fix PR48210

On Mon, 21 Mar 2011, Rainer Orth wrote: > Richard Guenther writes: > > > Index: gcc/params.def > > === > > --- gcc/params.def (revision 171229) > > +++ gcc/params.def (working copy) > > @@ -868,7 +868,7 @@ DEFPARAM (PARAM_DEVIRT_T

Re: [build, doc] Obsolete Solaris 8 support

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

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 * doc/invoke.texi (Ov

[patch commited] rs6000/t-freebsd 4.6 branch

All, I committed the obvious patch below after asking Jakub. Andreas 2011-03-21 Andreas Tobler * config/rs6000/t-freebsd: Remove duplication from file. Index: t-freebsd === --- t-freebsd (revision 171238) +++ t-free

Re: [PATCH] Fix PR 48143

2011/3/21 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

Re: [PATCH] Fix PR48210

On Mon, 21 Mar 2011, Joseph S. Myers wrote: > On Mon, 21 Mar 2011, Rainer Orth wrote: > > > Richard Guenther writes: > > > > > Index: gcc/params.def > > > === > > > --- gcc/params.def(revision 171229) > > > +++ gcc/params.d

[patch committed] rs6000/t-freebsd trunk

All, I committed this obvious patch also to trunk. Andreas 2011-03-21 Andreas Tobler * config/rs6000/t-freebsd: Remove duplication from file. Index: t-freebsd === --- t-freebsd (revision 171238) +++ t-freebsd (wor

Re: [PATCH] Fix PR48210

Richard Guenther 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 :-)^ Rainer

Re: [PATCH] Fix PR48210

On Mon, 21 Mar 2011, Rainer Orth wrote: > Richard Guenther 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

"H.J. Lu" 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 = convert_to

[PATCH] More ternary RHS, less single

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, testing

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

On Sun, 20 Mar 2011, Jakub Jelinek wrote: > 2011-03-20 Jakub Jelinek > > 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: New test.

Re: [build, doc] Obsolete Solaris 8 support

"Joseph S. Myers" writes: > 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). Like

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

> The problem is that abicheck only flags as incompatible additions to > GLIBCXX_3.4, but not other base versions.  Ideally it would flag addition > to any version that hasn't been added (addition of the version is flagged > by > OBJECT:0:CXXABI_1.3.5 > etc. entries (no @s in it). > This patch for

Re: [build, doc] Obsolete Solaris 8 support

On Mon, 21 Mar 2011, Rainer Orth wrote: > "Joseph S. Myers" writes: > > > 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 u

[cxx-mem-model] test invalid bitfield twiddling

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 inv

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

> Most of the linux targets are actually __GXX_TYPEINFO_EQUALITY_INLINE 1 > and thus perhaps if we want to maintain some ABI compatibility with 3.4-ish > inlined typeid comparisons, we might need to export _ZTS* for _ZTI* > that could somehow be emitted for user code too. Right, for C++98 types.

Re: [build, doc] Obsolete Solaris 8 support

"Joseph S. Myers" 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. Thanks.

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

On Mon, Mar 21, 2011 at 05:52:52PM +0100, Benjamin Kosnik wrote: > > The problem is that abicheck only flags as incompatible additions to > > GLIBCXX_3.4, but not other base versions.  Ideally it would flag addition > > to any version that hasn't been added (addition of the version is flagged > > b

Re: [PATCH][ARM] Fix RVCT interoperation issue

On 14/03/11 21:21, Ramana Radhakrishnan wrote: This is OK for trunk. Committed, thanks. Andrew

Fix a few use-after-free issues

-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 This fixes a couple use-after-free problems and one use-after-free non-problem. In cfgrtl.c, redirect_branch_edge may delete its first argument, so this code is clearly erroneous: e->flags &= ~EDGE_FALLTHRU; redirected = redirect

[patch middle-end c c++]: Optimize cost of comp_type_attributes

Hello, this patch does some search-cost optimization for comp_type_attributes target hook. By recent patch about diagnostic of attributes affecting type indentity, it is possible to avoid some searching for such kind of type checks. In general it would be possible to remove this target-hook compl

Re: Fix a few use-after-free issues

On Mon, Mar 21, 2011 at 13:37, Jeff Law wrote: > Bootstrapped and regression tested on x86_64-unknown-linux-gnu.  Ok for > the trunk? OK. Diego.

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

> 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: Fix a few use-after-free issues

On Mon, Mar 21, 2011 at 11:37:16AM -0600, Jeff Law wrote: > Similarly for redirect_edge_succ_nodup in this fragment: > > ret = redirect_edge_succ_nodup (e, dest); > if (dump_file) > fprintf (dump_file, "Fallthru edge %i->%i redirected to %i\n", >e->src->index,

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

> 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

> >  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 compile

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

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 > > > > PR c/48197 > > * c-common.c (shorten_compare): If primopN is first sign-extended > > to opN and then zero-extended to result type, set

Re: [Ada] Avoid duplicating branches in loop conditions

> 2011-03-21 Eric Botcazou > > * 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 i586-suse-linux,

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

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 } { > +# FIXM

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

> 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. > If baseline_symbols.txt right now miss some symbols, it is not a b

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

Mike Stump 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 { lib flags ex

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

On Mon, Mar 21, 2011 at 19:27, Mike Stump wrote: > 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-

PING: PATCH: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

On Mon, Mar 14, 2011 at 10:49 AM, H.J. Lu wrote: >>> >>> We shouldn't save call frame hard registers as "void *".  This patch >>> changes the unwind library to save call frame hard registers as >>> _Unwind_Word.  OK for 4.7? >> >> I think this will break the ABI for the MIP

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

On Mon, Mar 14, 2011 at 12:28 PM, H.J. Lu wrote: > On Thu, Jan 27, 2011 at 2:40 AM, Richard Guenther > wrote: >> On Thu, Jan 27, 2011 at 12:12 AM, H.J. Lu wrote: >>> On Tue, Dec 14, 2010 at 05:20:48PM -0800, H.J. Lu wrote: This patch uses .init_array/.fini_array sections instead of .ct

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

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 * config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop between a call and its CALL_ARG_LOCATION note. Bootstrapped a

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

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

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 enable

Re: [PATCH] prune warn_compact_unwind warnings on darwin

On Mar 12, 2011, at 1:01 PM, Jack Howarth wrote: > 2011-03-12 Jack Howarth > > 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 Changelogs slightly,

Re: [PATCH] prune warn_compact_unwind warnings on darwin

On Mon, Mar 21, 2011 at 12:18:43PM -0700, Mike Stump wrote: > I didn't put them into the release branch yet... RM? You have a > preference on timing? Now, after .0 goes out? After .0 goes out. RC2 is building already and the fewer changes after RC2, the better. Jakub

Re: Cleaning up expand optabs code

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_fr

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

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 > > * config/alpha/alpha.c (alpha_pad_noreturn): Do not emit an unop > betwe

Re: [PATCH] Fix PR 48143

On 21.03.2011 18:12, Uros Bizjak wrote: 2011/3/21 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

  1   2   >