Re: Fix libgcc for FreeBSD 10

2011-11-14 Thread Paolo Bonzini
On 11/13/2011 04:18 PM, Gerald Pfeifer wrote: The snippet below was written when a new major release of FreeBSD would happen every couple of years, not as frequently as recently the case. FreeBSD 1.x and 2.x, and a.out support, have been obsolete for what must be a decade or so, yet the regular

Re: [Patch testsuite/darwin] fix PR testsuite/51059

2011-11-14 Thread Rainer Orth
Iain, This probably qualifies as obvious - but having discussed some of the background with Mike .. .. there are other ways of solving the problem - although probably rather heavy-weight for this problem. .. So, I'll let him have the say... OK for trunk? Iain testsuite: PR

[v3] Remove constexpr from frexp, modf, remquo

2011-11-14 Thread Paolo Carlini
Hi, Daniel noticed that in cmath we have functions marked constexpr which aren't really viable anyway due to the pointer parameter. Seems basically harmless given our way to deal with these matters, but I agree can be misleading. Tested x86_64-linux, committed. Paolo.

Re: [Patch testsuite/darwin] fix PR testsuite/51059

2011-11-14 Thread Iain Sandoe
Hi Rainer, On 14 Nov 2011, at 09:50, Rainer Orth wrote: +/* { dg-options -O2 -fprofile-arcs -ftest-coverage - flat_namespace - undefined suppress { target *-*-darwin* } } */ /* { dg-require-weak } */ I'd prefer if you'd use the new dg-additional-options instead of duplicating the complete

[PATCH, PR51005] Fix slow down of compilation of 20001226-1.c by -ftree-tail-merge

2011-11-14 Thread Tom de Vries
Richard, This patch fixes the slow down in the compilation of 20001226-1.c caused by -ftree-tail-merge. -ftree-tree-tail-merge removes half of the 16000 basic blocks, but increases compilation time with a factor 2.4 from 34 seconds to 80 seconds (not counting the further increase of 46 seconds

CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Hans-Peter Nilsson
From: Bernd Schmidt ber...@codesourcery.com Date: Mon, 14 Nov 2011 10:51:56 +0100 On 11/11/11 20:13, Hans-Peter Nilsson wrote: AFAICT, your patch has got sufficiently testing now (on three targets to boot) to be considered safe to check in. Or is something amiss? (If it's the

Re: vector garbaged collected while still in use

2011-11-14 Thread Jan Hubicka
... probably here. Though I'd also approve a patch that simply removes the ggc_collect call (and the nested parameter). Honza - you probably added the ggc_collect - what's the reason to do it in this lowlevel place? The ggc_collect was always here. Before unit-at-a-time we used to run

Re: vector garbaged collected while still in use

2011-11-14 Thread Jan Hubicka
Here is the revised patch. Bootstrap and regression tested on linux/x86-64. Honza, can you comment on the implication of this change? Well, as I explained in previous email, this was traditinal place where frontends collect garbage while parsing, so you would need to verify that all the

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Ramana Radhakrishnan
Someone with approval rights: pretty please? Can I add my +1 pretty please as well here :) ? According to #c3 this fixes arm-linux-gnueabi cross-builds for C++ as well and potentially allows this to bootstrap again. I have kicked off a bootstrap and test run on arm-linux-gnueabi . cheers

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Rainer Orth
Ramana Radhakrishnan ramana.radhakrish...@linaro.org writes: Someone with approval rights: pretty please? Can I add my +1 pretty please as well here :) ? According to #c3 this fixes arm-linux-gnueabi cross-builds for C++ as well and potentially allows this to bootstrap again. I have kicked

[patch committed]: Fix recent obvious typo in gcov.c

2011-11-14 Thread Kai Tietz
Hi, this fixes bootstrap failure for Windows targets due recent change in gcov.c. 2011-11-14 Kai Tietz kti...@redhat.com * gcov.c (generate_results): Add missing semicolon and correct indent. Committed at revision 181353. Regards, Kai Index: gcov.c

