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

2011-03-15 Thread Kai Tietz
2011/3/14 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 ch

Re: [PATCH: ARM] PR 45335 Use ldrd and strd to access two consecutive words

2011-03-15 Thread Carrot Wei
The trunk is opened again, could any maintainers continue to review this patch? thanks Carrot On Tue, Jan 18, 2011 at 10:59 PM, Carrot Wei wrote: > Ramana's method is to put the instruction output and counting in on place. > So it's easy to keep them synchronized. > > My latest version of patch

Re: [PATCH v3] Re: avoid useless if-before-free tests

2011-03-15 Thread Janne Blomqvist
On Tue, Mar 8, 2011 at 19:53, Jim Meyering wrote: > Relative to v2, I've added libgo/ to the list of exempt directories and added > this recently discussed gfc_free patch, at the request of Tobias Burnus. > Also, I corrected an error in fortran's ChangeLog and removed all > whitespace changes from

[libgfortran, build] Remove AC_TRY_* macros from libgfortran configury

2011-03-15 Thread FX
Dear Fortran maintainers, dear autoconf gurus, The attached patch replaces the deprecated AC_TRY_{COMPILE,LINK,RUN} macros in libgfortran configury with their AC_*_IFELSE equivalents. Now (4.7 just branched) seems a good time, and I think I didn't screw it too much because the regenerated confi

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

2011-03-15 Thread Richard Guenther
On Mon, 14 Mar 2011, Rainer Orth wrote: > 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 avo

Re: Add a testcase for reload PR 47166

2011-03-15 Thread Richard Sandiford
Jakub Jelinek writes: > 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

[PATCH] Fix PR48031

2011-03-15 Thread Richard Guenther
We shouldn't create new VLA array accesses via fold when we are in gimple form as we have no way to fixup the gimplified element size and minimal value slots. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk (I'll make my way backwards to the release branches after 4.6.0 is o

Ping for two ARM patches

2011-03-15 Thread Richard Sandiford
Now that GCC 4.7 is open, are these two ARM patches OK to commit? http://gcc.gnu.org/ml/gcc-patches/2011-01/msg02266.html PR 47551: neon-related spill failure http://gcc.gnu.org/ml/gcc-patches/2011-01/msg02284.html PR 47553: vld1q_lane_u8 & co. don't accept lanes >= 8 Thanks, Ric

Re: Fix for PR target/47951

2011-03-15 Thread Bernd Schmidt
On 03/14/2011 01:10 PM, Henderson, Stuart wrote: > 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 m

[PATCH gcc/fortran] get rid of gfc_free

2011-03-15 Thread Jim Meyering
Janne Blomqvist wrote: > On Tue, Mar 8, 2011 at 19:53, Jim Meyering wrote: >> Relative to v2, I've added libgo/ to the list of exempt directories and added >> this recently discussed gfc_free patch, at the request of Tobias Burnus. >> Also, I corrected an error in fortran's ChangeLog and removed a

Re: [PATCH gcc/fortran] get rid of gfc_free

2011-03-15 Thread Jakub Jelinek
On Tue, Mar 15, 2011 at 11:02:38AM +0100, Jim Meyering wrote: > > Instead of "please let us know", maybe recommend using > > __builtin_expect instead? E.g. something like > > > > if (__builtin_expect (ptr != NULL, 0)) > > free (ptr); > > Good idea. Thanks. > Though how about avoiding the doub

Re: [PATCH gcc/fortran] get rid of gfc_free

2011-03-15 Thread Jim Meyering
Jakub Jelinek wrote: > On Tue, Mar 15, 2011 at 11:02:38AM +0100, Jim Meyering wrote: >> > Instead of "please let us know", maybe recommend using >> > __builtin_expect instead? E.g. something like >> > >> > if (__builtin_expect (ptr != NULL, 0)) >> > free (ptr); >> >> Good idea. Thanks. >> Thou

[PATCH] Fix PR41490

2011-03-15 Thread Richard Guenther
This fixes store-sinking (finally, broken since alias-improvements merge). To work reliably this adds an unconditional VUSE to all return statements. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-03-15 Richard Guenther PR tree-optimization/414

[PATCH] Updated DW_OP_GNU_entry_value/DW_TAG_GNU_call_site patch

