Re: [Patch, AVR]: PR42240 - Fix epilogue of naked functions

2011-03-04 Thread Georg-Johann Lay
Weddington, Eric schrieb: -Original Message- From: Georg-Johann Lay [mailto:a...@gjlay.de] Sent: Thursday, March 03, 2011 11:01 AM To: Denis Chertykov Cc: gcc-patches@gcc.gnu.org; Anatoly Sokolov; Weddington, Eric Subject: Re: [Patch, AVR]: PR42240 - Fix epilogue of naked functions

Re: libiberty/cplus-dem.c, ada-demangle: plug memory leak.

2011-03-04 Thread Tristan Gingold
On Mar 3, 2011, at 10:59 PM, Michael Snyder wrote: Jakub Jelinek wrote: On Thu, Mar 03, 2011 at 01:20:28PM -0800, Michael Snyder wrote: 2011-03-03 Michael Snyder msny...@vmware.com * libiberty/cplus-dem.c (ada_demangle): Stop memory leak. Also fix a one line indent problem. No

Re: build_function_call and TREE_ADDRESSABLE

2011-03-04 Thread Richard Guenther
On Fri, Mar 4, 2011 at 5:58 AM, Alan Modra amo...@gmail.com wrote: Warn off anyone trying to do as I did, and the bit about inlines is no longer relevant.  OK to apply?        * tree.h (TREE_ADDRESSABLE): Note that direct calls set the        flag on FUNCTION_DECLs. Index: gcc/tree.h

Re: [doc, PATCH] Remove redundant word See before @xref

2011-03-04 Thread Gerald Pfeifer
On Thu, 3 Mar 2011, Mingjie Xing wrote: 2011-03-03 Mingjie Xing mingjie.x...@gmail.com * doc/cfg.texi: Remove See before @ref. * doc/invoke.texi: Likewise. This is fine, thank you. If you want to apply this to the GCC 4.5 branch as well, that's fine, too. Gerald

Re: [Patch, libfortran] PR 47802 Update documentation for CTIME and FDATE

2011-03-04 Thread Tobias Burnus
On 03/01/2011 10:26 PM, Janne Blomqvist wrote: Ok for trunk? @code{CTIME} converts a system time value, such as returned by -@code{TIME8}, to a string of the form @samp{Sat Aug 19 18:13:14 1995}. +@code{TIME8}, to a localized string. Unless the application has called +@code{setlocale}, the

Re: ivopts improvement

2011-03-04 Thread Tom de Vries
On 03/04/2011 08:37 AM, Paolo Bonzini wrote: On 03/03/2011 03:28 PM, Tom de Vries wrote: reg-tested on x86_64. Better? Yes, very much so Great. Thanks for the review. (talking about patch 6.5; the other one is an optimization but not essential based on the new comments). Just one

Re: ivopts improvement

