Re: [pph] Remove all the -fpph flags and pph_catch_... calls that instrumented the incr compiler (issue4559070)

2011-06-02 Thread Gabriel Charette
(sorry about the triple-posting, just learnt this mailing list required plain text emails... hopefully, this works now..) Additional note: The patch was tested with a full bootstrap build and regression testing (no new bugs introduced). There might be header files that are now unused due to the c

Re: libquadmath with older versions of gcc

2011-06-02 Thread Basile Starynkevitch
On Thu, 2 Jun 2011 14:44:05 -0700 (PDT) Syed Bilal Mehdi wrote: [...] > I am using old version of Matlab that requires me to use gcc-4.1 (or > gcc-4.2.0 as the latest). [...] Do you understand why? My naive opinion is that, for C code, you could use a latest gcc, even 4.6. (for C++ it really cou

Re: PING: PATCH: PR target/46770: Use .init_array/.fini_array sections

2011-06-02 Thread H.J. Lu
On Wed, May 18, 2011 at 8:57 AM, H.J. Lu wrote: > On Tue, Apr 26, 2011 at 6:05 AM, H.J. Lu wrote: >> On Thu, Mar 31, 2011 at 7:57 AM, H.J. Lu wrote: >>> On Mon, Mar 21, 2011 at 11:40 AM, H.J. Lu wrote: On Mon, Mar 14, 2011 at 12:28 PM, H.J. Lu wrote: > On Thu, Jan 27, 2011 at 2:40 AM,

Re: [PR 48333] avoid -fcompare-debug errors from builtins in MEM attrs

2011-06-02 Thread Alexandre Oliva
Ping? This fixes a case in which -g might change the executable code, exposed with bootstrap-debug-lean. On Apr 2, 2011, Alexandre Oliva wrote: > PR debug/48333 > * calls.c (emit_call_1): Prefer the __builtin declaration of > builtin functions. http://gcc.gnu.org/ml/gcc-patc

Re: [patch] add -Wdelete-non-virtual-dtor

2011-06-02 Thread Jason Merrill
On 06/02/2011 06:02 PM, Jonathan Wakely wrote: + if (!dtor || !DECL_VINDEX (dtor)) Do we really want to warn about the case where the class has no/trivial destructor? + bool abstract = false; + for (x = TYPE_METHODS (type); x; x = DECL_CHAIN (x))

Re: introduce --param max-vartrack-expr-depth

2011-06-02 Thread Alexandre Oliva
On Jun 2, 2011, Bernd Schmidt wrote: > On 06/02/2011 10:46 AM, Jakub Jelinek wrote: >> On Wed, Jun 01, 2011 at 07:25:39PM -0300, Alexandre Oliva wrote: >>> Such as this one... >> >> I'd appreciate if this could go in... > Go on then. Ok, here's what I've just installed. for gcc/ChangeLog fr

Re: [PR48866] three alternative fixes

2011-06-02 Thread Alexandre Oliva
Ugh, failed to refresh the patch file, resending with the correct one. On May 30, 2011, Alexandre Oliva wrote: > On May 30, 2011, Alexandre Oliva wrote: >> 2. emit placeholders for replaceable DEFs and, when the DEFs are >> expanded at their point of use, emit the expansion next to the >> place

Re: [PR48866] three alternative fixes

2011-06-02 Thread Alexandre Oliva
On May 30, 2011, Alexandre Oliva wrote: > On May 30, 2011, Alexandre Oliva wrote: >> 3. expand dominators before dominated blocks, so that DEFs of >> replaceable SSA names are expanded before their uses. Expand them when >> they're encountered, but not requiring a REG as a result. Save the RTL

Re: [PR48866] three alternative fixes

2011-06-02 Thread Alexandre Oliva
On May 30, 2011, Alexandre Oliva wrote: > On May 30, 2011, Alexandre Oliva wrote: >> 2. emit placeholders for replaceable DEFs and, when the DEFs are >> expanded at their point of use, emit the expansion next to the >> placeholder, rather than at the current stream. The result of the >> expansi

Re: [PR48866] three alternative fixes

2011-06-02 Thread Alexandre Oliva
On May 30, 2011, Alexandre Oliva wrote: > On May 30, 2011, Alexandre Oliva wrote: >> 1. emit debug temps for replaceable DEFs that end up being referenced in >> debug insns. We already have some code to try to deal with this, but it >> emits the huge expressions we'd rather avoid, and it may cr

Re: [PR debug/47590] rework md option overriding to delay var-tracking

2011-06-02 Thread Alexandre Oliva
On Jun 1, 2011, Bernd Schmidt wrote: > On 06/02/2011 12:47 AM, Alexandre Oliva wrote: >> On Jun 1, 2011, Bernd Schmidt wrote: >>> Looks ok, except I think you need to update tm.texi.in and tm.texi? >> >> Oh, I didn't realize updating tm.texi.in; AFAICT tm.texi is generated >> the same regardl

Re: fix left-over debug insns in DCE

2011-06-02 Thread Alexandre Oliva
On Jun 2, 2011, Eric Botcazou wrote: > Why can't the problem be addressed purely within DF? Hmm... Maybe it could, I'm not sure. The problem is that DCE removes insns, and then DF associates remaining uses in debug insns to earlier DEFs. Adjusting debug insns in DCE is right per the VTA desi

libobjc: remove deprecated API (patch 3)

2011-06-02 Thread Nicola Pero
Another patch in the sequence. This patch removes objc/objc-list.h, a public header that is deprecated and no longer part of the public API. It is still used internally by libobjc, which has got its own private copy (which is now private and may change without notice). Committed to trunk. Thank

Re: Ping: Re: Improve DSE in the presence of calls

2011-06-02 Thread Easwaran Raman
Ping. On Sat, May 14, 2011 at 8:01 AM, Easwaran Raman wrote: > http://gcc.gnu.org/ml/gcc-patches/2011-05/msg00781.html >

libobjc: remove deprecated API (patch 1)

2011-06-02 Thread Nicola Pero
This patch removes a number of deprecated libobjc functions and methods, which are part of the "Traditional" Objective-C API that was deprecated in GCC 4.6.x and are to be removed in GCC 4.7.0. It's the first of a long sequence of patches that does this removal one bit at a time. This one remo

Re: Initialize INSN_COND (was: C6X port 5/11: Track predication conditions more accurately)

2011-06-02 Thread Steve Ellcey
On Thu, 2011-06-02 at 15:29 +0400, Alexander Monakov wrote: > Bernd, > > The problem is INSN_COND should be reset when initializing a new deps > structure, otherwise instructions may get stale conditions from other > previously analyzed instructions. Presuming that sd_init_insn is the > proper pl

[patch committed] Fix PR target/49163

2011-06-02 Thread Kaz Kojima
Hi, The attached patch is to fix PR target/49163. The problem occurs with the unrecognizable insn like (insn 66 141 110 4 (set (reg:SI 4 r4) (sign_extend:SI (subreg:QI (mem/s/v/u/c:DI (plus:SI (reg/f:SI 7 r7 [192]) (const_int 12 [0xc])) [3 s2array[1][0].f0+0 S8 A3

Re: [patch] add -Wdelete-non-virtual-dtor

2011-06-02 Thread Jonathan Wakely
On 2 June 2011 22:27, Jonathan Wakely wrote: > -Wnon-virtual-dtor isn't always what you want, defining a polymorphic > object without a virtual destructor is not necessarily a mistake. You > may never delete such an object so instead of warning when the class > is defined it's more useful to warn o

Re: Remove SETJMP_VIA_SAVE_AREA support

2011-06-02 Thread Eric Botcazou
> This exposed a couple of similar bugs in cse.c and postreload-gcse.c: the > code was effectively treating a basic block with a single, abnormal > incoming edge as if the edge was normal. I've installed the following refined fix, after testing on i586-suse-linux and sparc-sun-solaris2.10. Most

libquadmath with older versions of gcc

2011-06-02 Thread Syed Bilal Mehdi
Hello, I am new to Linux and there are some basic problems that I am stuck with. What I want to do is 1. Interface Matlab with gcc and 2. Use quad precision with gcc I am using old version of Matlab that requires me to use gcc-4.1 (or gcc-4.2.0 as the latest). 1. Can anyone please tell me how

[patch] add -Wdelete-non-virtual-dtor

2011-06-02 Thread Jonathan Wakely
-Wnon-virtual-dtor isn't always what you want, defining a polymorphic object without a virtual destructor is not necessarily a mistake. You may never delete such an object so instead of warning when the class is defined it's more useful to warn only when the class is deleted, as Clang does with -Wd

Re: [google]Backport r174549 Fix 3 test cases incorrectly run in Thumb/Xscale (issue4524090)

2011-06-02 Thread Carrot Wei
OK for google/main. thanks Carrot On Thu, Jun 2, 2011 at 12:51 PM, Jing Yu wrote: > http://gcc.gnu.org/ml/gcc-patches/2010-10/msg00134.html > Backport r174549 to fix three testcases that are specific to ARM mode > and therefore should be skipped when compiling for thumb. > > Thanks, > Jing > > 2

[PATCH, i386]: Introduce Y3 register constraint and merge SSE3 patterns

2011-06-02 Thread Uros Bizjak
Hello! 2011-06-02 Uros Bizjak * config/i386/constraints.md (Y3): New register constraint. * config/i386/sse.md (*vec_interleave_highv2df): Merge with *sse3_interleave_highv2df and *sse2_interleave_highv2df. (*vec_interleave_lowv2df): Merge with *sse3_interleave_

Re: PING^2 [PATCH] Support for AMD64 targets running GNU/kFreeBSD

2011-06-02 Thread Robert Millan
Hi, 2011/5/21 Joseph S. Myers : > Please send a patch against *current trunk* and CC *relevant target > architecture maintainers*.  linux*.h headers are no longer used on > non-Linux targets (since my 2011-04-28 patch - on which I CC:ed you) so > this patch version is no longer appropriate.  I thi

Re: __sync_swap* with acq/rel/full memory barrier semantics

2011-06-02 Thread Aldy Hernandez
On 06/02/11 14:25, Jakub Jelinek wrote: +case MEMMODEL_SEQ_CST: + op = sync_mem_exchange_seq_cst_optab; + break; +case MEMMODEL_ACQUIRE: + op = sync_mem_exchange_acq_optab; + break; +case MEMMODEL_RELEASE: + op = sync_mem_exchange_rel_optab; + break; +

[PATCH] [Bug c++/49118] fake template nesting for operator-> chain

2011-06-02 Thread David Krauss
This is my first frontend contribution. While it fixes the crash and produces an explanatory error message, the message isn't quite right. I don't understand the message generation system so I might need help. Or, it looks like there's an issue with template backtraces at the moment anyway, so t

Re: __sync_swap* with acq/rel/full memory barrier semantics

2011-06-02 Thread Jakub Jelinek
On Thu, Jun 02, 2011 at 02:12:38PM -0500, Aldy Hernandez wrote: > +/* This function expands a fine grained atomic exchange operation: > + atomically store VAL in MEM and return the previous value in MEM. > + > + MEMMODEL is the memory model variant to use. > + TARGET is an option place to st

Re: __sync_swap* with acq/rel/full memory barrier semantics

2011-06-02 Thread Aldy Hernandez
On 05/30/11 15:07, Andrew MacLeod wrote: Aldy was just too excited about working on memory model I think :-) I've been looking at this, and I propose we go this way : http://gcc.gnu.org/wiki/Atomic/GCCMM/CodeGen Still overly excited, but now with a more thorough plan :). I'm going to concen

Re: Fix for PR objc/48539 ("Missing warning when messaging a forward-declared class")

2011-06-02 Thread Mike Stump
On Jun 2, 2011, at 11:29 AM, Nicola Pero wrote: > This patch fixes PR objc/48539 ("Missing warning when messaging a > forward-declared class"). > Ok to commit to trunk ? Ok.

Fix for PR objc/48539 ("Missing warning when messaging a forward-declared class")

2011-06-02 Thread Nicola Pero
This patch fixes PR objc/48539 ("Missing warning when messaging a forward-declared class"). The problem occurs when using @class, and then messaging class or instance objects of the class. It can happen both with class and instance methods. -- An example with class methods is @class MyClass;

Re: [google] Use minimum cost circulation, not minimum cost flow to smooth profiles other minor fixes. (issue4536106)

2011-06-02 Thread Martin Thuresson
On Thu, Jun 2, 2011 at 11:17 AM, Xinliang David Li wrote: > Counter overflow? Infinite capacity should not be modified. If it is, there is a chance for overflow as it is stored as #define CAP_INFINITY INTTYPE_MAXIMUM (HOST_WIDEST_INT) Martin > > David > > On Thu, Jun 2, 2011 at 11:12 AM, Martin

Re: [google] Use minimum cost circulation, not minimum cost flow to smooth profiles other minor fixes. (issue4536106)

2011-06-02 Thread Xinliang David Li
Counter overflow? David On Thu, Jun 2, 2011 at 11:12 AM, Martin Thuresson wrote: > On Thu, Jun 2, 2011 at 11:05 AM, Xinliang David Li wrote: >> >> Smoothing works for sample FDO and profile data from multi-threaded >> programs. You won't see any difference in SPEC. > > Dehao reported some perfo

Re: [patch] testsuite: support board_info timeouts

2011-06-02 Thread DJ Delorie
Thanks! Committed.

Re: [google] Use minimum cost circulation, not minimum cost flow to smooth profiles other minor fixes. (issue4536106)

2011-06-02 Thread Martin Thuresson
On Thu, Jun 2, 2011 at 11:05 AM, Xinliang David Li wrote: > > Smoothing works for sample FDO and profile data from multi-threaded > programs. You won't see any difference in SPEC. Dehao reported some performance improvements from the algorithmic improvements he added in terms of extra fixup edges

Re: [google] Use minimum cost circulation, not minimum cost flow to smooth profiles other minor fixes. (issue4536106)

2011-06-02 Thread Xinliang David Li
ok for google/main. David On Thu, Jun 2, 2011 at 11:00 AM, Martin Thuresson wrote: > This patch from Neil Vachharajani and Dehao Chen improves mcf by using > minimum cost circulation instead of minimum cost flow to smooth profiles. > It also introduces a parameter for controlling running time of

Re: [google] Use minimum cost circulation, not minimum cost flow to smooth profiles other minor fixes. (issue4536106)

2011-06-02 Thread Xinliang David Li
Smoothing works for sample FDO and profile data from multi-threaded programs. You won't see any difference in SPEC. David On Thu, Jun 2, 2011 at 11:00 AM, Martin Thuresson wrote: > This patch from Neil Vachharajani and Dehao Chen improves mcf by using > minimum cost circulation instead of minimu

[google] Use minimum cost circulation, not minimum cost flow to smooth profiles other minor fixes. (issue4536106)

2011-06-02 Thread Martin Thuresson
This patch from Neil Vachharajani and Dehao Chen improves mcf by using minimum cost circulation instead of minimum cost flow to smooth profiles. It also introduces a parameter for controlling running time of the algorithm. This was what was originally presented in the academic work and handles cert

Re: [PATCH libcpp]: S_ISREG non-zero value does not always fit in a bool

2011-06-02 Thread Tom Tromey
> "John" == John Tytgat writes: John> 2011-05-29 John Tytgat John> * files.c (read_file_guts): Add test on non-zero value of S_ISREG. It seems reasonable enough to me. I am checking it in. Out of curiosity, do you know of a platform where this is an issue? Tom

Re: [PATCH] c-pragma: adding a data field to pragma_handler

2011-06-02 Thread Tom Tromey
> "Pierre" == Pierre writes: Pierre> I have changed this handler in order to accept a second parameter Pierre> which is a void *, allowing to give extra datas to the handler. I Pierre> think this data field might be of general use: we can have condition Pierre> or data at register time that

[PATCH, i386]: Introduce Y4 register constraint and merge SSE4_1 patterns

2011-06-02 Thread Uros Bizjak
Hello! ... and some unrelated cleanups involving simplifying a couple of switch statements. 2011-06-02 Uros Bizjak * config/i386/i386.c (standard_sse_constant_p) : Simplify switch statement. * config/i386/i386.md (*movdf_internal_rex64) : Ditto. (*movdf_interna

Re: [PATCH] PR fortran/49265 -- allow for double colon in module procedure statement

2011-06-02 Thread Steve Kargl
On Thu, Jun 02, 2011 at 06:39:18PM +0200, Thomas Koenig wrote: > Hi Steve, > > > >Oh phew. Good catch. I wasn't dealing with > >the possible white space issues. Here's an > >updated patch and testcase. > > > > OK for trunk. Could you also add the test case a second time, > without -std=f95,

Re: [patch] testsuite: support board_info timeouts

2011-06-02 Thread Mike Stump
On Jun 2, 2011, at 9:48 AM, DJ Delorie wrote: > I never got feedback from the testsuite maintainers on this one... Ok.

Re: Ping: [Patch] Make libstdc++'s abi_check more robust against readelf output format

2011-06-02 Thread Ian Lance Taylor
On Thu, Jun 2, 2011 at 3:08 AM, Paolo Carlini wrote: > Hi, > >> Ping. > > Did Ian Taylor see this patch? If he likes it, I'm also fine with it. I think this patch is fine, with or without Jonathan's suggestion. Ian

Re: Add missing ChangeLog entry

2011-06-02 Thread Ian Lance Taylor
Nathan Sidwell writes: > On 06/01/11 15:32, Ian Lance Taylor wrote: >> I noticed that we have a --with-specs option in gcc/configure.ac, added >> in revision 155208 with this e-mail message: >> http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00132.html > > sorry about that. How's the attachd docume

Re: [PATCH] PR fortran/49265 -- allow for double colon in module procedure statement

2011-06-02 Thread Steve Kargl
On Thu, Jun 02, 2011 at 06:39:18PM +0200, Thomas Koenig wrote: > Hi Steve, > > > >Oh phew. Good catch. I wasn't dealing with > >the possible white space issues. Here's an > >updated patch and testcase. > > > > OK for trunk. Could you also add the test case a second time, > without -std=f95,

Re: [patch] testsuite: support board_info timeouts

2011-06-02 Thread DJ Delorie
I never got feedback from the testsuite maintainers on this one... > Date: Mon, 9 Aug 2010 23:48:31 -0400 > From: DJ Delorie > Mailing-List: contact gcc-patches-h...@gcc.gnu.org; run by ezmlm > > > Is there any reason why we don't support board-level timeouts? It's > really hard to specify ti

Re: [google] Improve locus information during if-conversion (issue4526101)

2011-06-02 Thread शरद सिंघई
This patch improves precision of the line number information during coverage mode. Yes, I need to add an example/test case. I was planning to do that before I propose this patch for trunk as well. Thanks, Sharad On Thu, Jun 2, 2011 at 4:46 AM, Diego Novillo wrote: > > On Wed, Jun 1, 2011 at 21:0

Re: [PATCH, ARM] Cortex-A5 tuning [2/2] - tweak instruction conditionalisation

2011-06-02 Thread Julian Brown
On Wed, 01 Jun 2011 17:00:30 +0100 Richard Earnshaw wrote: > > On Wed, 2011-06-01 at 16:49 +0100, Julian Brown wrote: > > This patch tweaks the behaviour of arm_final_prescan_insn when > > tuning for Cortex-A5 cores, since branches are cheaper than long > > sequences of conditionalised instructi

Re: [PATCH] PR fortran/49265 -- allow for double colon in module procedure statement

2011-06-02 Thread Thomas Koenig
Hi Steve, Oh phew. Good catch. I wasn't dealing with the possible white space issues. Here's an updated patch and testcase. OK for trunk. Could you also add the test case a second time, without -std=f95, to make sure it keeps passing? Thanks for the patch! Thomas

Re: [PATCH] PR fortran/49265 -- allow for double colon in module procedure statement

2011-06-02 Thread Steve Kargl
On Thu, Jun 02, 2011 at 05:59:29PM +0200, Thomas Koenig wrote: > Hi Steve, > > it seems that, with your patch, > >interface foo > module procedure::bar >end interface > > is rejected, as is > > interface foo > module procuedure:: bar > end interface > > Is this the way i

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

2011-06-02 Thread Ira Rosen
On 26 May 2011 10:52, Ira Rosen wrote: > Hi, > > The vectorizer supports strided loads with gaps, e.g., when only a[4i] > and a[4i+2] are accessed, it generates a vector load a[4i:4i+3], i.e., > creating an access to a[4i+3], which doesn't exist in the scalar code. > This access maybe invalid as d

Re: [PATCH] PR fortran/49265 -- allow for double colon in module procedure statement

2011-06-02 Thread Thomas Koenig
Hi Steve, it seems that, with your patch, interface foo module procedure::bar end interface is rejected, as is interface foo module procuedure:: bar end interface Is this the way it is supposed to be? Thomas

[gc-improv] Fix all remaining C testsuite failures

2011-06-02 Thread Laurynas Biveinis
This patch - Fixes PCH failures by re-initializing struct function after PCH read. - Allocates couple of global RTXes in the permanent memory. - Fixes RTL copying by taking source and destination memory areas into account. I.e. RTXes that would be normally shared, if source is in the permanent, an

Re: [PATCH][ARM] add support for some missing 16-bit multiplication insns

2011-06-02 Thread Richard Earnshaw
On Fri, 2011-05-27 at 17:36 +0100, Andrew Stubbs wrote: > Hi all, > > This patch adds support for the ARM SMLALTB, SMLALTT, SMLATB, and SMLATT > instructions. > > These instructions do HImode -> DI/SImode widening > multiply-and-accumulate with one or both operands taken from the top 16 > bit

Re: [PATCH, ARM] Fix ABI for double-precision helpers on single-float-only CPUs

2011-06-02 Thread Richard Earnshaw
On Fri, 2011-05-27 at 17:32 +0100, Julian Brown wrote: > The helper functions used to implement double-precision arithmetic on > ARM processors that only support single-precision arithmetic in hardware > should use the soft-float ABI (i.e. passing and returning floating-point > arguments in core r

Re: [patch] Improve detection of widening multiplication in the vectorizer

2011-06-02 Thread Richard Guenther
On Thu, Jun 2, 2011 at 1:08 PM, Ira Rosen wrote: > On 2 June 2011 12:59, Richard Guenther wrote: >> On Thu, Jun 2, 2011 at 10:46 AM, Ira Rosen wrote: >>> On 1 June 2011 15:14, Richard Guenther wrote: On Wed, Jun 1, 2011 at 1:37 PM, Ira Rosen wrote: > On 1 June 2011 12:42, Richard Guen

Re: fix left-over debug insns in DCE

2011-06-02 Thread Eric Botcazou
> One of the issues was that DCE removed an insn that set a REG in a > certain mode, without adjusting a debug use of that REG. This was in > libstdc++, but I failed to take note of the affected file. DF later > attached that debug use to another SET to the same REG in a different, > incompatible

[PATCH] PR fortran/49265 -- allow for double colon in module procedure statement

2011-06-02 Thread Steve Kargl
The attached patch allows for F2008's optional double colon in a module procedure statement. Built and regression tested on trunk. OK for trunk and 4.6? Steven G. Kargl PR fortran/49265 * decl.c (gfc_match_modproc): Allow for a double colon in a module procedure stateme

[gcc patch 3/3] cp-demangle.c: Fix DMGL_RET_POSTFIX for inner func types

2011-06-02 Thread Jan Kratochvil
Hi, I do not need this patch in any way but I believe it should go in for the case anyone would want to use DMGL_RET_POSTFIX with C/C++. Without this fix the new testcase would: FAIL at line 3979, options --format=gnu-v3 --ret-postfix: in: _Z6outer2IsEPFilES1_ out: outer2(int (*)(long))int (*(in

[gcc patch 2/3] cp-demangle.c: New DMGL_RET_DROP

2011-06-02 Thread Jan Kratochvil
Hi, implement DMGL_RET_DROP as an alternative form of DMGL_RET_POSTFIX, as described in [gcc patch 0/3]. Thanks, Jan include/ 2011-05-24 Jan Kratochvil * demangle.h (DMGL_RET_POSTFIX): Extend the comment. (DMGL_RET_DROP): New. libiberty/ 2011-05-24 Jan Kratochvil

[gcc patch 1/3] cp-demangle.c: Make `options' a parameter

2011-06-02 Thread Jan Kratochvil
Hi, this patch makes no functionality change, it only reshuffles the code. After printing the toplevel function type we want to disable DMGL_RET_POSTFIX for any inner types. Without such DMGL_RET_POSTFIX disabling for inner types the new testcase would: FAIL at line 3973, options --format=gnu-v

[gcc patch 0/3] libiberty: New DMGL_RET_DROP

2011-06-02 Thread Jan Kratochvil
Hi, introducing DMGL_RET_DROP which suppresses return type demangled from linkage name for the toplevel function type. DMGL_RET_POSTFIX is now in use only for DMGL_JAVA. Besides Java return types in linkage name are in C++ present for function templates. GDB since 7.2 provides convenience alias

Re: [PATCH][all-langs] Defer size_t and sizetype setting to the middle-end

2011-06-02 Thread Janne Blomqvist
On Wed, Jun 1, 2011 at 14:34, Richard Guenther wrote: > > This patch defers the control over size_t and sizetype to the > middle-end which in turn consults the target.  This removes > various inconsistencies for frontends that do not seem to care > about size_t and will allow simplifying the globa

Re: [PATCH, ARM] Fix ABI for double-precision helpers on single-float-only CPUs

2011-06-02 Thread Paul Brook
> gcc/ > * config/arm/arm.c (arm_libcall_uses_aapcs_base) > (arm_init_cumulative_args): Use correct ABI for double-precision > helper functions in hard-float mode if only single-precision > arithmetic is supported in hardware. Ok, though I'd add a bit more explanation to the co

Re: [google] Improve locus information during if-conversion (issue4526101)

2011-06-02 Thread Diego Novillo
On Thu, Jun 2, 2011 at 08:46, Steven Bosscher wrote: > On Thu, Jun 2, 2011 at 1:46 PM, Diego Novillo wrote: >> On Wed, Jun 1, 2011 at 21:03, Sharad Singhai wrote: >> >>> 2011-06-01  Sharad Singhai   >>> >>>        Google Ref 39994 >>>        * ifcvt.c (noce_try_cmove_arith): Use the locus inform

Re: [google] Improve locus information during if-conversion (issue4526101)

2011-06-02 Thread Steven Bosscher
On Thu, Jun 2, 2011 at 1:46 PM, Diego Novillo wrote: > On Wed, Jun 1, 2011 at 21:03, Sharad Singhai wrote: > >> 2011-06-01  Sharad Singhai   >> >>        Google Ref 39994 >>        * ifcvt.c (noce_try_cmove_arith): Use the locus information >>        from the if-statment rather than the then path

[PATCH] make attribute((returns_twice)) actually work (PR tree-optimization/49243)

2011-06-02 Thread Mikael Pettersson
GCC has attribute((returns_twice)) which is supposed to allow the safe use of alternate implementations of setjmp-like functions. In particular, a function that calls a setjmp-like function must itself not be inlined, because that would enable unsafe optimizations. This works for calls to "setjmp

Re: [google] Improve locus information during if-conversion (issue4526101)

2011-06-02 Thread Diego Novillo
On Wed, Jun 1, 2011 at 21:03, Sharad Singhai wrote: > 2011-06-01  Sharad Singhai   > >        Google Ref 39994 >        * ifcvt.c (noce_try_cmove_arith): Use the locus information >        from the if-statment rather than the then path. Could you elaborate how it improves locus information? Is

Re: Initialize INSN_COND

2011-06-02 Thread Bernd Schmidt
On 06/02/2011 01:29 PM, Alexander Monakov wrote: > Bernd, > > The problem is INSN_COND should be reset when initializing a new deps > structure, otherwise instructions may get stale conditions from other > previously analyzed instructions. Presuming that sd_init_insn is the > proper place for tha

Initialize INSN_COND (was: C6X port 5/11: Track predication conditions more accurately)

2011-06-02 Thread Alexander Monakov
Bernd, The problem is INSN_COND should be reset when initializing a new deps structure, otherwise instructions may get stale conditions from other previously analyzed instructions. Presuming that sd_init_insn is the proper place for that, I'll test the following patch. 2011-06-02 Alexander Mon

Re: Ping: [Patch] Make libstdc++'s abi_check more robust against readelf output format

2011-06-02 Thread Jonathan Wakely
On 2 June 2011 08:55, Simon Baldwin wrote: >> Index: libstdc++-v3/scripts/extract_symvers.in >> === >> --- libstdc++-v3/scripts/extract_symvers.in     (revision 173951) >> +++ libstdc++-v3/scripts/extract_symvers.in     (working copy)

Re: [patch] Improve detection of widening multiplication in the vectorizer

2011-06-02 Thread Ira Rosen
On 2 June 2011 12:59, Richard Guenther wrote: > On Thu, Jun 2, 2011 at 10:46 AM, Ira Rosen wrote: >> On 1 June 2011 15:14, Richard Guenther wrote: >>> On Wed, Jun 1, 2011 at 1:37 PM, Ira Rosen wrote: On 1 June 2011 12:42, Richard Guenther wrote: > Did you think about moving pass_

Remove SETJMP_VIA_SAVE_AREA support

2011-06-02 Thread Eric Botcazou
This removes the (undocumented) support for SETJMP_VIA_SAVE_AREA from the compiler. This is a trick implemented on the SPARC exclusively to reuse the register save area present in all frames (because of the register windows) for part of the setjmp buffer. The benefit are marginal and dwarfed b

Re: Add missing ChangeLog entry

2011-06-02 Thread Basile Starynkevitch
On Thu, 02 Jun 2011 11:12:12 +0100 Nathan Sidwell wrote: > On 06/01/11 15:32, Ian Lance Taylor wrote: > > I noticed that we have a --with-specs option in gcc/configure.ac, added > > in revision 155208 with this e-mail message: > > http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00132.html > > sorry

Re: [PATCH][ARM] Add support for ADDW and SUBW instructions

2011-06-02 Thread Ramana Radhakrishnan
On 2 June 2011 10:03, Andrew Stubbs wrote: > On 02/06/11 09:23, Ramana Radhakrishnan wrote: >> >> Please remove the alternatives in the subsi3 pattern since that is just >> unnecessary. Please make the constraints internal only. > > Is this better? > OK. Ramana > Andrew >

Re: RFA: another patch to solve PR49154

2011-06-02 Thread Hans-Peter Nilsson
On Tue, 31 May 2011, Richard Sandiford wrote: > Gah, seems like I'd forgotten the "no subclasses" bit by the time > I started looking at code. Sorry for the false alarm. Still, the extra look made me realise that I should have restricted that statement to allocatable registers. (And I really do

Re: Add missing ChangeLog entry

2011-06-02 Thread Nathan Sidwell
On 06/01/11 15:32, Ian Lance Taylor wrote: I noticed that we have a --with-specs option in gcc/configure.ac, added in revision 155208 with this e-mail message: http://gcc.gnu.org/ml/gcc-patches/2009-12/msg00132.html sorry about that. How's the attachd documentation? -- Nathan Sidwell 2011-0

Re: Ping: [Patch] Make libstdc++'s abi_check more robust against readelf output format

2011-06-02 Thread Paolo Carlini
Hi, > Ping. Did Ian Taylor see this patch? If he likes it, I'm also fine with it. Paolo

Re: introduce --param max-vartrack-expr-depth

2011-06-02 Thread Bernd Schmidt
On 06/02/2011 10:46 AM, Jakub Jelinek wrote: > On Wed, Jun 01, 2011 at 07:25:39PM -0300, Alexandre Oliva wrote: >> Such as this one... > > I'd appreciate if this could go in... Go on then. Bernd

Re: [patch] Improve detection of widening multiplication in the vectorizer

2011-06-02 Thread Richard Guenther
On Thu, Jun 2, 2011 at 10:46 AM, Ira Rosen wrote: > On 1 June 2011 15:14, Richard Guenther wrote: >> On Wed, Jun 1, 2011 at 1:37 PM, Ira Rosen wrote: >>> On 1 June 2011 12:42, Richard Guenther wrote: >>> Did you think about moving pass_optimize_widening_mul before loop optimizations?

Re: [patch][simplify-rtx] Fix 16-bit -> 64-bit multiply and accumulate

2011-06-02 Thread Richard Earnshaw
On Thu, 2011-05-26 at 14:35 +0100, Andrew Stubbs wrote: > On 25/05/11 14:47, Joseph S. Myers wrote: > > The shift must be by a positive constant amount, strictly less than the > > precision (GET_MODE_PRECISION) of the mode (of the value being shifted). > > If that applies, the relevant number of b

Re: [PATCH][ARM] Add support for ADDW and SUBW instructions

2011-06-02 Thread Andrew Stubbs
On 02/06/11 09:23, Ramana Radhakrishnan wrote: Please remove the alternatives in the subsi3 pattern since that is just unnecessary. Please make the constraints internal only. Is this better? Andrew 2011-06-02 Andrew Stubbs gcc/ * config/arm/arm-protos.h (const_ok_for_op): Add prototype.

Re: introduce --param max-vartrack-expr-depth

2011-06-02 Thread Jakub Jelinek
On Wed, Jun 01, 2011 at 07:25:39PM -0300, Alexandre Oliva wrote: > Such as this one... I'd appreciate if this could go in... > Index: gcc/params.def > === > --- gcc/params.def.orig 2011-05-31 18:28:05.348070586 -0300 > +++ gcc/

Re: [patch] Improve detection of widening multiplication in the vectorizer

2011-06-02 Thread Ira Rosen
On 1 June 2011 15:14, Richard Guenther wrote: > On Wed, Jun 1, 2011 at 1:37 PM, Ira Rosen wrote: >> On 1 June 2011 12:42, Richard Guenther wrote: >> >>> Did you think about moving pass_optimize_widening_mul before >>> loop optimizations?  Does that pass catch the cases you are >>> teaching the p

Re: [PATCH, ARM] Thumb-2 12-bit immediates in ADD and SUB instructions

2011-06-02 Thread Ramana Radhakrishnan
Would you include this in your patch? Or should we submit it as a separate patch? Could you submit this as a follow-up patch that touches the costs. I would rather that these changes also went in when we were looking at this area ? cheers Ramana

Re: [PATCH][ARM] Add support for ADDW and SUBW instructions

2011-06-02 Thread Ramana Radhakrishnan
> OK? > This is largely OK modulo the following. Please remove the alternatives in the subsi3 pattern since that is just unnecessary. Please make the constraints internal only. cheers Ramana > Andrew

Re: [PATCH][ARM] Add support for ADDW and SUBW instructions

2011-06-02 Thread Andrew Stubbs
Ping 2. On 20/04/11 16:27, Andrew Stubbs wrote: This patch adds basic support for the Thumb ADDW and SUBW instructions. The patch permits the compiler to use the new instructions for constants that can be loaded with a single instruction (i.e. 16-bit unshifted), but does not support use of addw

Ping: [Patch] Make libstdc++'s abi_check more robust against readelf output format

2011-06-02 Thread Simon Baldwin
Ping. On 20 May 2011 17:05, Simon Baldwin wrote: > > Make libstdc++'s abi_check more robust against readelf output format. > > libstdc++-abi/abi_check in the libstdc++-v3 testsuite relies on a fixed > number of space separated fields in readelf output.  However, the field > count for readelf out

Re: -fdump-passes -fenable-xxx=func_name_list

2011-06-02 Thread Xinliang David Li
This is the version of the patch that walks through pass lists. Ok with this one? David On Wed, Jun 1, 2011 at 12:45 PM, Xinliang David Li wrote: > On Wed, Jun 1, 2011 at 12:29 PM, Richard Guenther > wrote: >> On Wed, Jun 1, 2011 at 6:16 PM, Xinliang David Li wrote: >>> On Wed, Jun 1, 2011 at