Re: [PATCH] Handle OPTIMIZATION_NODE in LTO

2012-01-09 Thread Richard Guenther
On Thu, 5 Jan 2012, Steven Bosscher wrote: On Thu, Jan 5, 2012 at 4:01 PM, Richard Guenther rguent...@suse.de wrote: Index: gcc/tree-streamer-out.c === --- gcc/tree-streamer-out.c     (revision 182908) +++

[patch] stdbool.h should not define bool, true or false as macros for C++

2012-01-09 Thread Jonathan Wakely
GCC's implementation of stdbool.h is not valid for C++11 because [support.runtime] p8 says The header cstdbool and the header stdbool.h shall not define macros named bool, true, or false. This patch adds a libstdc++ test for that requirement and adjusts stdbool.h to meet it. I've left _Bool

Re: [PATCH, ARM] Fix stack red zone bug (PR38644) for GCC 4.6

2012-01-09 Thread Sebastian Huber
What is missing to get this ported back to the GCC 4.6 branch? Btw. what is this ARM/embedded-4_6-branch for a branch? It seems to be more actively maintained. On 12/21/2011 09:10 AM, Sebastian Huber wrote: Would someone mind committing this to the 4.6 branch. Thanks. The test suite results

Re: [PATCH]: Add static branch predict heuristic of comparing IV to loop_bound variable

2012-01-09 Thread Dehao Chen
Hello, Attached is the updated version of the patch. Passed the bootstrap and gcc testsuite. Thanks, Dehao Index: gcc/testsuite/gcc.dg/predict-3.c === --- gcc/testsuite/gcc.dg/predict-3.c(revision 0) +++

Re: [PING] PR33919/preprocessor fix __BASE_FILE__ when included from the command line

2012-01-09 Thread Gary Funck
On 01/06/12 08:34:52, Tom Tromey wrote: The patch is ok with either that change or with those 2 lines removed. Applied as svn version 183003.

[PATCH] Adjust 'malloc' attribute documentation to match implementation

2012-01-09 Thread Richard Guenther
Since GCC 4.4 applying the malloc attribute to realloc-like functions does not work under the documented constraints because the contents of the memory pointed to are not properly transfered from the realloc argument (or treated as pointing to anything, like 4.3 behaved). The following adjusts

GCC 4.7.0 Status Report (2012-01-09), Stage 3 ends now

2012-01-09 Thread Richard Guenther
Status == Stage 3 is now officially over, after a bit more than two months. The GCC trunk is now in regression and documentation fixes only state (so-called stage4). The trunk will remain in this state until it is sufficiently stable for release. At this point we will create the 4.7

Re: patch: remove is_gimple_non_addressable

2012-01-09 Thread Richard Guenther
On Fri, Jan 6, 2012 at 6:05 PM, Aldy Hernandez al...@redhat.com wrote: On 01/05/12 09:36, Richard Guenther wrote: On Thu, Jan 5, 2012 at 3:48 PM, Aldy Hernandezal...@redhat.com  wrote: As you suggested here: http://gcc.gnu.org/ml/gcc-patches/2012-01/msg00232.html Bootregged on x86-64

Re: [patch] Fix PR tree-optimization/51624

2012-01-09 Thread Richard Guenther
On Fri, Jan 6, 2012 at 7:17 PM, Eric Botcazou ebotca...@adacore.com wrote: OK, I see.  Then the only way out I can think of is to stop going up the chain of COMPONENT_REFs as soon as the offset becomes negative. Yeah, that sounds like a better solution. PR ada/51775 shows that this

Re: Support Solaris 11/SPARC in MD_FALLBACK_FRAME_STATE_FOR (PR ada/41929)

2012-01-09 Thread Eric Botcazou
All EH-related tests fail with a recent upgrade on Solaris 10. Interestingly, the old implementation keeps working with the upgrade. I've applied the attached patch to mainline and 4.6 branch. It reverts back to the old pattern matching code in the Solaris 8+ multi-threaded case, which is

Re: PR tree-optimization/51680 (not inlining comdats)

2012-01-09 Thread Richard Guenther
On Sun, Jan 8, 2012 at 12:38 AM, Jan Hubicka hubi...@ucw.cz wrote: Hi, this patch fixes first half of the PR.  The PR is about a function that is supposed to be inlined at -O2.  The function contains two calls, one indirect and inlining the indirect call makes things a lot better. GCC

Re: [patch] stdbool.h should not define bool, true or false as macros for C++