2011-03-04 Thread Paolo Bonzini
On 03/04/2011 02:57 PM, Tom de Vries wrote: The MAX_EXPR however expands into control flow, and is not hoisted out of the outer loop, while the rest of the bound calculation is. That looks like a pass-ordering problem too (both on the tree level, for ivopts versus invariant motion, and on the

Re: [2/2] Reducing the overhead of dwarf2 location tracking

2011-03-04 Thread Richard Sandiford
Richard Guenther richard.guent...@gmail.com writes: On Fri, Mar 4, 2011 at 2:56 PM, Richard Sandiford richard.sandif...@linaro.org wrote: Suppose we have a function F that is inlined several times.  Suppose too that F has a local variable X, and that no real (as opposed to debug) references

Re: [2/2] Reducing the overhead of dwarf2 location tracking

2011-03-04 Thread Richard Guenther
On Fri, Mar 4, 2011 at 4:10 PM, Jakub Jelinek ja...@redhat.com wrote: On Fri, Mar 04, 2011 at 04:00:06PM +0100, Richard Guenther wrote: But then location lists for those variables depend on the function context and cannot be shared for different callers.  Am I missing something? Sure, they

Re: [ping] Re: [patch, powerpc] #undef LINK_EH_SPEC before defining it

2011-03-04 Thread Joseph S. Myers
On Fri, 4 Mar 2011, Alan Modra wrote: On Fri, Mar 04, 2011 at 08:38:18AM +0100, Jakub Jelinek wrote: We see it because we add --no-add-needed to linux LINK_EH_SPECs. OK. Adding the #undef is then required.. Including freebsd-spec.h in a Linux configuration is certainly very

Re: [Patch, libfortran] PR 47802 Update documentation for CTIME and FDATE

2011-03-04 Thread Janne Blomqvist
On Fri, Mar 4, 2011 at 15:56, Tobias Burnus bur...@net-b.de wrote: On 03/01/2011 10:26 PM, Janne Blomqvist wrote: Ok for trunk?  @code{CTIME} converts a system time value, such as returned by -@code{TIME8}, to a string of the form @samp{Sat Aug 19 18:13:14 1995}. +@code{TIME8}, to a

Re: libiberty/cplus-dem.c, ada-demangle: plug memory leak.

2011-03-04 Thread Tom Tromey
Michael How come 'xfree' in gdb/utils.c still checks for null? I don't know, but I assume just because nobody has bothered to remove the check. I think we also still have code doing `if (x) xfree (x);', which is kind of doubly wrong :) Tom

Re: ObjC/ObjC++: Fix property encoding on Apple

2011-03-04 Thread Nicola Pero
On 4 Mar 2011, at 16:41, Dominique Dhumieres wrote: Nicola, obj-c++.dg/property/property-encoding-1.mm fails to compile with: [...] You need to use the right C++ incantations, [...] Apologies. I now did the obvious changes that should make it work in both the ObjC and ObjC++

[Committed] S/390: Fix problems with literal pools refs

2011-03-04 Thread Andreas Krebbel
Hi, the attached patch fixes a problem with literal pool relative references. Only if these themself reference literal pool slots using these values as displacement in addresses is valid. CSE sometimes tries to pull such values out of the literal pool. This patch rejects non-literal pool

[PATCH] Avoid IPA-CP versioning if type mismatch prevents it (PR tree-optimization/47967)

2011-03-04 Thread Jakub Jelinek
Hi! If conversion between parameter type and passed argument is not useless, is not fold convertible and has different size, then even VCE is not appropriate, but for something so undefined we shouldn't be IMHO trying to optimize anything. Bootstrapped/regtested on x86_64-linux and i686-linux,

[PATCH] Teach var-tracking about some targets setmem/movmem patterns (PR debug/47991)

2011-03-04 Thread Jakub Jelinek
Hi! s390 uses (set (mem:BLK ...) (reg:DI ...)) pattern for setmem, but even i?86 uses (set (mem:BLK ...) (const_int ...)). Telling var-tracking in that case that (mem:BLK) has the (reg:DI) resp. (const_int) value is wrong and leads to mode mismatches. So, the following patch fixes it by

Re: ivopts improvement

2011-03-04 Thread Zdenek Dvorak
Hi, /* Whether the loop body includes any function calls. */ bool body_includes_call; + + /* Whether the loop body includes any function calls that possibly have side + effects. */ + bool body_includes_side_effect_call; }; /* An assignment of iv candidates to uses.

[PATCH] Fix gcc.dg/torture/pr47968.c testcase on i?86

2011-03-04 Thread Jakub Jelinek
Hi! This testcase fails on i686-linux, because of: gcc.dg/torture/pr47968.c:6:7: note: The ABI for passing parameters with 16-byte alignment has changed in GCC 4.6 gcc.dg/torture/pr47968.c:6:7: warning: SSE vector argument without SSE enabled changes the ABI [enabled by default] This patch

Re: [pph] Add preprocessor validation

2011-03-04 Thread Diego Novillo
On 03/04/2011 05:46 PM, Lawrence Crowl wrote: Add preprocessor symbol verification to PPH. Write the used symbol table instead of a dummy PPH file. Reuse PTH code for verifying these symbols are correct. Modify verification to allow a preprocessor symbol to match the PPH file's