Re: [PATCH] Fix PR47127: call cloog_state_malloc and cloog_state_free only once.

2011-03-14 Thread Jakub Jelinek
On Fri, Mar 11, 2011 at 03:38:49PM -0600, Sebastian Pop wrote: > we currently call cloog_state_malloc and cloog_state_free too many > times. In CLooG-Parma, these functions contain the init and fini > functions of PPL, and so calling these in the middle of graphite would > finalize all the PPL dat

Re: fix for pr47837

2011-03-14 Thread Jakub Jelinek
On Tue, Mar 08, 2011 at 09:54:42AM -0800, Xinliang David Li wrote: > Please review the attached patch, it does some simplification of the > complicated logical or expressions (x1 or x2 or x3 ...) constructed > from control flow analysis into simpler form. BTW, PR47679 is another case of predicates

Re: ivopts improvement

2011-03-14 Thread Zdenek Dvorak
Hi, > diff -u gcc/tree-ssa-loop-ivopts.c gcc/tree-ssa-loop-ivopts.c > --- gcc/tree-ssa-loop-ivopts.c(working copy) > +++ gcc/tree-ssa-loop-ivopts.c(working copy) > @@ -1194,6 +1300,7 @@ record_use (struct ivopts_data *data, tr > gimple stmt, enum use_type use_type) > { >

PING: Reducing the overhead of dwarf2 location tracking

2011-03-14 Thread Richard Sandiford
Ping for this patch (for 4.6 and 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: [libgfortran, patch] More than 26 temporary files with weak mktemp()

2011-03-14 Thread Janne Blomqvist
On Sat, Mar 12, 2011 at 17:21, FX wrote: > The attached patch fixes half of PR 47439. On platform with weak or > historical implementations of mktemp(), this function cannot generate more > than 26 unique filenames for a given base. This happens in particular on > Windows, and limits us to 26 s

Re: fix for pr47837

2011-03-14 Thread Richard Guenther
On Sat, Mar 12, 2011 at 1:44 AM, Xinliang David Li wrote: > On Fri, Mar 11, 2011 at 11:50 AM, Jeff Law wrote: >> -BEGIN PGP SIGNED MESSAGE- >> Hash: SHA1 >> >> On 03/09/11 09:24, Xinliang David Li wrote: >>> On Wed, Mar 9, 2011 at 6:03 AM, Jeff Law wrote: >>> On 03/09/11 02:45, Richard G

Add a testcase for reload PR 47166

2011-03-14 Thread Richard Sandiford
A reload patch that I posted in January: http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00548.html turned out to be related to PR 47166, so Bernd committed a combined fix: http://gcc.gnu.org/ml/gcc-patches/2011-01/msg01620.htmlhttp://gcc.gnu.org/ml/gcc-patches/2011-01/msg01620.html Is it

Re: [libgfortran, patch] More than 26 temporary files with weak mktemp()

2011-03-14 Thread Jakub Jelinek
On Mon, Mar 14, 2011 at 12:18:16PM +0200, Janne Blomqvist wrote: > On Sat, Mar 12, 2011 at 17:21, FX wrote: > A static variable which is modified introduces a race condition. That is true. > Secondly, you reset the count to 0 at the start of every loop which > makes the static thing superfluous?

Re: [libgfortran, patch] More than 26 temporary files with weak mktemp()

2011-03-14 Thread Kai Tietz
2011/3/14 Jakub Jelinek : > On Mon, Mar 14, 2011 at 12:18:16PM +0200, Janne Blomqvist wrote: >> On Sat, Mar 12, 2011 at 17:21, FX wrote: >> A static variable which is modified introduces a race condition. > > That is true. > >> Secondly, you reset the count to 0 at the start of every loop which >>

[PATCH] Fix PR48098

2011-03-14 Thread Richard Guenther
I am currently testing the following patch for PR48098 which exposes the fact that TREE_TYPE (vector-type) does not return T when vector-type is built using make_vector_type (T). This is because we move all qualifiers from T to the vector-type and keep an unqualified element type. This finally c

Re: Add a testcase for reload PR 47166

2011-03-14 Thread Richard Guenther
On Mon, Mar 14, 2011 at 11:25 AM, Richard Sandiford wrote: > A reload patch that I posted in January: > >    http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00548.html > > turned out to be related to PR 47166, so Bernd committed a combined fix: > >     > http://gcc.gnu.org/ml/gcc-patches/2011-01/msg0

Re: [libgfortran, patch] More than 26 temporary files with weak mktemp()

2011-03-14 Thread Janne Blomqvist
On Mon, Mar 14, 2011 at 12:31, Jakub Jelinek wrote: > On Mon, Mar 14, 2011 at 12:18:16PM +0200, Janne Blomqvist wrote: >> On Sat, Mar 12, 2011 at 17:21, FX wrote: >> A static variable which is modified introduces a race condition. > > That is true. > >> Secondly, you reset the count to 0 at the s

Re: ivopts improvement

2011-03-14 Thread Tom de Vries
On 03/14/2011 10:58 AM, Zdenek Dvorak wrote: > Hi, > >> diff -u gcc/tree-ssa-loop-ivopts.c gcc/tree-ssa-loop-ivopts.c >> --- gcc/tree-ssa-loop-ivopts.c (working copy) >> +++ gcc/tree-ssa-loop-ivopts.c (working copy) >> @@ -1194,6 +1300,7 @@ record_use (struct ivopts_data *data, tr >>

Fix for PR target/47951

2011-03-14 Thread Henderson, Stuart
Hi, The attached patch is an attempt to fix PR 47951, however I'm fairly new to GCC and have little confidence in it. I've done some testing with it and it avoids the problem and doesn't appear to cause any new problems, but I'd appreciate more experienced eyes looking over it and giving pointe

Re: ivopts improvement

2011-03-14 Thread Zdenek Dvorak
Hi, > >> + /* Determine if the exit test is formulated in terms of the phi or the > >> + increment of the use iv. */ > >> + use_uses_inced_iv > >> += gimple_code (SSA_NAME_DEF_STMT (use->iv->ssa_name)) != GIMPLE_PHI; > >> + > >> + /* Determine if the exit test is before or after the in

Re: ivopts improvement

2011-03-14 Thread Tom de Vries
On 03/14/2011 01:14 PM, Zdenek Dvorak wrote: > Hi, > + /* Determine if the exit test is formulated in terms of the phi or the + increment of the use iv. */ + use_uses_inced_iv += gimple_code (SSA_NAME_DEF_STMT (use->iv->ssa_name)) != GIMPLE_PHI; + + /* Det

Re: ivopts improvement

2011-03-14 Thread Zdenek Dvorak
Hi, > >> it is trying to allow for > >> > >> do > >> { > >> *p = '\0'; > >> i++; /* use_uses_inced_iv == true */ > >> p++; /* use_after_cand_inc == true */ > >> if (!(i < n)) > >> break; > >> } > >> while (1); > >> > >> and for > >> > >> do > >> { > >> *p = '\0'; >

Re: fix for pr47837

2011-03-14 Thread Diego Novillo
On Fri, Mar 11, 2011 at 14:32, Jeff Law wrote: >> Regarding this particular patch, I hope it can be checked in to make >> the test clean. It is a simple enhancement to a wheel that is already >> there. It also serves as a case that can be referenced in the future >> when the more general mechanis

[cxx-mem-model] options for controlling data races

2011-03-14 Thread Aldy Hernandez
The following patch adds the --param(eters) needed to tweak the optimizers into allowing or disallowing data races in cross-thread visible data. There will be additional flags added later to make these parameters available to the common user. It has been suggested to use -fmemory-model=xxx,

Re: ivopts improvement

2011-03-14 Thread Tom de Vries
On 03/14/2011 01:55 PM, Zdenek Dvorak wrote: > Hi, > it is trying to allow for do { *p = '\0'; i++; /* use_uses_inced_iv == true */ p++; /* use_after_cand_inc == true */ if (!(i < n)) break; } while (1); and

Re: ivopts improvement

2011-03-14 Thread Zdenek Dvorak
Hi, > > (since the use_uses_inced_iv test is meaningless). > > To me it seems use_uses_inced_iv has meaning: > - it models something: it states whether the comparison is using > the iv increment result or the iv phi result. but that has nothing to do with the value of the iv. For instance, in

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

2011-03-14 Thread Georg-Johann Lay
Boyapati, Anitha schrieb: > To be on same page, can you explain how gcc optimizes above case? in void bar0 (void); int bar1 (int); int foo (int x) { bar0(); return bar1 (x); } x must be saved somewhere. avr-gcc choses Y. Compiled -Os -mmcu=atmega8 -fno-optimize-sibling-calls reads foo

[PATCH][i386] Implement ix86_emit_swdivsf more efficiently

2011-03-14 Thread Richard Guenther
This rewrites the iteration step of swdivsf to be more register efficient (two registers instead of four, no load of a FP constant). This matches how ICC emits the rcp sequence and causes no overall loss of precision (Micha might still remember the exact details). The patch is fallout of the work

Re: fix for pr47837

2011-03-14 Thread Xinliang David Li
On Mon, Mar 14, 2011 at 6:16 AM, Diego Novillo wrote: > On Fri, Mar 11, 2011 at 14:32, Jeff Law wrote: > >>> Regarding this particular patch, I hope it can be checked in to make >>> the test clean. It is a simple enhancement to a wheel that is already >>> there. It also serves as a case that can

Re: fix for pr47837

2011-03-14 Thread Richard Guenther
On Mon, Mar 14, 2011 at 5:33 PM, Xinliang David Li wrote: > On Mon, Mar 14, 2011 at 6:16 AM, Diego Novillo wrote: >> On Fri, Mar 11, 2011 at 14:32, Jeff Law wrote: >> Regarding this particular patch, I hope it can be checked in to make the test clean. It is a simple enhancement to a wh

Re: [PATCH][ARM] Discourage use of NEON on Cortex-A8

2011-03-14 Thread Richard Earnshaw
On Sun, 2011-03-13 at 16:31 +, Andrew Stubbs wrote: > This patch discourages the use of NEON for integer operations on ARM > Cortex-A8. > > The problem is that transferring data from NEON/VFP registers to core > registers is prohibitively expensive on A8. This should not affect > Cortex-A9

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

2011-03-14 Thread 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. (remap_filename): Likewise. Ha

Re: PATCH: Enable __float128 in gcc.target/i386/builtin-copysign.c for 32bit

2011-03-14 Thread Uros Bizjak
On Sat, Mar 12, 2011 at 7:28 PM, H.J. Lu wrote: > Hi, > > __float128 has been supported on ia32 for a while.  This patch removes > __LP64__ check.  OK for 4.7? > > Thanks. > > H.J. > --- > 2011-03-12  H.J. Lu   > >        * gcc.target/i386/builtin-copysign.c: Remove __LP64__ check. OK. Thanks, U

Re: [PATCH][i386] Implement ix86_emit_swdivsf more efficiently

2011-03-14 Thread Uros Bizjak
Hello! > This rewrites the iteration step of swdivsf to be more register > efficient (two registers instead of four, no load of a FP constant). > This matches how ICC emits the rcp sequence and causes no overall loss > of precision (Micha might still remember the exact details). The patch is > fa

[DOC PATCH] 4.6 branching changes

2011-03-14 Thread Jakub Jelinek
Hi! I've committed these changes to reflect the creation of 4.6 branch. --- c99status.html 6 Apr 2010 11:34:44 - 1.55 +++ c99status.html 14 Mar 2011 13:52:00 - 1.56 @@ -21,6 +21,7 @@ paragraph 6) do not count as library iss This page describes the C99 support in ma

[4.6 PATCH] 4.6 branching changes

2011-03-14 Thread Jakub Jelinek
Hi! I've committed following patch to 4.6 branch: 2011-03-14 Jakub Jelinek * doc/invoke.texi, doc/standards.texi: Refer to gcc-4.6/c99status.html. * DEV-PHASE: Mark "prerelease". --- gcc/doc/standards.texi (.../trunk) (revision 170924) +++ gcc/doc/standards.

[PATCH] Trunk is now 4.7

2011-03-14 Thread Jakub Jelinek
Hi! 2011-03-14 Jakub Jelinek * BASE-VER: Change to 4.7.0. --- gcc/BASE-VER(revision 170924) +++ gcc/BASE-VER(revision 170945) @@ -1 +1 @@ -4.6.0 +4.7.0 Jakub

Re: Add a testcase for reload PR 47166

2011-03-14 Thread Jakub Jelinek
On Mon, Mar 14, 2011 at 12:34:16PM +0100, Richard Guenther wrote: > On Mon, Mar 14, 2011 at 11:25 AM, Richard Sandiford > wrote: > > A reload patch that I posted in January: > > > >    http://gcc.gnu.org/ml/gcc-patches/2011-01/msg00548.html > > > > turned out to be related to PR 47166, so Bernd co

Re: [PATCH][ARM] Discourage use of NEON on Cortex-A8

2011-03-14 Thread Bernd Schmidt
On 03/14/2011 06:10 PM, Richard Earnshaw wrote: > > On Sun, 2011-03-13 at 16:31 +, Andrew Stubbs wrote: >> This patch discourages the use of NEON for integer operations on ARM >> Cortex-A8. >> >> The problem is that transferring data from NEON/VFP registers to core >> registers is prohibitiv

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

2011-03-14 Thread H.J. Lu
On Sun, Mar 6, 2011 at 4:15 PM, H.J. Lu wrote: > On Sun, Mar 6, 2011 at 3:40 PM, H.J. Lu wrote: >> On Sun, Mar 6, 2011 at 3:23 PM, Richard Guenther >> wrote: >>> On Sun, Mar 6, 2011 at 10:28 PM, H.J. Lu wrote: On Sun, Mar 6, 2011 at 1:15 PM, Andrew Pinski wrote: > On Sun, Mar 6, 2011

Re: Patch: suppress -Woverlength-strings in asm()

2011-03-14 Thread Tom Tromey
>> 2011-02-02 Tom Tromey >> >> * c-parser.c (c_parser_asm_string_literal): Clear >> warn_overlength_strings. >> >> 2011-02-02 Tom Tromey >> >> * gcc.dg/Woverlength-strings-pedantic-c90-asm.c: New file. >> * gcc.dg/Woverlength-strings-pedantic-c89-asm.c: New file. >> * gcc.dg/Woverlength-st

Re: RFA: let __extension__ suppress overlength string warning

2011-03-14 Thread Tom Tromey
>> 2011-01-18 Tom Tromey >> >> * c-parser.c (disable_extension_diagnostics): Save >> warn_overlength_strings. >> (restore_extension_diagnostics): Restore warn_overlength_strings. >> >> 2011-01-18 Tom Tromey >> >> * gcc.dg/Woverlength-strings-pedantic-c89-ext.c: New file. >> * gcc.dg/Woverl

Re: [PATCH][ARM] Discourage use of NEON on Cortex-A8

2011-03-14 Thread Andrew Stubbs
On 14/03/11 17:10, Richard Earnshaw wrote: Finally, alternatives from an insn are normally selected left-to-right from those available in a pattern, all other things being equal. So really the A8-only alternative should come after the core registers alternatives if its less preferable. OK, I c

Re: [PATCH][ARM] Discourage use of NEON on Cortex-A8

2011-03-14 Thread Richard Earnshaw
On Mon, 2011-03-14 at 17:55 +, Andrew Stubbs wrote: > On 14/03/11 17:10, Richard Earnshaw wrote: > > Finally, alternatives from an insn are normally selected left-to-right > > from those available in a pattern, all other things being equal. So > > really the A8-only alternative should come af

Re: Patch: add systemtap-style marker to _Unwind_DebugHook

2011-03-14 Thread Tom Tromey
Roland> That test would work. (That macro is not defined at all in Roland> earlier versions.) Tom> Here is an updated version. Tom> 2011-01-14 Tom Tromey Tom>* unwind-dw2.c: Include sys/sdt.h if it exists. Tom>(_Unwind_DebugHook): Use STAP_PROBE2. Tom>* config.in, configure: Rebui

[v3] libstdc++/48114

2011-03-14 Thread Paolo Carlini
Hi, a couple of issues with the C++0x , very easy to fix but very serious from the user point of view: due to a stupid thinko of mine we have been computing the binomial distribution incorrectly for p > 0.5 + we had the TR1 != C++0x definition for the geometric_distribution. Both fixes appli

Re: [RFC PATCH, i386]: Vectorize calls to floor, ceil, trunc and rint functions.

2011-03-14 Thread Uros Bizjak
On Sun, Feb 27, 2011 at 2:45 PM, Uros Bizjak wrote: > Attached [RFC] patch vectorizes calls to floor, ceil, trunc and rint > (and their float variants) functions using roundps/roundpd SSE4.1 > instruction. > > 2011-02-27  Uros Bizjak   > >        * config/i386/i386.md (ROUND_FLOOR): New constant.

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

2011-03-14 Thread Rainer Orth
Richard Guenther writes: >> > Can you update your patch with the tri-state solution? >> >> Sure if the solution is deemed acceptable. There isn't much point in >> following that route if you see problems up front. > > If that solution avoids 3) then yes, I'm fine with going that route. > Both 1

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

2011-03-14 Thread H.J. Lu
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 >>> .ctors/.dtors sections if mixing .init_array/.fini_array and

[v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Rainer Orth
In their infinite wisdom, Oracle has moved all contents from docs.sun.com to the Oracle Technical Network without putting specific redirects in place, thus invalidating more than a decade of links in a single stroke. Oh well ;-( The following patch deals with the outfall. I've decided to remove

PATCH: PR target/48084: [x32] internal compiler error: in copy_to_mode_reg, at explow.c:630

2011-03-14 Thread H.J. Lu
Hi Uros, Is this patch: http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00679.html OK for trunk? Thanks. H.J.

Re: [v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Benjamin Kosnik
> libstdc++-v3: > * doc/xml/manual/abi.xml: Replace docs.sun.com URLs by their > OTN equivalents. This is ok. -benjamin

Patch ping

2011-03-14 Thread Jakub Jelinek
Hi! http://gcc.gnu.org/ml/gcc-patches/2011-02/msg01749.html - PR middle-end/47917, snprintf folding Jakub

Re: [v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Rainer Orth
Benjamin Kosnik writes: >> libstdc++-v3: >> * doc/xml/manual/abi.xml: Replace docs.sun.com URLs by their >> OTN equivalents. > > This is ok. Thanks. Jakub, ok for 4.6, too? Unless I'm mistaken, even doc patches require RM approval now? Rainer -- ---

Re: Patch ping

2011-03-14 Thread Diego Novillo
On Mon, Mar 14, 2011 at 16:19, Jakub Jelinek wrote: > Hi! > > http://gcc.gnu.org/ml/gcc-patches/2011-02/msg01749.html >  - PR middle-end/47917, snprintf folding OK. Diego.

Re: FYI: update libstdc++ pretty-printers

2011-03-14 Thread Tom Tromey
> "Tom" == Tom Tromey writes: Tom> Doug Evans recently added some features to gdb to make it possible for Tom> the user to individually enable or disable pretty-printers. In order Tom> for this to work, the printers must conform to a certain extended Tom> protocol. Tom> I happened to notice

Re: [v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Jakub Jelinek
On Mon, Mar 14, 2011 at 09:26:56PM +0100, Rainer Orth wrote: > Benjamin Kosnik writes: > >>libstdc++-v3: > >>* doc/xml/manual/abi.xml: Replace docs.sun.com URLs by their > >> OTN equivalents. > > > > This is ok. > > Thanks. Jakub, ok for 4.6, too? Yeah. Jakub

Re: [pph] Refactor pth/pph code out of cp/parser.c

2011-03-14 Thread Diego Novillo
On Tue, Jan 4, 2011 at 11:11, Diego Novillo wrote: > On Thu, Dec 2, 2010 at 18:28, Jason Merrill wrote: >> On 12/02/2010 04:55 PM, Diego Novillo wrote: >>> >>> Jason, would it be OK if I ported cp/parser.h to mainline now? >>> It does not change anything in terms of functionality but it >>> would

Re: RFC: add a testsuite for libstdc++ pretty-printers

2011-03-14 Thread Tom Tromey
Tom> * testsuite/libstdc++-pp/simple.cc: New file. Tom> * testsuite/lib/gdb-test.exp: New file. Tom> * testsuite/libstdc++-pp/pp.exp: New file. Benjamin> feel free to spell out prettyprinters instead of using pp, if you want Benjamin> to make it very obvious what is going on in this directory. Be

Re: [v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Rainer Orth
Jakub Jelinek writes: > On Mon, Mar 14, 2011 at 09:26:56PM +0100, Rainer Orth wrote: >> Benjamin Kosnik writes: >> >> libstdc++-v3: >> >> * doc/xml/manual/abi.xml: Replace docs.sun.com URLs by their >> >> OTN equivalents. >> > >> > This is ok. >> >> Thanks. Jakub, ok for 4.6, too? > > Yeah

Re: RFC: add a testsuite for libstdc++ pretty-printers

2011-03-14 Thread Jonathan Wakely
On 14 March 2011 20:33, Tom Tromey wrote: > Tom> * testsuite/libstdc++-pp/simple.cc: New file. > Tom> * testsuite/lib/gdb-test.exp: New file. > Tom> * testsuite/libstdc++-pp/pp.exp: New file. > > Benjamin> feel free to spell out prettyprinters instead of using pp, if you > want > Benjamin> to make

Re: [v3] Replace docs.sun.com URLs by their OTN equivalents

2011-03-14 Thread Jonathan Wakely
On 14 March 2011 20:37, Rainer Orth wrote: > Jakub Jelinek writes: > >> On Mon, Mar 14, 2011 at 09:26:56PM +0100, Rainer Orth wrote: >>> Benjamin Kosnik writes: >>> >>   libstdc++-v3: >>> >>   * doc/xml/manual/abi.xml: Replace docs.sun.com URLs by their >>> >> OTN equivalents. >>> > >>> > This is

[PATCH, committed] Refix 48053, do not abort in loading 0 into VSX register under 32-bit

2011-03-14 Thread Michael Meissner
After the fix for 48053 was installed, we discovered that if reload wanted to load a 0 into a VSX register, it would fail on 32-bit. This is due to the fact that a define_split was trying to break the load into 2 separate GPR registers, but in this case there is only a single VSX register. I boot

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

2011-03-14 Thread Kai Tietz
Hello, This patch addresses some general issues occuring for DOS-based filesystems. In some cases it might be better to replace the strrchr by lbasename, but I did this initial patch more mechanical. Of special interest is the code-change in gcov.c's function make_gcov_file_name. Most of the code

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

2011-03-14 Thread Ramana Radhakrishnan
On 11/03/11 17:21, Andrew Stubbs wrote: On 09/03/11 16:12, Ramana Radhakrishnan wrote: On Fri, 2011-03-04 at 11:23 +, Andrew Stubbs wrote: The attached patch, submitted on behalf of Dan Jacobowitz, fixes an unwind bug when using RealView and libgcc. It's an old patch that has been in CodeS

[Ping] [PR19351, C++] Fix heap overflow in operator new[]

2011-03-14 Thread Florian Weimer
* Florian Weimer: > I have run "make check-c++" with no new failures on x86_64-gnu-linux > twice, with the operator new[] check enabled and disabled; there were > no new failures. If the check is disabled, trunk and patch produce > identical assembler code for the test case. > > I still need some

Re: RFC: add a testsuite for libstdc++ pretty-printers

2011-03-14 Thread Benjamin Kosnik
> Trying to dynamically adjust the tests for the gdb version looks like > a pain. > > Maybe I could change the tests to check the gdb version and xfail the > tests for gdb < 7.3. Would that be ok? Yeah. > Or I could leave it as-is, but I assume that nobody wants to see new > FAILs. Right. Un

Re: RFC: add a testsuite for libstdc++ pretty-printers

2011-03-14 Thread Mike Stump
On Mar 14, 2011, at 1:33 PM, Tom Tromey wrote: > These new tests require a very new gdb in order to fully pass. In > particular these tests pointed out some buglets that are only fixed in > CVS, not in any release. > > > Trying to dynamically adjust the tests for the gdb version looks like a > p

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

2011-03-14 Thread Pedro Alves
On Monday 14 March 2011 21:19:52, Kai Tietz wrote: > @@ -1572,6 +1579,12 @@ get_file_realbasename (const input_file > { >const char *f = get_input_file_name (inpf); >const char *lastslash = strrchr (f, '/'); > +#ifdef HAVE_DOS_BASED_FILE_SYSTEM > + const char *lastslash2 = strrchr (f, '\\

Re: PR debug/47510

2011-03-14 Thread Dodji Seketeli
Tom Tromey writes: > After a lot of discussion on irc, we came up with another idea: extend > this patch to add DW_AT_linkage_name == 't' to the anonymous > structure. This makes the DWARF remain a faithful representation of > the C++, but also makes it simple for debuginfo readers to understand

Re: [pph] Refactor pth/pph code out of cp/parser.c

2011-03-14 Thread Jason Merrill
On 03/14/2011 04:33 PM, Diego Novillo wrote: Now that we are in stage 1, I am planning to commit this to trunk soonish. Would this cause any conflicts with any branch you're planning to merge? Nope. Jason

[google] Skip g++.dg/rtti/repo1.C (issue4290043)

2011-03-14 Thread Diego Novillo
The use of --sysroot conflicts with this test because it's not passed to COLLECT_GCC_OPTIONS when using -frepo. Since we don't use -frepo, this patch skips the test. Tested on x86_64/{-m32,-m64}. Committed to google/integration. testsuite/ChangeLog.google-integration 2011-03-14 Diego Novil

[google] Do not run gcc.dg/cproj-fails-with-broken-glibc.c (issue4281048)

2011-03-14 Thread Diego Novillo
We are using an older version of glibc. I'm marking this test as compile-only in google/integration for now. Tested on x86_64/{-m32,-m64}. Committed to google/integration. 2011-03-14 Diego Novillo * gcc.dg/cproj-fails-with-broken-glibc.c: Just compile. Index: testsuite/gcc.dg/cpr

[patch, fortran] Function call optimization

2011-03-14 Thread Thomas Koenig
Hello world, the attached patch is a front-end optimization which replaces multiple calls to a function with identical argument lists with an assignment to a temporary variable, and then uses that variable in the original expression. AFAIK, this is permitted by the Fortran standard because s

[google] XFAIL some guality tests (issue4289045)

2011-03-14 Thread Diego Novillo
These tests fail on trunk as well as google/integration. XFAILing to pacify testers. Tested on x86_64/{-m32,-m64}. Committed to google/integration. 2011-03-14 Diego Novillo * gcc.dg/guality/vla-1.c: XFAIL. * gcc.dg/guality/vla-2.c: XFAIL. * gcc.dg/guality/sra-1.c:

[google] XFAIL libmudflap.c/pass49-frag.c. (issue4290044)

2011-03-14 Thread Diego Novillo
This test is failing on trunk as well as google/integration. XFAIL to pacify the testers. Tested on x86_64{-m32,-m64}. Committed to google/integration. 2011-03-14 Diego Novillo * testsuite/libmudflap.c/pass49-frag.c: XFAIL. Index: testsuite/libmudflap.c/pass49-frag.c

Re: [DOC PATCH] Add -Wunused-but-set-* note to gcc-4.6/changes.html and mention removal of in STL headers

2011-03-14 Thread Gerald Pfeifer
On Thu, 10 Mar 2011, Jakub Jelinek wrote: > This mentions -Wunused-but-set-* warnings (which are C/C++/ObjC/ObjC++, so > not sure how to mention them in the language specific areas instead) and > briefly mentions removal of includes. More should go probably > into gcc-4.6/porting_to.html. This i

Ping^2 Re: [PATCH, ARM] Fix PR 43872, incorrectly aligned VLAs

2011-03-14 Thread Chung-Lin Tang
Ping. On 2011/3/7 10:10 PM, Chung-Lin Tang wrote: > Ping. > > On 2011/2/17 06:01 PM, Chung-Lin Tang wrote: >> Hi, >> this PR is a case where we have a leaf function with a zero-size frame, >> that calls alloca() (here through a C99 VLA). >> >> The ARM backend recognizes the leaf-and-no-frame oppo

Ping^3 Re: [patch, ARM] ICE in get_arm_condition_code()

2011-03-14 Thread Chung-Lin Tang
Ping. On 2011/3/7 10:10 PM, Chung-Lin Tang wrote: > Ping. > > On 2011/1/26 11:09 AM, Chung-Lin Tang wrote: >> Ping. >> (I'll fix the minor nits before actually committing, thanks Ramana :) >> >> On 2011/1/13 08:24, Chung-Lin Tang wrote: >>> Hi, >>> there's an ICE in arm.c:get_arm_condition_code()

Ping^3 Re: [PATCH, ARM] Fix PR44557, Thumb-1 ICE

2011-03-14 Thread Chung-Lin Tang
Ping. On 2011/3/7 10:11 PM, Chung-Lin Tang wrote: > Ping. > > On 2011/1/26 11:07 AM, Chung-Lin Tang wrote: >> On 2011/1/1 19:21, Chung-Lin Tang wrote: >>> On 2010/12/21 02:03, Richard Earnshaw wrote: On Thu, 2010-12-09 at 14:08 +0800, Chung-Lin Tang wrote: > Hi, > this patch fix

Re: Fix pr48032, powerpc64 -mcmodel=medium invalid ld offset

2011-03-14 Thread Alan Modra
After some discussion with David and Richard, this is what I'm about to commit. Differs from the previous patch in returning false for BLKmode (via GET_MODE_SIZE(mode)==0 test) in the !decl branch of offsettable_ok_by_alignment. PR target/48032 * config/rs6000/rs6000.c (offsettabl

[PATCH] Re-enable lto on powerpc-apple-darwin*

2011-03-14 Thread Jack Howarth
The assembler bug in Xcode 3.2.6/4.0 can't impact powerpc-apple-darwin* (except in the old-ball case of building the powerpc-apple-darwin* target under Rosetta on Snow Leopard), lto can be re-enabled for that target in gcc trunk. Re-enabling lto for powerpc-apple-darwin* in gcc trunk allows us to

[v3] regex versioned namespaces

2011-03-14 Thread Benjamin Kosnik
Ugh. While doing other things I realized that the namespace versioning for regex header files is off: it's not on the inner-most nested namespace. Fixed thusly. tested x86/linux tested x86/linux --enable-symvers=gnu-namespace-versioned -benjamin2011-03-14 Benjamin Kosnik * include/bits/reg

Re: [DOC PATCH] Add -Wunused-but-set-* note to gcc-4.6/changes.html and mention removal of in STL headers

2011-03-14 Thread Matthias Klose
On 10.03.2011 17:04, Jakub Jelinek wrote: > +default by -Wall flag and > -Wunused-but-set-parameter > +by -Wall -W flags. -W is documented as "old option". Maybe use -Wextra instead? Matthias

Re: [v3] typeinfo tuning

2011-03-14 Thread Hans-Peter Nilsson
On Mon, 14 Mar 2011, Benjamin Kosnik wrote: > This is versioned for 4.6.0, but I'll wait a day or two to see how it > goes before I plead with the RMs. It broke build, hopefully for trivial reasons. It seems like your bugzilla options are set to not receive email, so I'll have to bug you like thi

Re: [v3] regex versioned namespaces

2011-03-14 Thread Hans-Peter Nilsson
On Mon, 14 Mar 2011, Benjamin Kosnik wrote: > > Ugh. While doing other things I realized that the namespace versioning > for regex header files is off: it's not on the inner-most nested > namespace. > > Fixed thusly. But it seems like it's not committed? brgds, H-P

Re: [v3] typeinfo tuning

2011-03-14 Thread Benjamin Kosnik
> It broke build, hopefully for trivial reasons. It seems like > your bugzilla options are set to not receive email, so I'll have > to bug you like this, and ask that you please have a look at > PR48130. No problem. Sorry about the breakage. Indeed, it was trivial. > > tested x86_64/linux t

Re: [PATCH, ARM] Fix PR 43872, incorrectly aligned VLAs

2011-03-14 Thread Ramana Radhakrishnan
On 17/02/11 10:01, Chung-Lin Tang wrote: Hi, this PR is a case where we have a leaf function with a zero-size frame, that calls alloca() (here through a C99 VLA). The ARM backend recognizes the leaf-and-no-frame opportunity to save an unneeded stack alignment. But when calling alloca(), the allo