Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-12 Thread Eric Botcazou
> Any ELF target that overrides ASM_GENERATE_INTERNAL_LABEL is at risk > of not building any more due to the recent elfos.h changes. > > Those changes require that the label format generated by > ASM_GENERATE_INTERNAL_LABEL and TARGET_ASM_INTERNAL_LABEL are in sync, > but that is only being ensured

Re: [patch tree-optimization 1/2]: Branch-cost optimizations

2011-11-12 Thread Kai Tietz
2011/11/10 Jeff Law : > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 11/09/11 14:09, Kai Tietz wrote: >> >> Well, such a comparison-logic-folder helper - like affine-tree for >> add/subtract/scale) - is for sure something good for inner gimple >> passes building up new logic-truth express

gcov patch, multiple paths

2011-11-12 Thread Nathan Sidwell
Hi, I've committed this patch to gcov, It deals with cases where the same header file, containing inline functions etc, is included by multiple different pathnames. The current behavior treats these as distinct sources, which is (a) misleading and (b) in the absence of -p all but one instance

Fix bootstrap with solaris as

2011-11-12 Thread Jan Hubicka
Hi, my reorg of wekarefs broke solaris bootstrap. The problem is that C++ FE produces aliases for external variables/functions and expect them to be always thrown away. I think this was originally an oversight, since they are completely useless for rest of compilation but with new alias represent

Re: [PATCH 0/3] Conversion to __atomic builtins

2011-11-12 Thread David Edelsohn
On Sat, Nov 12, 2011 at 1:46 AM, Richard Henderson wrote: > The first patch removes two avoidable warnings in rs6000.md. > It seems like we could avoid many more of the remaining, but > those are harder; this one was obvious. > > The second patch is a build error.  It has appeared on this > list

[Patch Powerpc/Darwin, committed] fix PR45233

2011-11-12 Thread Iain Sandoe
OK'd by Mike in the PR thread. Applied to trunk and 4.6 Iain gcc: PR target/45233 * config/rs6000/rs6000.c (rs6000_legitimize_reload_address): Only expand a symbol ref. into an access when the entity is defined in the TU. PR bootstrap/50822 Index: gcc/con

Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-12 Thread Jason Merrill
On 11/12/2011 03:57 AM, Eric Botcazou wrote: And ASM_GENERATE_INTERNAL_LABEL uses stpcpy, which isn't portable. We just need to declare it in system.h in order to use the definition in libiberty. Jason

Re: PING 1 [Patch Ada RFA] make sure that multilibs are built with correct s-oscons.ads

2011-11-12 Thread Iain Sandoe
On 9 Nov 2011, at 09:54, Arnaud Charlet wrote: * Iain Sandoe, 2011-11-07 : Subject: PING 1 [Patch Ada RFA] make sure that multilibs are built with correct s-oscons.ads Patch looks fine to me. It's an official 'OK' then. Thanks, done (r181319). May I repeat my question re. 4.6? Since

Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-12 Thread Eric Botcazou
> We just need to declare it in system.h in order to use the definition in > libiberty. OK, this should be fine. -- Eric Botcazou

Re: [PATCH 0/4][CFT] Handle legacy __sync libcalls

2011-11-12 Thread Richard Henderson
On 11/11/2011 07:39 PM, Kaz Kojima wrote: > It seems that expand_builtin sets "target" variable to > const0_trx when "ignore" argument is set and this causes > the above ICE. I'm trying a patch ... I think the fix belongs in expand_builtin_compare_and_swap. I'm testing the following. r~ diff --

[v3] fix libstdc++/51083

2011-11-12 Thread Jonathan Wakely
This allows __promote to be used like enable_if to constrain the TR1 and C++11 maths function overloads so they only work for floating point and integral types. PR libstdc++/51083 * include/ext/type_traits.h (__promote): Only define __type member for integral and floating p

Re: PING 1 [Patch Ada RFA] make sure that multilibs are built with correct s-oscons.ads

2011-11-12 Thread Arnaud Charlet
> May I repeat my question re. 4.6? > Since this is a wrong-code situation, I would have thought it eligible for > a back-port? Well, it's not a wrong-code as in "wrong code generated by the back-end", and it's not a regression. In any case, backporting to 4.6 is fine with me. Arno