Re: [google] ThreadSanitizer instrumentation pass (issue 5303083)

2011-11-14 Thread Xinliang David Li
Ok for google/main after compiler bootstrap and regression test (without ftsan), and some large tests with tsan turned on (as many as you can but at your discretion). David On Sun, Nov 13, 2011 at 11:59 PM, dvyu...@google.com wrote: On 2011/11/11 00:00:35, davidxl wrote: Have you run through

[pph] Add flag -fpph-include-tree (issue5376093)

2011-11-14 Thread Diego Novillo
This patch is an enhanced variant of -H. It prints more information about the included files. I'm using it as input to a tool to automatically generate PPH mappings for large TUs. I don't think this will survive for long, but it is useful for now. Tested on x86_64. Committed to branch.

Re: [PATCH] Account for devirtualization opportunities in inliner

2011-11-14 Thread Jan Hubicka
Hi, the patch is OK. Thanks! Honza

Re: Memset/memcpy patch

2011-11-14 Thread H.J. Lu
On Mon, Nov 14, 2011 at 9:03 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is hopefully final variant of patch. The epilogue code was broken in some scenarios for memset, but should work safely now.  I also fixed the tables for core/buldozer/amdfam10 chips. But before it can be comitted, we

[PATCH] postreload: Invalidate reg_state info at barrier/volatile insns

2011-11-14 Thread Andreas Krebbel
Hi, a problem in postreload causes a miscompilation of libgomp on s390x. insn 1: r1 = r1 + r2 insn 2: r3 = r1 insn 3: unspec_volatile insn 4: use r1 With the patch from http://gcc.gnu.org/bugzilla/show_bug.cgi?id=49452 the regstate information is resetted at the unspec_volatile instead of

Fix emutls bootstrap issues