2011-03-15 Thread Jakub Jelinek
Hi! Now that we are back in stage 1, I'd like move on with the entry_value/call_site debug info extensions. Here is the http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01793.html patch updated to current trunk, bootstrapped/regtested on x86_64-linux and i686-linux. Ok for trunk? 2011-03-15 Jakub

[PATCH] Updated ENTRY_VALUE patch (PR debug/45882)

2011-03-15 Thread Jakub Jelinek
Hi! Here is http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01794.html patch updated to current trunk, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk? 2011-03-15 Jakub Jelinek PR debug/45882 * rtl.def (ENTRY_VALUE): Change format from "e" to "0". * rt

[PATCH] Adjustment to DW_TAG_GNU_call_site patch for ICF debug

2011-03-15 Thread Jakub Jelinek
Hi! Here is the http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01795.html patch updated to current trunk, bootstrapped/regtested on x86_64-linux and i686-linux. Ok for trunk? 2011-03-15 Jakub Jelinek * calls.c (emit_call_1): Set MEM_EXPR on call's MEM. * var-tracking.c (prepare

[PATCH] Remove -fenable-icf-debug support

2011-03-15 Thread Jakub Jelinek
Hi! As Cary said there are no consumers of the .debug_dcall/.debug_vcall sections yet and DW_TAG_GNU_call_site info should contain a superset of that information, this patch removes -fenable-icf-debug support, if you think it is desirable to remove it. Bootstrapped/regtested on x86_64-linux and i6

[v3] Regenerate Solaris 2 baselines for GCC 4.6.0 release

2011-03-15 Thread Rainer Orth
I think we agreed that it would be useful to regenerate the Solaris 2 libstdc++ baselines close to the GCC 4.6.0 release, especially given that symbol versioning on Solaris appears in 4.6.0 for the first time and several symbols have been added to the libstdc++.so ABI since the last update. This p

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