Re: [v3] fix libstdc++/51083

2011-11-12 Thread Paolo Carlini
Hi > I'm undecided whether this counts as a regression or not and if it > should be applied to the 4.6 branch - the testcases worked before TR1 > was added or without -std=c++0x. Thanks a lot for fixing this! About the branch, the patch seems safe enough to me, maybe just wait a couple if weeks?

Re: [v3] fix libstdc++/51083

2011-11-12 Thread Jonathan Wakely
On 12 November 2011 16:24, Paolo Carlini wrote: > Hi > >> I'm undecided whether this counts as a regression or not and if it >> should be applied to the 4.6 branch - the testcases worked before TR1 >> was added or without -std=c++0x. > > Thanks a lot for fixing this! About the branch, the patch see

Re: [Patch 001] [x86 backend] Define march/mtune for upcoming AMD Bulldozer procesor.

2011-11-12 Thread Uros Bizjak
On Fri, Nov 11, 2011 at 1:47 PM, Venkataramanan Kumar wrote: > Can you please check if these changes as Ok. > > On Thu, Nov 10, 2011 at 7:59 PM, Jan Hubicka wrote: >> >> > Hello! >> > >> > > This patch defines -march=bdver1 and -mtune=bdver1 flag for the >> > > upcoming >> > > AMD Bulldozer proc

Re: [PATCH 0/4][CFT] Handle legacy __sync libcalls

2011-11-12 Thread Richard Henderson
On 11/12/2011 07:56 AM, Richard Henderson wrote: > On 11/11/2011 07:39 PM, Kaz Kojima wrote: >> It seems that expand_builtin sets "target" variable to >> const0_trx when "ignore" argument is set and this causes >> the above ICE. I'm trying a patch ... > > I think the fix belongs in expand_builtin

Re: PATCH [1/n] addr32: Properly use Pmode and word_mode

2011-11-12 Thread Uros Bizjak
On Sat, Nov 12, 2011 at 3:19 AM, H.J. Lu wrote: > The current x32 implementation uses LEAs to convert 32bit address to > 64bit.  However, we can use addr32 prefix to use 32bit address directly. > It improves performance by 5% in SPEC CPU 2K/2006.  All changes are done > in x86 backend, except for

Re: [v3] fix libstdc++/51083