2012-01-09 Thread Jonathan Wakely
I've just seen Richard's status email, so I guess this should wait for 4.8 On 9 January 2012 08:48, Jonathan Wakely wrote: GCC's implementation of stdbool.h is not valid for C++11 because [support.runtime] p8 says The header cstdbool and the header stdbool.h shall not define macros named bool,

Re: [Patch, Fortran] PR51578 Fix symbol import with renaming and only

2012-01-09 Thread Tobias Burnus
Dear all, this email contains a updated patch, which replaces the FIXME by an implementation. See below. Dear Dominique, On 01/08/2012 11:13 PM, Dominique Dhumieres wrote: Your patch passed my tests and in top of that fixes the ICE for pr51522.f90. :-) However I don't understand the

[Patch, Fortran] PR 51758 - fix ICE with null() in elemental procedures

2012-01-09 Thread Tobias Burnus
The issue was discovered when looking at the optional + elemental + scalarizer issue (PR 50981, 4.4-4.7 regression). However, the example of this PR never worked. Passing null() to denote an absent argument (for nonallocatable/nonpointer dummies) is a Fortran 2008 / GCC 4.6 feature. Build and

Re: [patch] Fix PR tree-optimization/51624

2012-01-09 Thread Eric Botcazou
Can you get me a testcase that I can compile? This should be fixed in FRE. This is gnat.dg/pack9.ad[sb]. You need a bare cross-compiler to s390x-linux: configure, build with make CFLAGS=-g, when the build fails, do gcc/ada then make gnatlib. Go back to the build dir, copy

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-01-09 Thread Dave Korn
On 17/12/2011 11:07, Eric Botcazou wrote: I am happily using this patch and await this to be committed. Only after the import library issue is addressed. What do the other libraries bundled with GCC do here? Sorry for the delay guys, I got rather busy over the holidays. I see we're

Re: PR tree-optimization/51680 (not inlining comdats)

2012-01-09 Thread Jan Hubicka
That surely should be MAX_INLINE_INSNS_AUTO instead. At _least_. You are triggering very much more inlining during early inlining now - I don't see where we did this for -Os already as you claim. In fact it is totally against the spirit of early inlining now :( This is IPA inlining, not

Re: [PING] PR33919/preprocessor fix __BASE_FILE__ when included from the command line

2012-01-09 Thread Tobias Burnus
On 01/09/2012 10:02 AM, Gary Funck wrote: On 01/06/12 08:34:52, Tom Tromey wrote: The patch is ok with either that change or with those 2 lines removed. Applied as svn version 183003. I get the build failure: libcpp/macro.c:220:26: error: variable 'map' set but not used

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-01-09 Thread Dave Korn
On 17/12/2011 10:06, Christian Joensson wrote: I can add that I also, manually, copy the lto-plugin/.libs/cyglto_plugin-0.dll into $bindir too. Why? It deliberately installs into libexecsubdir rather than bindir because it's only ever invoked by the gcc driver, which passes the full

Re: [patch] Fix PR tree-optimization/51624

2012-01-09 Thread Richard Guenther
On Mon, Jan 9, 2012 at 11:33 AM, Eric Botcazou ebotca...@adacore.com wrote: Can you get me a testcase that I can compile?  This should be fixed in FRE. This is gnat.dg/pack9.ad[sb].  You need a bare cross-compiler to s390x-linux: configure, build with make CFLAGS=-g, when the build fails, do

Re: PR tree-optimization/51680 (not inlining comdats)

2012-01-09 Thread Richard Guenther
On Mon, Jan 9, 2012 at 12:07 PM, Jan Hubicka hubi...@ucw.cz wrote: That surely should be MAX_INLINE_INSNS_AUTO instead.  At _least_. You are triggering very much more inlining during early inlining now - I don't see where we did this for -Os already as you claim.  In fact it is totally

Re: [patch] Fix PR tree-optimization/51624