2011-11-14 Thread Jan Hubicka
Hi, emutls is creating variables late that confuses varpool since varpool_finalize_decl is supposed to be used only before cgraph construction. This patch fixes the problem by introducing varpool_add_new_variable that works same as cgraph_add_new_function (i.e. allows middle-end/back-end to

Re: Memset/memcpy patch

2011-11-14 Thread Jan Hubicka
On Mon, Nov 14, 2011 at 9:03 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is hopefully final variant of patch. The epilogue code was broken in some scenarios for memset, but should work safely now.  I also fixed the tables for core/buldozer/amdfam10 chips. But before it can be

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Richard Henderson
On 11/14/2011 04:10 AM, Hans-Peter Nilsson wrote: Looks like all we need is a positive review of http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01409.html and a ChangeLog entry to unbreak three or more targets. Someone with approval rights: pretty please? That patch is ok. r~

Re: PR c++/30195

2011-11-14 Thread Jason Merrill
Looks good; I fixed a few whitespace and formatting issues and checked in the patch. Thanks a lot! Jason

Re: [PATCH 1/4] arm: Install __sync libfuncs for Linux.

2011-11-14 Thread David Gilbert
On 11 November 2011 23:32, Richard Henderson r...@redhat.com wrote: Cc: Richard Earnshaw richard.earns...@arm.com Cc: Ramana Radhakrishnan ramana.radhakrish...@arm.com ---  gcc/config/arm/arm.c |    4  1 files changed, 4 insertions(+), 0 deletions(-) diff --git a/gcc/config/arm/arm.c

PATCH: Check ia32 instead of ilp32

2011-11-14 Thread H.J. Lu
Hi, ilp32 includes both x32 and ia32. -march=i486 only applies to ia32. I checked in this patch to check ia32 instead of ilp32. H.J. --- Index: ChangeLog === --- ChangeLog (revision 181359) +++ ChangeLog (working copy) @@ -1,3

Re: [PATCH 1/4] arm: Install __sync libfuncs for Linux.

2011-11-14 Thread Richard Henderson
On 11/14/2011 08:08 AM, David Gilbert wrote: There is unfortunately no guarantee that your kernel has support for the 64bit cases, since this was only recently added - and the libgcc code checks and aborts if linked in. (As far as I can tell there is approximately one potential user of 64bit

Re: [PATCH 1/4] arm: Install __sync libfuncs for Linux.

2011-11-14 Thread Ramana Radhakrishnan
There is unfortunately no guarantee that your kernel has support for the 64bit cases, since this was only recently added - and the libgcc code checks and aborts if linked in. I don't see how this patch will change status-quo. Richard: I've not been able to test the patch yet given that trunk

[PATCH, i386]: Vectorize (int) {floor,ceil,round}{,f} with SSE4.1+

2011-11-14 Thread Uros Bizjak
Hello! Currently, gcc vectorizes {floor,ceil,round}{,f}, but not i.e. (int) floor [and other rounding functions] due to conversion of (int) floor - ifloor that inhibits vectorization. Attached patch introduces new expanders that handle rounding+conversions with expansion to

Re: [PATCH, testsuite]: Do not run simulate-thread on powerpc*-*-darwin*

2011-11-14 Thread Mike Stump
On Nov 13, 2011, at 1:32 PM, Iain Sandoe develo...@sandoe-acoustics.co.uk wrote: we have a problem on powerpc-darwin9 OK for trunk? Ok, please. Include a comment line just before staying why.

Re: [PATCH 1/4] arm: Install __sync libfuncs for Linux.

2011-11-14 Thread Andrew MacLeod
On 11/14/2011 01:18 PM, Richard Henderson wrote: On 11/14/2011 08:08 AM, David Gilbert wrote: Andrew, what's the state of discussion on the atomics library? While I understand that vendors want to be able to replace that library, and thus we want to settle on a standard ABI, I think *not*

Couple of fixes to GRAPHITE-OpenCL

2011-11-14 Thread Alexey Kravets
Hi guys, These are couple of patches for Graphite-OpenCL. First patch fixes lack of MINUS_EXPR processing in OpenCL kernel code generation. The second one fixed VEC related issue, which caused ISE on some tests from NPB (NASA parallel benchmark). ChangeLogs: [1] *

Re: gcov patch, multiple paths

2011-11-14 Thread Alen Skondro
This breaks the mingw64 build with S_ISLNK being unresolved. Since windows doesn't know anything about symbolic links, could something like this help? diff --git a/gcc/config/i386/xm-mingw32.h b/gcc/config/i386/xm-mingw32.h index e0dd3f3..34663ac 100644 --- a/gcc/config/i386/xm-mingw32.h +++

Re: gcov patch, multiple paths

2011-11-14 Thread Nathan Sidwell
On 11/14/11 19:51, Alen Skondro wrote: This breaks the mingw64 build with S_ISLNK being unresolved. Since windows doesn't know anything about symbolic links, could something like this help? thanks, Joseph also pointed this out to me. I have a slightly different patch in testing (posix

Re: [Patch]: PR49868: Named address space support for AVR, #4

2011-11-14 Thread Georg-Johann Lay
Denis Chertykov wrote: Let's wait for http://gcc.gnu.org/ml/gcc-patches/2011-10/msg01874.html As that patch is now upstream, here the updated version. Difference to the prior implementation of AS1 is: o If no LPMX instruction is available and more than 2 bytes have to be loaded, a libgcc

[PATCH, testsuite]: Cleanup rounding *-vec.c test

2011-11-14 Thread Uros Bizjak
Hello! 2011-11-14 Uros Bizjak ubiz...@gmail.com * gcc.target/i386/avx-recip-vec.c: New test. * gcc.target/i386/avx-lrintf-vec.c: Ditto. * gcc.target/i386/avx-lrint-vec.c: Ditto. * gcc.target/i386/avx-ceilf-vec.c: Include sse4_1-ceilf-vec.c. *

Re: [Patch] Take 2: Move Objective-C runtime flags to modern options system.

2011-11-14 Thread Iain Sandoe
V2 is simpler, than http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01335.html because (using the approach suggested by Joseph) we can initialize flag_next_runtime early enough to avoid having to deal with any other flags. [I'll still get rid of the flag_objc_sjlj_exceptions flag in a

Re: Memset/memcpy patch

2011-11-14 Thread H.J. Lu
2011/11/14 Jan Hubicka hubi...@ucw.cz: On Mon, Nov 14, 2011 at 9:03 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is hopefully final variant of patch. The epilogue code was broken in some scenarios for memset, but should work safely now.  I also fixed the tables for

Re: Memset/memcpy patch

2011-11-14 Thread Iain Sandoe
On 14 Nov 2011, at 20:36, H.J. Lu wrote: 2011/11/14 Jan Hubicka hubi...@ucw.cz: On Mon, Nov 14, 2011 at 9:03 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is hopefully final variant of patch. The epilogue code was broken in some scenarios for memset, but should work safely now. I also

Re: [PATCH, PR51005] Fix slow down of compilation of 20001226-1.c by -ftree-tail-merge

2011-11-14 Thread Richard Guenther
On Mon, Nov 14, 2011 at 2:02 PM, Tom de Vries tom_devr...@mentor.com wrote: Richard, This patch fixes the slow down in the compilation of 20001226-1.c caused by -ftree-tail-merge. -ftree-tree-tail-merge removes half of the 16000 basic blocks, but increases compilation time with a factor 2.4

Re: Memset/memcpy patch

2011-11-14 Thread H.J. Lu
On Mon, Nov 14, 2011 at 12:40 PM, Iain Sandoe develo...@sandoe-acoustics.co.uk wrote: On 14 Nov 2011, at 20:36, H.J. Lu wrote: 2011/11/14 Jan Hubicka hubi...@ucw.cz: On Mon, Nov 14, 2011 at 9:03 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is hopefully final variant of patch. The

Re: Memset/memcpy patch

2011-11-14 Thread Paolo Carlini
Hi, 2011-11-14 Zolotukhin Michael michael.v.zolotuk...@gmail.com Jan Hubicka j...@suse.cz Zolotukhin Michael works for Intel and has copyright assignment with FSF. Looks like we have a bootstrap issue, thus sorry if may message may appear stupid nitpicking: why Zolotukhin

Re: [PATCH, PR51005] Fix slow down of compilation of 20001226-1.c by -ftree-tail-merge

2011-11-14 Thread David Miller
From: Richard Guenther richard.guent...@gmail.com Date: Mon, 14 Nov 2011 21:43:30 +0100 On Mon, Nov 14, 2011 at 2:02 PM, Tom de Vries tom_devr...@mentor.com wrote: This patch fixes the slow down in the compilation of 20001226-1.c caused by -ftree-tail-merge. ... ok for trunk? Ok. Tom,

Re: Memset/memcpy patch

2011-11-14 Thread Iain Sandoe
On 14 Nov 2011, at 20:44, H.J. Lu wrote: On Mon, Nov 14, 2011 at 12:40 PM, Iain Sandoe develo...@sandoe-acoustics.co.uk wrote: On 14 Nov 2011, at 20:36, H.J. Lu wrote: 2011/11/14 Jan Hubicka hubi...@ucw.cz: On Mon, Nov 14, 2011 at 9:03 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this

gcov, fix S_ISLNK

2011-11-14 Thread Nathan Sidwell
I've committed this to fix windows and other non-modern posix systems. nathan 2011-11-14 Nathan Sidwell nat...@acm.org * gcov.c (canonicalize_name): Protect use of S_ISLNK. Index: gcov.c === --- gcov.c (revision

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

2011-11-14 Thread David Edelsohn
On Sat, Nov 12, 2011 at 1:46 AM, Richard Henderson r...@redhat.com wrote: There are a couple of instances in which the paper doesn't cover the handling of memory_model_consume, and I made a best guess.  These are indicated by /* ??? */ markers.  I would be obliged if someone could verify

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Alan Modra
On Mon, Nov 14, 2011 at 07:48:03AM -1000, Richard Henderson wrote: On 11/14/2011 04:10 AM, Hans-Peter Nilsson wrote: Looks like all we need is a positive review of http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01409.html and a ChangeLog entry to unbreak three or more targets. Someone

coverage, propagate visibility

2011-11-14 Thread Nathan Sidwell
This patch propagates visibility information in the same way that weakness is for externally visible coverage objects. I don't know whether this will fix the reported firefox build failure -- can you try? Even if it doesn't, I'm pretty sure it's an improvement, but I'll leave it for comment

Go patch committed: Use some of 6g runtime.c

2011-11-14 Thread Ian Lance Taylor
This patch to the Go library uses some of the runtime.c file from the 6g compiler. This is to make it easier to merge changes to the scheduler. There are no functional changes with this patch. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r

Re: coverage, propagate visibility

2011-11-14 Thread Markus Trippelsdorf
On 2011.11.14 at 21:59 +, Nathan Sidwell wrote: This patch propagates visibility information in the same way that weakness is for externally visible coverage objects. I don't know whether this will fix the reported firefox build failure -- can you try? The patch doesn't help

[patch] Require parentheses when parsing transaction expressions.

2011-11-14 Thread Torvald Riegel
This patch fixes parsing of transaction expressions, so that two txn expressions that are part of a single expression do not get treated as nested txns anymore, but become two separate ones (see test case). This was approved off-line by Richard Henderson for the trans-mem branch and posted before

Re: [patch] Require parentheses when parsing transaction expressions.

2011-11-14 Thread Richard Henderson
On 11/14/2011 12:41 PM, Torvald Riegel wrote: Require parentheses when parsing transaction expressions. gcc/ * c-parser.c (c_parser_transaction_expression): Require parentheses when parsing transaction expressions. gcc/cp/ * parser.c

Re: Memset/memcpy patch

2011-11-14 Thread Jan Hubicka
bootstrap completed on i686-darwin9, so I've applied the following as requested, Thank you and my apologizes for the breakage! Honza Iain gcc: 2011-11-14 Jan Hubicka j...@suse.cz * config/i386/i386.c (core cost model): Correct pasto. ndex: gcc/config/i386/i386.c

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Hans-Peter Nilsson
From: Richard Henderson r...@redhat.com Date: Mon, 14 Nov 2011 18:48:03 +0100 On 11/14/2011 04:10 AM, Hans-Peter Nilsson wrote: Looks like all we need is a positive review of http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01409.html and a ChangeLog entry to unbreak three or more targets.

Re: Memset/memcpy patch

2011-11-14 Thread H.J. Lu
On Mon, Nov 14, 2011 at 9:03 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this is hopefully final variant of patch. The epilogue code was broken in some scenarios for memset, but should work safely now.  I also fixed the tables for core/buldozer/amdfam10 chips. But before it can be comitted, we

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Richard Henderson
On 11/14/2011 11:56 AM, Alan Modra wrote: On Mon, Nov 14, 2011 at 07:48:03AM -1000, Richard Henderson wrote: On 11/14/2011 04:10 AM, Hans-Peter Nilsson wrote: Looks like all we need is a positive review of http://gcc.gnu.org/ml/gcc-patches/2011-11/msg01409.html and a ChangeLog entry to

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

2011-11-14 Thread Richard Henderson
On 11/14/2011 11:46 AM, David Edelsohn wrote: On Sat, Nov 12, 2011 at 1:46 AM, Richard Henderson r...@redhat.com wrote: There are a couple of instances in which the paper doesn't cover the handling of memory_model_consume, and I made a best guess. These are indicated by /* ??? */ markers.

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread Bernd Schmidt
On 11/15/11 01:43, Richard Henderson wrote: On 11/14/2011 11:56 AM, Alan Modra wrote: * function.c (thread_prologue_and_epilogue_insns): Guard emitting return with single_succ_p test. Hmm. This looks plausible too. Bernd's patch made sure that cfglayout didn't do something

Re: CFG review needed for fix of PowerPC shrink-wrap support 3 of 3

2011-11-14 Thread David Miller
From: Bernd Schmidt ber...@codesourcery.com Date: Tue, 15 Nov 2011 01:54:34 +0100 On 11/15/11 01:43, Richard Henderson wrote: On 11/14/2011 11:56 AM, Alan Modra wrote: * function.c (thread_prologue_and_epilogue_insns): Guard emitting return with single_succ_p test. Hmm. This looks

[PATCH, debug] Emit basic block markers in .debug_line section

2011-11-14 Thread Roberto Agostino Vitillo
With this patch DW_LNS_set_basic_block opcodes are emitted in the .debug_line section marking the instructions that indicate the beginning of a basic block as specified by the dwarf standards 2,3 and 4. Tested on x86_64-linux also against gdb's 7.3.1 testsuite. r gcc/Changelog: 2011-11-14

[wwwdocs] POWER7 announcement in News section

2011-11-14 Thread Gerald Pfeifer
I figured I'd make the POWER7 donation to the GCC Compile Farm a news item. Hint to everyone else: donate, and I'll be happy to add an entry as well. :-) Installed. Gerald Index: index.html === RCS file:

Minor contrib.texi update

2011-11-14 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 OK for trunk? * doc/contrib.texi: Add entry for David Binderman. -BEGIN PGP SIGNATURE- Version: GnuPG v1.4.11 (GNU/Linux) Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org/

Re: coverage, propagate visibility

2011-11-14 Thread Nathan Sidwell
On 11/14/11 22:33, Markus Trippelsdorf wrote: On 2011.11.14 at 21:59 +, Nathan Sidwell wrote: This patch propagates visibility information in the same way that weakness is for externally visible coverage objects. I don't know whether this will fix the reported firefox build failure -- can

Re: [PATCH 0/3] Predication support for selective scheduler

2011-11-14 Thread Andrey Belevantsev
Hello, On 26.10.2011 21:11, Alexander Monakov wrote: Hello, This patch series implements predication support for the selective scheduler. It is implemented as a transformation similar to speculative execution, but simpler in some aspects (e.g. does not require recovery code). On ia64,

Re: [PATCH 1/3] Transformation replay mechanism

2011-11-14 Thread Andrey Belevantsev
Hello, On 26.10.2011 21:26, Alexander Monakov wrote: (Note, this is a prerequisite for predication support but also is an improvement on its own. This patch can be installed separately.) ... diff --git a/gcc/sel-sched.c b/gcc/sel-sched.c index f11faca..91fb0fe 100644 ---

Re: [CRIS] Hookize FUNCTION_VALUE_REGNO_P

2011-11-14 Thread Hans-Peter Nilsson
From: Anatoly Sokolov ae...@post.ru Date: Wed, 9 Nov 2011 22:42:53 +0100 Regression tested on cris-axis-elf. OK to install? * config/cris/cris.c (cris_function_value_regno_p): Make static. (TARGET_FUNCTION_VALUE_REGNO_P): Define. * config/cris/cris.h

Re: coverage, propagate visibility

2011-11-14 Thread Markus Trippelsdorf
On 2011.11.15 at 07:09 +, Nathan Sidwell wrote: On 11/14/11 22:33, Markus Trippelsdorf wrote: On 2011.11.14 at 21:59 +, Nathan Sidwell wrote: This patch propagates visibility information in the same way that weakness is for externally visible coverage objects. I don't know