2011-11-12 Thread Gabriel Dos Reis
On Sat, Nov 12, 2011 at 9:59 AM, Jonathan Wakely wrote: > This allows __promote to be used like enable_if to constrain the TR1 > and C++11 maths function overloads so they only work for floating > point and integral types. > >        PR libstdc++/51083 >        * include/ext/type_traits.h (__promo

PATCH [2/n] addr32: Output REX prefix for UNSPEC_GOTNTPOFF

2011-11-12 Thread H.J. Lu
X86-64 linker optimizes TLS_MODEL_INITIAL_EXEC to TLS_MODEL_LOCAL_EXEC by checking movq foo@gottpoff(%rip), %reg and addq foo@gottpoff(%rip), %reg It uses the REX prefix to avoid the last byte of the previous instruction. With 32bit Pmode, we may not have the REX prefix and the

Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-12 Thread Dimitrios Apostolou
Hi, On Sat, 12 Nov 2011, Eric Botcazou wrote: We just need to declare it in system.h in order to use the definition in libiberty. OK, this should be fine. do the patches I sent for bug #51094 solve the problems? http://gcc.gnu.org/bugzilla/show_bug.cgi?id=51094 Thanks, Dimitris

Re: [Gcc.amd] [Patch 002] Document bdver1 in changes.html for GCC4.6

2011-11-12 Thread Gerald Pfeifer
On Fri, 11 Nov 2011, venkataramanan.ku...@amd.com wrote: > Added bdver1 information to changes.html for GCC4.6 > > is Ok to commit? Looks good to me, yes. Thanks, Gerald

Re: [PATCH 3/4] hppa: Install __sync libfuncs for linux.

2011-11-12 Thread Gerald Pfeifer
On Fri, 11 Nov 2011, Richard Henderson wrote: > @@ -136,3 +136,6 @@ along with GCC; see the file COPYING3. If not see > /* Linux always uses gas. */ > #undef TARGET_GAS > #define TARGET_GAS 1 > + > +#undef TARGET_SYNC_LIBCALL > +#define TARGET_SYNC_LIBCALL 1 John, Richard, while you are at it

Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-12 Thread David Miller
From: Eric Botcazou Date: Sat, 12 Nov 2011 09:57:04 +0100 >> Eric, it seems that most if not all of the other ELF sparc targets >> will need something like this as well but I was only able to validate >> Linux at the moment. > > Aren't all ELF targets of all architectures potentially affected?

Fix permissions in update_web_docs_svn

2011-11-12 Thread Gerald Pfeifer
Benjamin maintains the libstdc++ manuals under /onlinedocs on gcc.gnu.org, but we keep running into permissions problems. The patch below addresses this such that he, and others in the gcc group with login access, can take care of such changes. Unless there are any objections, I am planning on ch

Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-12 Thread Eric Botcazou
> Again, only those ELF targets which have an override for > ASM_GENERATE_INTERNAL_LABEL, which if you check is almost entirely > Sparc. We probably aren't looking at the same tree then. Alpha, IA-64, MIPS, HP-PA, SPARC and a few others have an override for ASM_GENERATE_INTERNAL_LABEL. > The no

Re: [PATCH] Fix Linux/sparc build after generic asm output optimizations.

2011-11-12 Thread David Miller
From: Eric Botcazou Date: Sun, 13 Nov 2011 00:26:51 +0100 >> Again, only those ELF targets which have an override for >> ASM_GENERATE_INTERNAL_LABEL, which if you check is almost entirely >> Sparc. > > We probably aren't looking at the same tree then. Alpha, IA-64, MIPS, HP-PA, > SPARC and a f

Re: [PATCH 0/4][CFT] Handle legacy __sync libcalls

2011-11-12 Thread Kaz Kojima
Richard Henderson wrote: >> I think the fix belongs in expand_builtin_compare_and_swap. >> I'm testing the following. > > Full test completed on x86_64-linux. I verified that the test > you mentioned no longer ICEs on sh4-linux. > > Committed. Thanks! Regards, kaz

PATCH for to use tree clobbers for c++/51060 (temporary re-use)

2011-11-12 Thread Jason Merrill
Now that we have a way of explicitly marking a variable as dead, we can use that to indicate the end of a temporary's lifetime by adding it as a cleanup for that temporary. Since gimple_push_cleanup still deals in trees I needed to tweak a couple of places to avoid trying to treat a clobber as

Re: [PATCH 3/4] hppa: Install __sync libfuncs for linux.

2011-11-12 Thread Dave Anglin
On 12-Nov-11, at 3:42 PM, Gerald Pfeifer wrote: John, Richard, while you are at it, mind making this GNU/Linux per guidance from RMS? (That'll save us work later on.) Yes. I don't want to participate in this controversy. I believe the file was initially contributed by developers working und

[libstdc++] Reference GNU/Linux in doc/xml/manual/using.xml

2011-11-12 Thread Gerald Pfeifer
This is the third to last patch necessary for libstdc++ from what I can tell; committed to trunk. Gerald 2011-11-13 Gerald Pfeifer * using.xml: Use GNU/Linux. Index: doc/xml/manual/using.xml === --- doc/xml/manual/using

Re: [doc] fix broken anchor in install.texi

2011-11-12 Thread Gerald Pfeifer
On Mon, 7 Nov 2011, Jonathan Wakely wrote: > We currently have a broken link in the installation docs, see below. > I've tried to fix it, but on both Fedora 15 and Debian squeeze the > docs generated by "make html" do not match the online docs because of > a different version of makeinfo (4.13 vers

C++ PATCH for c++/986 (warn about temporary bound to reference in constructor)

2011-11-12 Thread Jason Merrill
My recent work on fixing lifetime extension for temporaries bound to references makes it trivial to add this warning. Tested x86_64-pc-linux-gnu, applied to trunk. commit 1db25121594ff9405adeb5bd6892d72679bf2ba1 Author: Jason Merrill Date: Sat Nov 12 20:42:21 2011 -0500 PR c++/986