2011-03-15 Thread Pedro Alves
Thanks. On Tuesday 15 March 2011 07:25:10, Kai Tietz wrote: > @@ -989,6 +989,13 @@ handle_toplevel_file (directive_handler_ > >in_fname = read_md_filename; >lastsl = strrchr (in_fname, '/'); > +#ifdef HAVE_DOS_BASED_FILE_SYSTEM > + { > +char *lsl2 = strrchr (in_fname, '\\'); > +

Re: [PATCH] Remove -fenable-icf-debug support

2011-03-15 Thread Rainer Orth
Hi Jakub, > As Cary said there are no consumers of the .debug_dcall/.debug_vcall > sections yet and DW_TAG_GNU_call_site info should contain a superset > of that information, this patch removes -fenable-icf-debug support, > if you think it is desirable to remove it. this should also fix PR debug/

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

2011-03-15 Thread Chung-Lin Tang
On 2011/3/15 02:41 PM, Ramana Radhakrishnan wrote: > 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

[committed] Fix fallout from snprintf folding patch (PR tree-optimization/48129)

2011-03-15 Thread Jakub Jelinek
Hi! On some targets implicit_built_in_decls[BUILT_IN_SNPRINTF] is NULL, but as we want only the return type of the snprintf function and fold_builtin_snprintf in that case is only used when __builtin_* variant is used explicitly, using built_in_decls is better and avoids the ICE. Bootstrapped/reg

Re: [v3] typeinfo tuning

2011-03-15 Thread Jonathan Wakely
On 14 March 2011 17:29, Benjamin Kosnik wrote: > > I hacked up scripts/check_compile and ran the libstdc++ testsuite to > generate 4k+ .s files, and then analyzed the resulting assembly files > for vague linkage. > > Here are the clear wins that are ABI-compatible, or in C++0x code. There > are oth

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

2011-03-15 Thread 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 Index: gcc/gcc/c-family/c-ada-spec.c ===

Re: RFA/RFC: Enable both gold and ld in a single toolchain

2011-03-15 Thread Nick Clifton
Hi Diego, Any news on this patch? :) We may be interested in using it. (Sorry for the delay in replying - I am backed up at the moment). The patch is stalled at the moment :-( I would love to push it forward, but I just do not have the time right now. I do intend to come back to it, but

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

2011-03-15 Thread Pedro Alves
On Tuesday 15 March 2011 11:46:23, Kai Tietz wrote: > 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. Thanks. -- Pedro Alves

[PATCH] Fix part of PR48037

2011-03-15 Thread Richard Guenther
This avoids spilling SSE registers to memory just because we access vector components in a C array way. The trick is to simply rewrite those accesses to proper vector selects on the tree level and promote the vector to SSA form. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to tru

[google] XFAIL execution of libmudflap.c/pass49-frag.c (issue4278049)

2011-03-15 Thread Diego Novillo
When I XFAIL'd this test, I forgot to XFAIL its execution as well. Tested on x86_64. Committed to branch. 2011-03-15 Diego Novillo * testsuite/libmudflap.c/pass49-frag.c: Also XFAIL execution. Index: libmudflap/testsuite/libmudflap.c/pass49-frag.c ==

Fix PR 45844, gfortran.dg/vect/pr45714-b.f ICE on power7

2011-03-15 Thread Alan Modra
The vsx splat load from memory insns manage to evade reg_offset_addressing_ok_p since their MEM mode is DFmode or DImode, which are not (usually) vector modes. This patch prevents reload damage in rs6000_legitimize_reload_address, and ensures the splat insn predicate is sufficiently strict. Boots

Re: [Patch] [C++0x] Support decltype as base-specifier (Bug 42603)

2011-03-15 Thread Jason Merrill
Now that we're in stage 1 of 4.7, I'm looking to apply this soon, but now I notice that it doesn't have a testcase. Could you add one? Thanks, Jason

Re: [ARM] PR 47553: vld1q_lane_u8 & co. don't accept lanes >= 8

2011-03-15 Thread Ramana Radhakrishnan
On 31/01/11 13:27, Richard Sandiford wrote: The v{ld,st}1q_lane_{u,s,p}8 intrinsic functions are supposed to take a range between 0 and 15. The function is then converted to a vld1 or vst1 of one half of the quad value. The problem is that the lane predicate doesn't account for this, and only a

[google] Created branch google/gcc-4_6 (issue4285048)

2011-03-15 Thread Diego Novillo
Created svn://gcc.gnu.org/svn/gcc/branches/google/gcc-4_6 as a sub-branch of google/main. Committed. 2011-03-15 Diego Novillo * BASE-VER: Change to 4.6.0-google * DEV-PHASE: Change to prerelease. Index: gcc/DEV-PHASE ==

[PATCH] Fix PR13954

2011-03-15 Thread Richard Guenther
This fixes PR13954, the failure to look through memcpy functions during value-numbering. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk. Richard. 2011-03-09 Richard Guenther PR tree-optimization/13954 * tree-ssa-sccvn.c (vn_reference_lookup_3): Look th

[PATCH] Fix PR47650

2011-03-15 Thread Richard Guenther
This fixes PR47650, bogus function type dumping for unprototyped and variadic functions. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied. Richard. 2011-03-15 Richard Guenther PR middle-end/47650 * tree-pretty-print.c (dump_function_declaration): Properly

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

2011-03-15 Thread Boyapati, Anitha
> >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-sibl

Re: Fix PR 45844, gfortran.dg/vect/pr45714-b.f ICE on power7

2011-03-15 Thread David Edelsohn
On Tue, Mar 15, 2011 at 8:51 AM, Alan Modra wrote: > The vsx splat load from memory insns manage to evade > reg_offset_addressing_ok_p since their MEM mode is DFmode or DImode, > which are not (usually) vector modes.  This patch prevents reload > damage in rs6000_legitimize_reload_address, and ens

Re: [PATCH gcc/fortran] get rid of gfc_free

2011-03-15 Thread Janne Blomqvist
On Tue, Mar 15, 2011 at 12:02, Jim Meyering wrote: > Janne Blomqvist wrote: >> On Tue, Mar 8, 2011 at 19:53, Jim Meyering wrote: >>> Relative to v2, I've added libgo/ to the list of exempt directories and >>> added >>> this recently discussed gfc_free patch, at the request of Tobias Burnus. >>>

Re: ivopts improvement

2011-03-15 Thread Tom de Vries
Hi Zdenek, I rewrote the patch to remove the use of use_uses_inced_iv. On 03/14/2011 03:03 PM, Zdenek Dvorak wrote: > 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 usi

Re: ivopts improvement

2011-03-15 Thread Zdenek Dvorak
Hi, > I'll try to explain what my intention with the code is, by looking at a > pre-ivopt level example. > > : > p_5 = p_3(D) + i_4(D); > > : > # p_1 = PHI > # i_2 = PHI > *p_1 = 0; > p_6 = p_1 + 1; > i_7 = i_2 + 1; > if (i_7 < n_8(D)) > goto ; > else > goto ; > > : >

Re: ivopts improvement

2011-03-15 Thread Tom de Vries
On 03/15/2011 04:10 PM, Zdenek Dvorak wrote: > Hi, > >> I'll try to explain what my intention with the code is, by looking at a >> pre-ivopt level example. >> >> : >> p_5 = p_3(D) + i_4(D); >> >> : >> # p_1 = PHI >> # i_2 = PHI >> *p_1 = 0; >> p_6 = p_1 + 1; >> i_7 = i_2 + 1; >> if

Re: PR debug/47510

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

Re: [v3] Regenerate Solaris 2 baselines for GCC 4.6.0 release

2011-03-15 Thread Rainer Orth
Rainer Orth writes: > I think we agreed that it would be useful to regenerate the Solaris 2 > libstdc++ baselines close to the GCC 4.6.0 release, especially given > that symbol versioning on Solaris appears in 4.6.0 for the first time > and several symbols have been added to the libstdc++.so ABI

C++ PATCH for c++/34758 (unclear error message about circular dependency in default arg)

2011-03-15 Thread Jason Merrill
Looking at the previous discussion of this PR, it seemed that the patch review stalled on the exact wording of the error message. I thought I'd pick it up again and try my hand at new wording; my take is "call to %qD uses the default argument for parameter %P, which is not yet defined" This

C++0x PATCH to implement core 1191 (deleted base destructor)

2011-03-15 Thread Jason Merrill
This patch implements the proposed resolution for core issue 1191: base destructors do not affect the triviality of derived constructors, but they do affect the exception specification and deletedness. Tested x86_64-pc-linux-gnu, applying to trunk. commit 15e4a01c978314724435064f20c373abf7a1bf8

Re: [PATCH gcc/fortran] get rid of gfc_free

2011-03-15 Thread Jim Meyering
Janne Blomqvist wrote: ... >> Hi Janne, >> >> These requested changes are in addition to (and independent of) >> the changes that I've already posted here. > > Yes, it was perhaps a bit unreasonable to ask you to fix this. OTOH > with your changes gfc_free() was just a wrapper around free() and > s

Re: RFA/RFC: Enable both gold and ld in a single toolchain

2011-03-15 Thread Matthias Klose
On 08.03.2011 20:52, Diego Novillo wrote: > On Tue, Aug 17, 2010 at 04:58, Nick Clifton wrote: >> Hi Raymes, >> >>> What is the status of this patch? I see the binutils part applied but >>> not the gcc part. >>> >>> http://gcc.gnu.org/ml/gcc-patches/2010-04/msg00402.html >> >> Mark Mitchell recent

C++0x PATCH to add -fconstexpr-depth

2011-03-15 Thread Jason Merrill
The draft standard specifies a minimum limit of 512 recursive constexpr calls in order to avoid, say, recursing through the entire range of int. This patch adds a check for this recursion limit and a flag to change the limit. Tested x86_64-pc-linux-gnu, applying to trunk. commit caa7a7a82f9f9

[PATCH][bug 48123] Install cpu_defines.h for freestanding C++ library.

2011-03-15 Thread 關振德
Hi, This patch fixes the problem in bug 48123. The problem is that some headers include c++config.h, which in turn includes cpu_defines.h but cpu_defines.h is not installed. This has been broken for a long time. Patch was tested by bootstrapping on x86_64 and building a cross-gcc for arm-li

C++ PATCH for core 1074 (specification of dependency of noexcept-expression)

2011-03-15 Thread Jason Merrill
Nobody could think of a reason why value-dependency of the operand of a noexcept-expression should make the noexcept itself value-dependent, so it's dropped. Tested x86_64-pc-linux-gnu, applying to trunk. commit dc2157101b5083c82984936144097fb484bdbad4 Author: Jason Merrill Date: Wed Nov 10

Re: [PATCH][bug 48123] Install cpu_defines.h for freestanding C++ library.

2011-03-15 Thread Paolo Carlini
Hi, Hi, This patch fixes the problem in bug 48123. The problem is that some headers include c++config.h, which in turn includes cpu_defines.h but cpu_defines.h is not installed. This has been broken for a long time. Patch was tested by bootstrapping on x86_64 and building a cross-gcc for

Re: [PATCH][bug 48123] Install cpu_defines.h for freestanding C++ library.

2011-03-15 Thread Paolo Carlini
... in any case, remember to add "PR libstdc++/48123" on top of the ChangeLog entry otherwise Bugzilla will not pick the commit. Paolo.

Re: [v3] Regenerate Solaris 2 baselines for GCC 4.6.0 release

2011-03-15 Thread Benjamin Kosnik
> I think we agreed that it would be useful to regenerate the Solaris 2 > libstdc++ baselines close to the GCC 4.6.0 release, especially given > that symbol versioning on Solaris appears in 4.6.0 for the first time > and several symbols have been added to the libstdc++.so ABI since the > last upda

Re: [v3] Regenerate Solaris 2 baselines for GCC 4.6.0 release

2011-03-15 Thread Benjamin Kosnik
> The lines in question are > > ##_ZTISt16nested_exception (glob) > _ZTISt16nested_exception; > ##_ZTSSt16nested_exception (glob) > _ZTSSt16nested_exception; > ##_ZTVSt16nested_exception (glob) > _ZTVSt16nested_exception; > > in CXXABI_1.3.5, but the same symbols are also

Re: [v3] Regenerate Solaris 2 baselines for GCC 4.6.0 release

2011-03-15 Thread Paolo Carlini
On 03/15/2011 08:05 PM, Benjamin Kosnik wrote: I've also changed the bad_function_call exports to not be in libsupc++, or the freestanding version. (ie to GLIBCXX from CXXABI). I think this was probably a simple thinko, right Paolo? Yes, as far as I can see now. Sorry about that and thanks for ca

Re: [v3] typeinfo tuning

2011-03-15 Thread Benjamin Kosnik
> Does exporting some of these symbols and vtables from the library mean > we're less able to change them? I'm not confident all ofthe C++0x > bits are entirely stable yet. Yes, it does. What are the new exports that concern you? There are some things I purposely did not include. For instance

Re: [PATCH][bug 48123] Install cpu_defines.h for freestanding C++ library.

2011-03-15 Thread Benjamin Kosnik
> 2011-03-14 Doug Kwan > > * include/Makefile.am (install-freestanding-headers): Install > cpu_defines.h > * include/Makefile.in: Regenerate. Looks fine to me too. -benjamin

Re: [v3] Regenerate Solaris 2 baselines for GCC 4.6.0 release

2011-03-15 Thread Paolo Carlini
... and I suppose the thinko alone should be fixed in 4_6-branch too, right? Paolo.

Re: [PATCH][bug 48123] Install cpu_defines.h for freestanding C++ library.

2011-03-15 Thread Jakub Jelinek
On Tue, Mar 15, 2011 at 07:38:27PM +0100, Paolo Carlini wrote: > > This patch fixes the problem in bug 48123. The problem is that > >some headers include c++config.h, which in turn includes cpu_defines.h > >but cpu_defines.h is not installed. This has been broken for a long > >time. Patch wa

[PATCH] Fix fallout of my fold_binary EQ/NE patch (PR middle-end/48136)

2011-03-15 Thread Jakub Jelinek
Hi! The addition of STRIP_NOPS for EQ_EXPR/NE_EXPR revealed a bunch of cases which rely on arg0 and arg1 having the same type. Only the last hunk fixes the testcase, the rest is what I saw and it was possible the types wouldn't match. Tried to construct testcases for the other cases, but didn't

Re: [v3] Regenerate Solaris 2 baselines for GCC 4.6.0 release

2011-03-15 Thread Benjamin Kosnik
> ... and I suppose the thinko alone should be fixed in 4_6-branch too, > right? Yeah. The versioning for the latest symbols has to match, or be done in a compatible way. But hang on, I'll wait on Rainer and move trunk to branch if he can confirm trunk is version-able on solaris. -benjamin

[PATCH][4.5] Fix backport of PR 47862 to 4.5 branch

2011-03-15 Thread Pat Haugen
The following changes are already present on trunk as part of the fix for PR44364. Not having them on 4.5 caused problems with the subject fix when trying to spill FP regs with a stack > 32K. Bootstrap/regtest 4.5 branch on powerpc with no new failures, ok for 4.5? -Pat 2011-03-15 Pat Hauge

Re: [v3] typeinfo tuning

2011-03-15 Thread Jonathan Wakely
On 15 March 2011 19:19, Benjamin Kosnik wrote: > >> Does exporting some of these symbols and vtables from the library mean >> we're less able to change them?  I'm not confident all ofthe C++0x >> bits are entirely stable yet. > > Yes, it does. What are the new exports that concern you? thread::_Im

Re: [C++0x patch] constexpr in attribute argument

2011-03-15 Thread Jason Merrill
Applied on trunk. Jason

Minor VRP jump threading improvement

2011-03-15 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 I noticed that VRP was missing many obvious jump threading opportunities; investigation showed that it wasn't threading through conditionals with pointer types, just those with integral types. Fix is rather obvious. Bootstrapped and regression test

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

2011-03-15 Thread Richard Henderson
On 03/11/2011 05:43 AM, Georg-Johann Lay wrote: > I did not find a way to make this work together with -mcall-prologues. > Please let me know if you have suggestion on how call prologues can be > combine with tail calls. You need a new symbol in libgcc for this. It should be easy enough to have t

[PATCH] Fix RTL DSE compile time hog (PR rtl-optimization/48141)

2011-03-15 Thread Jakub Jelinek
Hi! On the attached testcase we spend > 7 minutes in RTL DSE, as we end up with active_local_stores chain with up to 10 entries and we walk it through on each store. The following patch fixes it by throwing away from active_local_stores list stores that don't have any positions needed, but ca

Re: [PATCH] Fix RTL DSE compile time hog (PR rtl-optimization/48141)

2011-03-15 Thread Kenneth Zadeck
so how much time does this save? I agree that this is a useful simplification, but it seems unlikely to be that important in real code. it seems like the 5000 store test would in general provide a better safety valve. Kenny On 03/15/2011 07:05 PM, Jakub Jelinek wrote: Hi! On the attached t

Re: [PATCH] Fix RTL DSE compile time hog (PR rtl-optimization/48141)

2011-03-15 Thread Jakub Jelinek
On Tue, Mar 15, 2011 at 07:12:13PM -0400, Kenneth Zadeck wrote: > so how much time does this save? >From those 7 minutes back to 16 seconds (--enable-checking=yes, it was 4 seconds in 4.3 with release checking), DSE{1,2} takes each 1%, while previously it was together well over 99%. > I agree tha

Re: [v3] typeinfo tuning

2011-03-15 Thread Benjamin Kosnik
> thread::_Impl_base was one I'd been prototyping some changes to. If we > need to rework it in order for thread::id to be a trivially-copyable > (and less-than-comparable) type then we might need additional virtuals > in Impl_base, but as all I have are untested ideas for now it probably > should

Re: [v3] typeinfo tuning

2011-03-15 Thread Benjamin Kosnik
Like so. -benjamin2011-03-15 Benjamin Kosnik * include/std/thread: Revert. * src/thread.cc: Same. * config/abi/pre/gnu.ver: Same. Index: src/thread.cc === --- src/thread.cc (revision 171019) +++ src/thread.cc (working copy) @

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

2011-03-15 Thread Michael Meissner
This is a reduced version of the testcase that shows the bug that was fixed. However it is odd, in that the code that triggered the bug is setting hard register FR3 to 0 as a DImode value just before the call to __gcc_qmul to multiply the two long double values, so it needs some more looking into.

Re: [PATCH] Fix part of PR48037

2011-03-15 Thread Andrew Pinski
On Tue, Mar 15, 2011 at 5:19 AM, Richard Guenther wrote: > > This avoids spilling SSE registers to memory just because we access > vector components in a C array way.  The trick is to simply rewrite > those accesses to proper vector selects on the tree level and promote > the vector to SSA form. >

[c++] refactor some data structures into cp/parser.h (issue4295052)

2011-03-15 Thread Diego Novillo
This patch refactors some data structures out of cp/parser.c into cp/parser.h and converts the main lexer buffer into a VEC. No functional changes, other than minor data structure tweaks: - The token buffer is now a VEC. - Instead of clobbering purged tokens, we have a purged_p marker bitfield

Re: [PATCH] Fix RTL DSE compile time hog (PR rtl-optimization/48141)

2011-03-15 Thread Mike Stump
On Mar 15, 2011, at 4:05 PM, Jakub Jelinek wrote: > --- gcc/testsuite/gcc.dg/pr48141.c.jj 2011-03-15 21:48:46.0 +0100 > +++ gcc/testsuite/gcc.dg/pr48141.c2011-03-15 21:48:27.0 +0100 > @@ -0,0 +1,17 @@ > +/* PR rtl-optimization/48141 */ > +/* { dg-do compile } */ > +/* { dg-optio

Re: [PATCH] Add check before potentially dividing by 0

2011-03-15 Thread asharif
I have seen an ICE in 4.4.3 when a corrupted profile file causes a division by 0 in gcc. This is the equivalent code in trunk and this check should help fix it. Thoughts? 2011-03-15  Ahmad Sharif          * predict.c (probably_never_executed_bb_p): Add check to see        whether profile_info->ru

[x32] PATCH: Properly save the x32 new stack boundary

2011-03-15 Thread H.J. Lu
Hi, I checked in this patch to properly save the x32 new stack boundary. H.J. --- commit 556b6e068c17f0b5775140c29cd5323014775b37 Author: H.J. Lu Date: Tue Mar 15 18:00:01 2011 -0700 Properly save the x32 new stack boundary. diff --git a/libgcc/ChangeLog.x32 b/libgcc/ChangeLog.x32 index

Re: [PATCH] Updated DW_OP_GNU_entry_value/DW_TAG_GNU_call_site patch

2011-03-15 Thread Richard Henderson
On 03/15/2011 04:19 AM, Jakub Jelinek wrote: > Hi! > > Now that we are back in stage 1, I'd like move on with the > entry_value/call_site debug info extensions. > > Here is the http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01793.html > patch updated to current trunk, bootstrapped/regtested on x86_

Re: [PATCH] Updated ENTRY_VALUE patch (PR debug/45882)

2011-03-15 Thread Richard Henderson
On 03/15/2011 04:20 AM, Jakub Jelinek wrote: > Here is http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01794.html > patch updated to current trunk, bootstrapped/regtested on x86_64-linux and > i686-linux, ok for trunk? Ok. r~

Re: [PATCH] Adjustment to DW_TAG_GNU_call_site patch for ICF debug

2011-03-15 Thread Richard Henderson
On 03/15/2011 04:22 AM, Jakub Jelinek wrote: > Here is the > http://gcc.gnu.org/ml/gcc-patches/2010-12/msg01795.html > patch updated to current trunk, bootstrapped/regtested on x86_64-linux > and i686-linux. Ok for trunk? Ok. r~

Re: [PATCH] Remove -fenable-icf-debug support

2011-03-15 Thread Richard Henderson
On 03/15/2011 04:25 AM, Jakub Jelinek wrote: > As Cary said there are no consumers of the .debug_dcall/.debug_vcall > sections yet and DW_TAG_GNU_call_site info should contain a superset > of that information, this patch removes -fenable-icf-debug support, > if you think it is desirable to remove i

Re: [x32] PATCH: Update gcc.target/i386 for x32

2011-03-15 Thread H.J. Lu
On Tue, Mar 15, 2011 at 10:50:33AM -0700, H.J. Lu wrote: > On Sat, Mar 12, 2011 at 03:29:30PM -0800, H.J. Lu wrote: > > On Sat, Mar 12, 2011 at 02:44:26PM -0800, H.J. Lu wrote: > > > Many tests gcc.target/i386 use ilp32 to check ia32 and use lp64 to > > > check hardware long long support. I checke

[x32] PATCH: Also check x32 for vect_cmdline_needed target.

2011-03-15 Thread H.J. Lu
x32 is similar to x86-64. I checked this patch into x32 branch. H.J. --- diff --git a/gcc/testsuite/ChangeLog.x32 b/gcc/testsuite/ChangeLog.x32 index 32c3e34..9bcf223 100644 --- a/gcc/testsuite/ChangeLog.x32 +++ b/gcc/testsuite/ChangeLog.x32 @@ -1,5 +1,10 @@ 2011-03-15 H.J. Lu + * li

Re: [libgfortran, build] Remove AC_TRY_* macros from libgfortran configury

2011-03-15 Thread Ralf Wildenhues
* FX wrote on Tue, Mar 15, 2011 at 10:33:01AM CET: > The attached patch replaces the deprecated AC_TRY_{COMPILE,LINK,RUN} > macros in libgfortran configury with their AC_*_IFELSE equivalents. > Now (4.7 just branched) seems a good time, and I think I didn't screw > it too much because the regenerat

[patch] Enhance conditional store sinking

2011-03-15 Thread Ira Rosen
Hi, This patch adds a support of conditional store sinking for cases with multiple data references in then and else basic blocks. The correctness of the transformation is checked by verifying that there are no read-after-write and write-after-write dependencies. Bootstrapped and tested on powerpc