2012-01-09 Thread Eric Botcazou
Hmm, it seems to be because we do not value-number loads that satisfy stmt_could_throw_p (for whatever reason ...). Seems to date back to rev. 131610, the fix for PR34648. Looking at that bug it seems that we could at least allow stmts that only throw externally (but generally CSE should

Re: [patch] Fix cygwin ada install [was Re: Yet another issue with gcc current trunk with ada on cygwin]

2012-01-09 Thread Eric Botcazou
Sorry for the delay guys, I got rather busy over the holidays. I see we're now discussing a patch for next stage 1. No, not necessarily, the patch is specific to Ada on Windows so the risk is quite low. It does however solve the problem of wanting the DLL to be in the /bin directory,

[RFC/ARM] Prefer branches over conditional execution for Cortex-A15.

2012-01-09 Thread Matthew Gretton-Dann
The attached patch tunes the costs of conditional execution and branches for Cortex-A15 to be the same as that for Cortex-A5. This gives an average improvement of over 6% on a popular embedded benchmark. Note that the tuning parameter structure added for Cortex-A15 is only tuned for branch

Re: PR tree-optimization/51680 (not inlining comdats)

2012-01-09 Thread Jan Hubicka
On Mon, Jan 9, 2012 at 12:07 PM, Jan Hubicka hubi...@ucw.cz wrote: That surely should be MAX_INLINE_INSNS_AUTO instead.  At _least_. You are triggering very much more inlining during early inlining now - I don't see where we did this for -Os already as you claim.  In fact it is totally

Re: [patch] Fix PR tree-optimization/51624

2012-01-09 Thread Richard Guenther
On Mon, Jan 9, 2012 at 12:45 PM, Eric Botcazou ebotca...@adacore.com wrote: Hmm, it seems to be because we do not value-number loads that satisfy stmt_could_throw_p (for whatever reason ...).  Seems to date back to rev. 131610, the fix for PR34648.  Looking at that bug it seems that we could

Re: increase call_saved_regs[] in caller-save.c

2012-01-09 Thread Michael Matz
Hi, On Sat, 7 Jan 2012, Peter Bergner wrote: While digging through ira-color.c tracking down an IRA shuffle copy issue, I noticed we only seem to do real copy coalescing for spilled pseudos. It seems we rely on coloring to try and assign the same hard reg to pseudos connected by a copy so

Re: RFC: An alternative -fsched-pressure implementation

2012-01-09 Thread Bernd Schmidt
On 12/23/2011 12:46 PM, Richard Sandiford wrote: In the end I tried an ad-hoc approach in an attempt to do something about (2), (3) and (4b). The idea was to construct a preliminary model schedule in which the only objective is to keep register pressure to a minimum. This schedule ignores

Ping^2: Re: [patch libstdc++]: Fix PR/51673

2012-01-09 Thread Kai Tietz
Ping 2012/1/5 Kai Tietz ktiet...@googlemail.com: Ping! 2012/1/2 Paolo Carlini paolo.carl...@oracle.com: Hi, Hello, additionally to the suggested patch by Pawel Sikora, I added the adjustments for mt-allocator to it too. ChangeLog 2012-01-01  Kai Tietzkti...@redhat.com         PR

Re: [PING] PR33919/preprocessor fix __BASE_FILE__ when included from the command line

2012-01-09 Thread Dominique Dhumieres
I get the build failure: ... I got the same failure. The following patch --- ../_clean/libcpp/macro.c2012-01-09 11:15:22.0 +0100 +++ ../work/libcpp/macro.c 2012-01-09 12:28:06.0 +0100 @@ -217,7 +217,7 @@ static const char * const monthnames[] = const uchar *

Re: [RFC/ARM] Prefer branches over conditional execution for Cortex-A15.

2012-01-09 Thread Richard Earnshaw
On 09/01/12 12:01, Matthew Gretton-Dann wrote: The attached patch tunes the costs of conditional execution and branches for Cortex-A15 to be the same as that for Cortex-A5. This gives an average improvement of over 6% on a popular embedded benchmark. Note that the tuning parameter

[PATCH] Fix PR51775

2012-01-09 Thread Richard Guenther
This fixes PR51775, we do not need to avoid value-numbering for stmts that can throw. PRE already ensures to not enter anything into the EXP/TMP_GEN sets for those so it won't ever end up trying to insert throwing expressions. Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.

Re: [PING] PR33919/preprocessor fix __BASE_FILE__ when included from the command line

2012-01-09 Thread Gary Funck
On 01/09/12 12:08:22, Tobias Burnus wrote: I get the build failure: libcpp/macro.c:220:26: error: variable 'map' set but not used [-Werror=unused-but-set-variable] Tobias, thanks. Will fix. - Gary

[PATCH] Fix bootstrap

2012-01-09 Thread Richard Guenther
Committed as obvious. Richard. 2012-01-09 Richard Guenther rguent...@suse.de libcpp/ * macro.c (_cpp_builtin_macro_text): Remove unused variable map. Index: macro.c === --- macro.c (revision 183012) +++

Re: [Patch, Fortran] PR 51758 - fix ICE with null() in elemental procedures

2012-01-09 Thread Mikael Morin
Sorry Tobias, I was about to post a patch about this when I saw your message. The issue is that the code handling NULL() doesn't consume the gfc_ss struct created for it. Your fix, which advances to the next one anyway would work just well, but I think it is slightly cleaner to not create

[Patch, Fortran] gfortran.texi: Update (C) year and F2003 status

2012-01-09 Thread Tobias Burnus
Ok for the trunk? (Build + make pdf tested) Tobias 2012-01-09 Tobias Burnus bur...@net-b.de * gfortran.texi: Bump copyright year. (Fortran 2003 Status): Update polymorphism item, add item for generic interface with DT name. diff --git a/gcc/fortran/gfortran.texi

Re: [PING] PR33919/preprocessor fix __BASE_FILE__ when included from the command line

2012-01-09 Thread Gary Funck
On 01/09/12 06:04:28, Gary Funck wrote: On 01/09/12 12:08:22, Tobias Burnus wrote: I get the build failure: libcpp/macro.c:220:26: error: variable 'map' set but not used [-Werror=unused-but-set-variable] Richard, thanks. 2012-01-09 Richard Guenther rguent...@suse.de libcpp/

Re: [Patch, Fortran] PR 51758 - fix ICE with null() in elemental procedures

2012-01-09 Thread Tobias Burnus
On 01/09/2012 03:34 PM, Mikael Morin wrote: The issue is that the code handling NULL() doesn't consume the gfc_ss struct created for it. Your fix, which advances to the next one anyway would work just well, but I think it is slightly cleaner to not create the struct in the first place, as it

Re: patch: remove is_gimple_non_addressable

2012-01-09 Thread Aldy Hernandez
Ok with simply excluding SSA_NAMEs like @@ -4411,16 +4411,27 @@ gimplify_modify_expr_rhs (tree *expr_p, /* It's OK to use the target directly if it's being initialized. */ use_target = true; - else if (TREE_CODE (*to_p) != SSA_NAME

libiberty: Fix make pdf for copying-lib.texi

2012-01-09 Thread Tobias Burnus
make pdf fails with: libiberty/copying-lib.texi:481: This command can appear only outside of any environment, not in environment @enumerate. @badenverr ...temp , not @inenvironment @thisenv } @checkenv ...@ifx @thisenv @temp @else @badenverr

Re: Find more shrink-wrapping opportunities

2012-01-09 Thread Bernd Schmidt
On 12/04/2011 01:27 PM, Richard Sandiford wrote: OK, here it is. As well as switching to the backward scan and incremental liveness updates, I added a test for another case that I stumbled over: /* Reject targets of abnormal edges. This is needed for correctness on ports like Alpha

[PATCH, AVR] Fix PR target/50925, use hard_frame_pointer_rtx

2012-01-09 Thread Denis Chertykov
Hi Georg. I have found that conversion AVR port to using hard_frame_pointer have resolved PR 50925 . I have tested the patch without regressions, but I'm worry about it. Can you test it with your testsuite for regressions ? May be you have your own special difficult tests (special for addressing)

[PATCH][Graphite] Fix PR50561

2012-01-09 Thread Richard Guenther
We have two Graphite related release blockers for 4.7. The following patch fixes the first one. The asserts in static inline ppl_dimension_type psct_dynamic_dim (poly_bb_p pbb, graphite_dim_t level) { graphite_dim_t result = 1 + 2 * level; gcc_assert (result pbb_nb_scattering_transform

Re: [Patch, Fortran] gfortran.texi: Update (C) year and F2003 status

2012-01-09 Thread Steve Kargl
On Mon, Jan 09, 2012 at 03:38:21PM +0100, Tobias Burnus wrote: Ok for the trunk? (Build + make pdf tested) Tobias The two which clauses in the 1st sentence seem awkward. How about the following changes? OK, after considering the suggested changes. +@item Generic interface name which have

[PATCH][Graphite]

2012-01-09 Thread Richard Guenther
This fixes the 2nd P1 ICE. There is a disconnect on how we analyze data-references during SCOP detection (outermost_loop is the root of the loop tree) and during SESE-to-poly where outermost is determined by outermost_loop_in_sese_1 (). That influences the SCEV result and thus we do not break

Re: patch: remove is_gimple_non_addressable

2012-01-09 Thread Richard Guenther
On Mon, Jan 9, 2012 at 3:57 PM, Aldy Hernandez al...@redhat.com wrote: Ok with simply excluding SSA_NAMEs like @@ -4411,16 +4411,27 @@ gimplify_modify_expr_rhs (tree *expr_p,                /* It's OK to use the target directly if it's being                   initialized. */                

Re: Unreviewed libffi, libgfortran, libitm patches

2012-01-09 Thread Rainer Orth
Richard Henderson r...@redhat.com writes: On 01/04/2012 11:27 PM, Rainer Orth wrote: [libgfortran, libitm] Link with -shared-libgcc http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01382.html This will need a fortran resp. libitm maintainer. Does the following alleviate the need for

Unreviewed^3 libffi patch

2012-01-09 Thread Rainer Orth
The following patch has remained unreviewed for almost a month now, despite two reminders: [libffi] Build 64-bit multilib for i?86-linux http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01011.html In the absence of a listed libffi maintainer (Anthony Green used to be listed in

Re: Unreviewed^3 libffi patch

2012-01-09 Thread Andrew Haley
On 01/09/2012 04:09 PM, Rainer Orth wrote: In the absence of a listed libffi maintainer (Anthony Green used to be listed in MAINTAINERS, but isn't anymore despite still taking care of upstream libffi), it probably needs a global reviewer. Sorry, I usually do it but missed this one. It's fine.

[build] Check if linker supports R_386_TLS_GD_PLT, R_386_TLS_LDM_PLT relocs

2012-01-09 Thread Rainer Orth
While the combination of Sun as with GNU ld isn't a recommended configuration on Solaris, it used to work half a year ago, but doesn't right now: stage 1 libgomp fails to link: /vol/gcc/bin/gld-2.22: .libs/team.o: unrecognized relocation (0xc) in section `.text' /vol/gcc/bin/gld-2.22: final link

Re: [build] Check if linker supports R_386_TLS_GD_PLT, R_386_TLS_LDM_PLT relocs

2012-01-09 Thread Paolo Bonzini
On 01/09/2012 05:14 PM, Rainer Orth wrote: While the combination of Sun as with GNU ld isn't a recommended configuration on Solaris, it used to work half a year ago, but doesn't right now: stage 1 libgomp fails to link: /vol/gcc/bin/gld-2.22: .libs/team.o: unrecognized relocation (0xc) in

Re: [RFC][libitm] Convert to c++11 atomics

2012-01-09 Thread Andrew MacLeod
On 01/02/2012 01:19 PM, Torvald Riegel wrote: On Tue, 2011-12-20 at 01:13 +0100, Torvald Riegel wrote: On Mon, 2011-12-19 at 15:17 -0800, Richard Henderson wrote: On 12/19/2011 02:58 PM, Torvald Riegel wrote: In the particular case (the validated loads technique used in method-gl.cc, load(),

Re: [PATCH, ARM] Fix stack red zone bug (PR38644) for GCC 4.6

2012-01-09 Thread Ramana Radhakrishnan
On 9 January 2012 08:49, Sebastian Huber sebastian.hu...@embedded-brains.de wrote: What is missing to get this ported back to the GCC 4.6 branch? I ran a sanity check again today and backported them to the 4.6 branch. Sorry about the delay. I prefer to do the same for the 4.4 and 4.5 branches.

Re: [PATCH, AVR] Fix PR target/50925, use hard_frame_pointer_rtx

2012-01-09 Thread Georg-Johann Lay
Denis Chertykov wrote: Hi Georg. I have found that conversion AVR port to using hard_frame_pointer have resolved PR 50925 . I have tested the patch without regressions, but I'm worry about it. Can you test it with your testsuite for regressions ? May be you have your own special difficult

Ping [ARM back-end and middle-end patch] stack check for threads

2012-01-09 Thread Thomas Klein
Hello Even if I'm going to bore everyone. Is it possible to get an answer from the ARM maintainers? This is a serious question, and I hope there will be a serious answer. Regards Thomas Klein reference: http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01774.html

[PATCH, i386]: Optimize AND with 0xffffffff

2012-01-09 Thread Uros Bizjak
Hello! Attached patch fixes oversight with AND pattern and 0x immediate. While ANDs with 0xff and 0x are converted to equivalent zero_extend pattern, AND with 0x isn't. This problem leaves important optimization that would substitute movq%rdi, %rax; andl $4294967295,

[SPARC] Small tweak to Solaris unwinder

2012-01-09 Thread Eric Botcazou
On the recent Solaris 10 version I have access to (s10_72), the kernel/libc sometimes calls the call_user_handler routines with a null returned address. This is responsible for the last ACATS failure (cb1010c). But, in most cases, the address is the expected one, so it isn't clear if this is a

Re: [libitm] Port to ARM

2012-01-09 Thread Eric Botcazou
To get the ball rolling for other targets, and to let port maintainers see how easy it really is, here's a first cut at a port to ARM. Do you plan to do something for the SPARC? I'm asking because you still are a maintainer of this arch as well, so if we can avoid doing the work twice... --

Re: [Patch, Fortran] PR 51758 - fix ICE with null() in elemental procedures

2012-01-09 Thread Mikael Morin
On 09.01.2012 15:45, Tobias Burnus wrote: On 01/09/2012 03:34 PM, Mikael Morin wrote: The issue is that the code handling NULL() doesn't consume the gfc_ss struct created for it. Your fix, which advances to the next one anyway would work just well, but I think it is slightly cleaner to not

Re: [PATCH] [MIPS] Add Octeon2 cpu support to GCC

2012-01-09 Thread Richard Sandiford
Gerald Pfeifer ger...@pfeifer.com writes: On Wed, 4 Jan 2012, Andrew Pinski wrote: Woops, I had missed that. Here is the patch which adds both octeon+ and octeon2 to that list. I committed as obvious after a build. You may want to update the release notes as well

[Patch, Fortran] PR51197 - print signal number before backtrace [RFC]

2012-01-09 Thread Tobias Burnus
Hello, in GCC 4.6, gfortran printed before the backtrace, e.g., Program received signal 8 (SIGFPE): Floating-point exception. to STDERR followed by the backtrace. This was done by registering an error handler and - finishing with a sys_exit(5). Since 4.7 with the patch for PR 48915 (comment

Re: [PING] [wwwdocs] - changes to GUPC page

2012-01-09 Thread Nenad Vukicevic
Is it ok to make this update to the web page? Someone has to approve this change, even though it is GUPC specific. Nenad ? A Index: htdocs/projects/gupc.html === RCS file: /cvs/gcc/wwwdocs/htdocs/projects/gupc.html,v retrieving

[Ada] Fix old thinko

2012-01-09 Thread Eric Botcazou
This very likely has no consequence in release mode, but you can get ICES with tree checking enabled in some perculiar cases and on some platforms. Tested on i586-suse-linux, applied on the mainline, 4.6 and 4.5 branches. 2012-01-09 Eric Botcazou ebotca...@adacore.com *

Re: [SPARC] Small tweak to Solaris unwinder

2012-01-09 Thread Rainer Orth
Eric Botcazou ebotca...@adacore.com writes: On the recent Solaris 10 version I have access to (s10_72), the kernel/libc s10_72 is anything but recent: this is a Solaris 10 Express/Beta build which even predates Solaris 10 FCS (s10_74L2a), so this is ancient history by now. I suggest not

Re: [Patch, Fortran] PR51197 - print signal number before backtrace [RFC]

2012-01-09 Thread Steve Kargl
On Mon, Jan 09, 2012 at 08:21:31PM +0100, Tobias Burnus wrote: With the patch, one can get (cf. PR) an output to STDERR like: Program received signal 8 (SIGFPE): Floating-point exception. Backtrace for this error: #0 0x805891F in _gfortrani_show_backtrace at backtrace.c:261 #1

Re: [PATCH] PR51471. Don't speculate frame-related insns during delayed branch scheduling.

2012-01-09 Thread Richard Sandiford
Tom de Vries tom_devr...@mentor.com writes: 2012-01-08 Tom de Vries t...@codesourcery.com Andrew Pinski apin...@cavium.com * reorg.c (fill_slots_from_thread): Don't speculate frame-related insns. OK, thanks. Richard

Re: [Patch, fortran] PR51791 - [OOP] Failure to resolve typebound function call with base object in parentheses

2012-01-09 Thread Paul Richard Thomas
Dear All, Committed as r183032. Thanks Paul On Sun, Jan 8, 2012 at 10:32 PM, Tobias Burnus bur...@net-b.de wrote: Dear Paul, Paul Richard Thomas: A question for the standard aficianados: Are there other base object expressions that are legal? I don't think so. (Ignoring RESHAPE,

Re: [Patch, Fortran] PR51197 - print signal number before backtrace [RFC]

2012-01-09 Thread Janne Blomqvist
On Mon, Jan 9, 2012 at 21:21, Tobias Burnus bur...@net-b.de wrote: Hello, in GCC 4.6, gfortran printed before the backtrace, e.g.,  Program received signal 8 (SIGFPE): Floating-point exception. to STDERR followed by the backtrace. This was done by registering an error handler and -

[Patch,AVR]: ad PR49868: Extend __pgmx semantics to flatten memory

2012-01-09 Thread Georg-Johann Lay
This patch extends the semantics of 24-bit __pgmx address space qualifier to cover RAM and Flash. RAM is represented by setting the high byte of 24-bit address to 0x80. The code to read from 24-bit address space decides at runtime what instruction to use to read by if-else decision depending on

[Ada] Fix ICE on function call returning dynamic array

2012-01-09 Thread Eric Botcazou
This is a regression present on the mainline. It occurs only when the return value is assigned to an array with fixed size (this is possible with subtypes of the same unconstrained array type). Tested on i586-suse-linux, applied on the mainline. 2012-01-09 Eric Botcazou

[PATCH] Fix some powerpc debug options

2012-01-09 Thread Michael Meissner
I was recently trying to turn on some of the debug options I put in when doing the initial power7 work to track down some performance regressions. The following patches fix problems that come up in telling the compiler to use the traditional floating point insns for scalar mode on power7

[v3] add -Wabi to build flags

2012-01-09 Thread Benjamin Kosnik
As requested by Jason, I've added -Wabi to the libstdc++ build flags. The patch is trivial. The results are in line with what we expect. After I did this part and verified the build went swimmingly, I ran the testsuite with -Wabi as well. This part had some problems, I'm getting some odd results

[google/main] Fix erroneous output of pubnames when no .debug_section is emitted. (issue5528054)

2012-01-09 Thread Sterling Augustine
Certain source files will not produce a .debug_info section. This patch to google/main adds a check for that case to output_pubnames, preventing a link failure. Tested: By rebuild and bootstrap. ChangeLog: 2012-01-09 Sterling Augustine saugust...@google.com * gcc/dwarf2out.c

Re: [4.7][google] Adding a new option -fstack-protector-strong. (issue5461043)

2012-01-09 Thread 沈涵
Hi, all, it's been a long time, I've slightly modified my code - TREE_ADDRESSABLE is only applied when TREE_CODE is VAR_DECL, and it's combined with test for arrays/struct-union-contain-array, which makes the code a little bit cleaner. Comments, approvals? Thanks! = Patch starts == diff

Re: [PATCH][Graphite] Fix PR50561

2012-01-09 Thread Tobias Grosser
On 01/09/2012 04:31 PM, Richard Guenther wrote: We have two Graphite related release blockers for 4.7. The following patch fixes the first one. Hi Richard, thanks a lot for working on this. The asserts in static inline ppl_dimension_type psct_dynamic_dim (poly_bb_p pbb, graphite_dim_t

Re: [Patch, Fortran] PR51197 - print signal number before backtrace [RFC]

2012-01-09 Thread Tobias Burnus
Dear Janne, - The integer values for the signal numbers are not standardized, hence printing them might give the user a false impression that these numbers convey some information beyond whichever SIG* macro they map to on that particular target. - It doesn't test all the signals which are

[wwwdocs] correct libstdc++ links

2012-01-09 Thread Benjamin Kosnik
Correct libstdc++ naming convention to be the same as used by the other html tarballs, ie library-html.tar.bz2. I just checked this in as I was fixing the recent gcc mailing list report of broken links. -benjamin2012-01-09 Benjamin Kosnik b...@redhat.com * onlinedocs/index.html: Use

Re: [PATCH][Graphite]

2012-01-09 Thread Tobias Grosser
On 01/09/2012 04:34 PM, Richard Guenther wrote: This fixes the 2nd P1 ICE. There is a disconnect on how we analyze data-references during SCOP detection (outermost_loop is the root of the loop tree) and during SESE-to-poly where outermost is determined by outermost_loop_in_sese_1 (). That

Re: [google/main] Fix erroneous output of pubnames when no .debug_section is emitted. (issue5528054)

2012-01-09 Thread Cary Coutant
Certain source files will not produce a .debug_info section. This patch to google/main adds a check for that case to output_pubnames, preventing a link failure. Tested:        By rebuild and bootstrap. ChangeLog: 2012-01-09   Sterling Augustine  saugust...@google.com        *

Re: [PING] [wwwdocs] - changes to GUPC page

2012-01-09 Thread Gerald Pfeifer
On Mon, 9 Jan 2012, Nenad Vukicevic wrote: Is it ok to make this update to the web page? Someone has to approve this change, even though it is GUPC specific. Yes, and no. ;-) This patch is okay, and I suggest you consider yourself maintainer of this page and change it as you see fit (just

[Patch, Fortran, committed] PR 46328 - add another test case

2012-01-09 Thread Tobias Burnus
Dear all, I have committed that attached test case as Rev. 183039. The test case is the one of PR 46328, comment 2. The failure was solved by Paul's patch for PR 51791. Tobias Index: gcc/testsuite/gfortran.dg/typebound_operator_11.f90

Re: [patch libstdc++]: Fix PR/51673

2012-01-09 Thread Benjamin Kosnik
This is ok, but I realized that some of the newer CXXABI symbols were missing, so I added them as well and then checked all of this in. See attached patch. best, Benjamin 2012-01-09 Kai Tietz kti...@redhat.com PR libstc++/51673 part 2 * config/abi/pre/gnu-versioned-namespace.ver: Adjusted

Re: [PATCH] Fix some powerpc debug options

2012-01-09 Thread David Edelsohn
On Mon, Jan 9, 2012 at 4:30 PM, Michael Meissner meiss...@linux.vnet.ibm.com wrote: I was recently trying to turn on some of the debug options I put in when doing the initial power7 work to track down some performance regressions.  The following patches fix problems that come up in telling the

Re: [libitm] Port to ARM

2012-01-09 Thread Richard Henderson
On 01/10/2012 05:52 AM, Eric Botcazou wrote: To get the ball rolling for other targets, and to let port maintainers see how easy it really is, here's a first cut at a port to ARM. Do you plan to do something for the SPARC? I'm asking because you still are a maintainer of this arch as

Re: [PATCH] PR51271. Fix handling of frame-related insn in delay slot of annulled branch

2012-01-09 Thread Richard Henderson
On 01/08/2012 11:50 PM, Tom de Vries wrote: * dwarf2cfi.c (scan_trace): Save and restore cur_row-reg_save when handling annulled branch. Ok. r~

Re: Unreviewed libffi, libgfortran, libitm patches

2012-01-09 Thread Richard Henderson
On 01/10/2012 03:06 AM, Rainer Orth wrote: Richard Henderson r...@redhat.com writes: On 01/04/2012 11:27 PM, Rainer Orth wrote: [libgfortran, libitm] Link with -shared-libgcc http://gcc.gnu.org/ml/gcc-patches/2011-12/msg01382.html This will need a fortran resp. libitm maintainer.

Re: [RFC, patch] libitm: Filter out undo writes that overlap with the libitm stack.

2012-01-09 Thread Richard Henderson
On 01/09/2012 06:26 AM, Torvald Riegel wrote: libitm: Filter out undo writes that overlap with the libitm stack. libitm/ * config/generic/tls.h (GTM::mask_stack_top, GTM::mask_stack_bottom): New. * local.cc (gtm_undolog::rollback): Filter out any updates that

Re: patch ping^2: PR 51516 (TM clone aliases)

2012-01-09 Thread Richard Henderson
On 01/07/2012 05:47 AM, Aldy Hernandez wrote: Patrick Marlier patrick.marl...@gmail.com PR middle-end/51516 * trans-mem.c (get_cg_data): Traverse aliases if requested. (ipa_tm_scan_calls_block): Update parameters to get_cg_data. (ipa_tm_note_irrevocable):

[Patch, Fortran] Minor wording changed for gfc_error and polym. (intr.) assignment

2012-01-09 Thread Tobias Burnus
Hi all, that's a follow up to my review comment for http://gcc.gnu.org/ml/fortran/2012-01/msg00077.html As stated, I think in intrinsic assignment makes the error message clearer and correcter. Build and regtested on x86-64-linux. OK for the trunk? Tobias PS: Other pending patches: *

Re: [libitm] Port to ARM

2012-01-09 Thread Eric Botcazou
I have no immediate plans. If you have time to knock something together for Sparc, that would be fantastic. OK, will try, thanks. -- Eric Botcazou

Re: [Patch,AVR]: ad PR49868: Extend __pgmx semantics to flatten memory

2012-01-09 Thread Denis Chertykov
2012/1/10 Georg-Johann Lay a...@gjlay.de: This patch extends the semantics of 24-bit __pgmx address space qualifier to cover RAM and Flash. RAM is represented by setting the high byte of 24-bit address to 0x80. The code to read from 24-bit address space decides at runtime what instruction