Re: fix left-over debug insns in DCE

On Jun 3, 2011, Eric Botcazou wrote: > Does the same logic need to be replicated in all passes that do? On > the other hand, these passes call into DF when they remove insns, so > DF is a central place here. I went over (again?) a number of passes that call delete_insn directly or indirectly.

Re: fix left-over debug insns in DCE

> It might be too late for DF to do anything sensible to preserve the > debug info rather than just throw it away, as your partial approval > suggests. OK, let me think about this a little more. > Indeed, sorry, I misread it. Mind installing these bits separately? My understanding is that they

Re: [lto] Unify decl and type registration (issue4515186)

On Fri, 3 Jun 2011, Diego Novillo wrote: > > As discussed in http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00063.html, > this patch moves decl registration in symbol tables to the LTO front > end. It makes type and symbol registration happen at the same time in > uniquify_nodes. > > Tested with

Re: [PATH] PR/49139 fix always_inline failures diagnostics

OK, the only difference is that we don't have the node analyzed here, so externally_visible, etc are not set. With the initial proposal the warning was emitted only if the function could not be inlined. Now it will be emitted for each DECL_COMDAT (decl)&& !DECL_DECLARED_INLINE_P (decl)) even

Ping Re: Create common hooks structure shared between driver and cc1

Ping. This patch is pending review. -- Joseph S. Myers jos...@codesourcery.com

Ping Re: Move option-related hooks to common structure

Ping. This patch is pending review (as is the patch on which it depends). -- Joseph S. Myers jos...@codesourcery.com

Re: [build] Move ENABLE_EXECUTE_STACK to toplevel libgcc

Paolo Bonzini writes: >> * Except for Darwin, the code uses TRAMPOLINE_SIZE. This only exists in >>the backend headers. While it could be duplicated somewhere in the >>libgcc configury, I'd rather propose that gcc define >>__TRAMPOLINE_SIZE__ (in gcc/c-family/c-cppbuiltin.c (c_cpp_b

Re: [build] Move ENABLE_EXECUTE_STACK to toplevel libgcc

On 06/06/2011 11:17 AM, Rainer Orth wrote: * Instead of __FreeBSD__, one could use HAVE_SYSCTLBYNAME instead, but that would need a new libgcc config.h header. In addition, we might have to check for kern.stackprot to make sure the code really works. * Similarly, instead of testing __sun_

Re: [PATCH] Fix fold_stmt ICE (PR c++/49264)

On Fri, Jun 3, 2011 at 3:55 PM, Jakub Jelinek wrote: > Hi! > > If memcpy is folded into an assignment, that assignment can be for C++ > folded into nothing (if it is copying of 1 byte from or to empty C++ class). > gimple-fold.c was changed to handle that case in some spots, but not all, > particu

[build] Use crtfastmath.c on IRIX 6

On top of the IRIX 6 toplevel libgcc move, I noticed that IRIX doesn't use crtfastmath.o yet. This patch corrects this, bootstrapped without regressions on mips-sgi-irix6.5. While doing this, I noticed that libgcc/config/mips/t-crtfm is identical (modulo $(cpu_type)) to the generic libgcc/config/

Re: [PATCH] Only run pr48377.c testcase on i?86/x86_64

On Fri, Jun 3, 2011 at 3:59 PM, Jakub Jelinek wrote: > Hi! > > This limits this testcase to i?86/x86_64 (moving to gcc.target/ would > be harder because it relies on all the weirdo vectorization options to be > passed), because apparently on strict alignment targets we don't handle > aligned (1) n

Re: [vta, graphite?] propagate degenerate phi nodes into debug stmts

On Fri, Jun 3, 2011 at 4:33 PM, Alexandre Oliva wrote: > According to http://gcc.gnu.org/ml/gcc-patches/2009-11/msg00999.html > on Nov 19, 2009, Richard Guenther wrote: > >> On Thu, Nov 19, 2009 at 4:05 AM, Alexandre Oliva wrote: >>> On Nov 17, 2009, Richard Guenther wrote: >>> >> This look

Re: don't dump decl_uid with TDF_NOUID

On Fri, Jun 3, 2011 at 4:17 PM, Alexandre Oliva wrote: > A recent change introduced decl_uid in the “;; Function ” header in dump > files.  This breaks -fcompare-debug (and bootstrap-debug-lean), because > decl uids aren't kept in sync between -g and non-g compilations. > > This patch rearranges t

Re: [testsuite] Run TLS torture tests with -fpic etc.

Jakub Jelinek writes: > On Fri, Jun 03, 2011 at 09:38:31PM +0200, Rainer Orth wrote: >> Rainer Orth writes: >> Jakub, any suggestion how to properly test for -fpie/-fPIE support? >> Otherwise, I'll remove that part of the patch for now and just commit >> the -fpic/-fPIC one. > > You want to comp

Re: [build] Use crtfastmath.c on IRIX 6

On 06/06/2011 11:32 AM, Rainer Orth wrote: On top of the IRIX 6 toplevel libgcc move, I noticed that IRIX doesn't use crtfastmath.o yet. This patch corrects this, bootstrapped without regressions on mips-sgi-irix6.5. While doing this, I noticed that libgcc/config/mips/t-crtfm is identical (modu

Re: [PATCH] Only run pr48377.c testcase on i?86/x86_64

Richard Guenther writes: > On Fri, Jun 3, 2011 at 3:59 PM, Jakub Jelinek wrote: >> Hi! >> >> This limits this testcase to i?86/x86_64 (moving to gcc.target/ would >> be harder because it relies on all the weirdo vectorization options to be >> passed), because apparently on strict alignment targe

Re: [ARM] TLS Descriptor support

On 06/03/11 23:50, Ramana Radhakrishnan wrote: How are things handled for Thumb1 in case someone builds a routine for Thumb1 which uses tlsdesc ? I went and read the doc and didn't see any difference between T1 and T2 in the specification . Would the linker and everyone else do the right thing o

Re: [ARM] TLS Descriptor support

> If you're asking something else, can you rephrase the question? Sorry if I wasn't too clear. My question really should have read "why do we have to special case Thumb2" ? The linker should be able to veneer the t-> a calls either through the veneering sequence (in case of T1 without blx) or con

Re: [PATH] PR/49139 fix always_inline failures diagnostics

On Mon, Jun 6, 2011 at 10:58 AM, Christian Bruel wrote: > > >>> OK, the only difference is that we don't have the node analyzed here, so >>> externally_visible, etc are not set. With the initial proposal the >>> warning >>> was emitted only if the function could not be inlined. Now it will be >>>

Re: [ARM] TLS Descriptor support

On 06/06/11 10:53, Ramana Radhakrishnan wrote: If you're asking something else, can you rephrase the question? Sorry if I wasn't too clear. My question really should have read "why do we have to special case Thumb2" ? The linker should be able to veneer the t-> a calls either through the vene

[Ada] Fix ICE on Taft amendment types with -g

Taft amendment types are incomplete types declared in a package spec whose completion is declared in the body. This means that other units don't have access to their full view at all and treat them as incomplete types. This is problematic when debugging information is requested because their c

Re: fix left-over debug insns in DCE

On Jun 6, 2011, Eric Botcazou wrote: >> Indeed, sorry, I misread it. > Mind installing these bits separately? Nope. Testing it separately now. > My understanding is that they are > independent correctness fixes. Yeah, I guess they are. They won't make debug info any worse, although they m

[Ada] Fix ICE on deferred constant with tagged type

This is again the compiler trying to create a temporary for a VIEW_CONVERT_EXPR and the type doesn't allow it. Fixed by not generating the VIEW_CONVERT_EXPR in the first place. Tested on i586-suse-linux, applied on the mainline 2011-06-06 Eric Botcazou * gcc-interface/trans.c (Ide

[Ada] Fix ICE on conditional expression with unconstrained type

This is again the compiler trying to create a temporary, for an unconstrained type hence whose size isn't fixed. Fixed by not creating the temporary as it is superfluous here. Tested on i586-suse-linux, applied on the mainline 2011-06-06 Eric Botcazou * gcc-interface/utils2.c (gna

[Ada] Propagate TREE_THIS_NOTRAP flag

Self-explanatory. Tested on i586-suse-linux, applied on the mainline. 2011-06-06 Eric Botcazou * gcc-interface/utils2.c (gnat_stabilize_reference): Propagate TREE_THIS_NOTRAP flag. -- Eric Botcazou Index: gcc-interface/utils2.c =

Re: [PATCH] gimple_val_nonnegative_real_p (PR46728 patch 7 of 7)

On Wed, Jun 1, 2011 at 9:27 PM, William J. Schmidt wrote: > This patch cleans up the FIXME logic in gimple_expand_builtin_pow by > introducing gimple_val_nonnegative_real_p for the same purpose that > tree_expr_nonnegative_p served in the expand logic.  This completes the > work for PR46728. > > B

Re: Dump before flag

On Wed, Jun 1, 2011 at 11:23 PM, Xinliang David Li wrote: > On Wed, Jun 1, 2011 at 2:12 PM, Basile Starynkevitch > wrote: >> On Wed, 1 Jun 2011 13:26:24 -0700 >> Xinliang David Li wrote: >> >>> Hi, this is a simple patch that support dump_before flag. E.g, >>> >>> -fdump-tree-pre-before >>> >>>

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

On Thu, Jun 2, 2011 at 2:02 PM, Mikael Pettersson wrote: > 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 w

Re: [PATCH] Fix fold_stmt ICE (PR c++/49264)

On Mon, Jun 06, 2011 at 11:30:19AM +0200, Richard Guenther wrote: > On Fri, Jun 3, 2011 at 3:55 PM, Jakub Jelinek wrote: > > --- gcc/tree-inline.c.jj        2011-06-02 10:15:20.0 +0200 > > +++ gcc/tree-inline.c   2011-06-03 09:29:15.0 +0200 > > @@ -4108,6 +4108,14 @@ fold_marked_st

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

On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li wrote: > The attached is the split #1 patch that enhances -fenable/disable. > > Ok after testing? I expect the testcases will be quite fragile, so while I appreciate test coverage for new options I think we should go without those that involve any

Re: Ping: The TI C6X port

Ping^3 for the C6X port. Now with extra patches: Additional preliminary scheduler tweaks: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg02408.html Allow alternatives in attr "predicable": http://gcc.gnu.org/ml/gcc-patches/2011-06/msg00094.html regrename across basic block boundaries: http://gcc.g

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

On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li wrote: > This is the version of the patch that walks through pass lists. > > Ok with this one? +/* Dump all optimization passes. */ + +void +dump_passes (void) +{ + struct cgraph_node *n, *node = NULL; + tree save_fndecl = current_function_decl

Re: [PATCH] Fix fold_stmt ICE (PR c++/49264)

On Mon, Jun 6, 2011 at 1:19 PM, Jakub Jelinek wrote: > On Mon, Jun 06, 2011 at 11:30:19AM +0200, Richard Guenther wrote: >> On Fri, Jun 3, 2011 at 3:55 PM, Jakub Jelinek wrote: >> > --- gcc/tree-inline.c.jj        2011-06-02 10:15:20.0 +0200 >> > +++ gcc/tree-inline.c   2011-06-03 09:29:1

Re: [PING] [PATCH] PR preprocessor/48532 (Wrong location in pragma involving macros)

Tom Tromey a écrit: > Dodji> + context of 'P'. The problem is, if we are beeing > > Typo, "being". > > Ok with that fixed. Thanks. Fixed and committed to revision r174694. -- Dodji

Re: [PATCH] gimple_val_nonnegative_real_p (PR46728 patch 7 of 7)

On Mon, 2011-06-06 at 13:00 +0200, Richard Guenther wrote: > On Wed, Jun 1, 2011 at 9:27 PM, William J. Schmidt > wrote: > > +/* Return true iff VAL is a gimple expression that is known to be > > + non-negative. Restricted to floating-point inputs. When changing > > + this function, revie

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

This is follow-up patch that fixes rtx costs for CONST_INT in PLUS pattern. Original discussion is here: http://gcc.gnu.org/ml/gcc-patches/2011-05/msg01427.html 2011-06-06 Dmitry PLotnikov gcc/ * config/arm/arm.c (arm_rtx_costs_1): Fixed costs for CONST_INT in PLUS pattern. diff --git a/

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

On 06/06/11 13:15, Dmitry Plotnikov wrote: + && (const_ok_for_op (INTVAL (x), outer) + || const_ok_for_op (~INTVAL (x), outer The second call is redundant. const_ok_for_op should already do that. Andrew

Re: Ping: The TI C6X port

Hi Bernd, not a direct approval for any of the outstanding patches, but I am happy to report that the steering committee is appointing you maintainer of the C6X port. Please go ahead and add yourself to the MAINTAINERS file as part of the patch that actually adds the port (10/11 if I recall cor

Re: Ping: The TI C6X port

On 06/06/2011 02:53 PM, Gerald Pfeifer wrote: > not a direct approval for any of the outstanding patches, but I am happy > to report that the steering committee is appointing you maintainer of the > C6X port. Thanks! > Thanks for contributing this work, and happy hacking! And do think of > con

Re: Skip building target libiberty for arm*-*-linux-androideabi

Hi Jing Yu, > 2011-05-08 Jing Yu > > * configure.ac: Skip target-libiberty for arm*-*-linux-androideabi. > * configure: Regenerated. Approved - please apply. Cheers Nick

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

Richard Guenther writes: > Thanks. I would hope that we eventually can get rid of the > pattern recognizer ... at least for SSE there is also always > a scalar variant instruction for each vectorized one. AFAIK, that isn't true for ARM and NEON. E.g. I don't know of a single instruction that do

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

On 06/06/2011 04:41 PM, Andrew Stubbs wrote: On 06/06/11 13:15, Dmitry Plotnikov wrote: + && (const_ok_for_op (INTVAL (x), outer) + || const_ok_for_op (~INTVAL (x), outer The second call is redundant. const_ok_for_op should already do that. Fixed patch is attached. Ok? 201

Re: fix left-over debug insns in DCE

On Jun 6, 2011, Eric Botcazou wrote: >> It might be too late for DF to do anything sensible to preserve the >> debug info rather than just throw it away, as your partial approval >> suggests. > OK, let me think about this a little more. >> Indeed, sorry, I misread it. > Mind installing these

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

On 06/06/11 14:26, Dmitry Plotnikov wrote: if (const_ok_for_arm (INTVAL (x)) - || const_ok_for_arm (~INTVAL (x))) + || const_ok_for_arm (~INTVAL (x)) + || (TARGET_THUMB2&& outer == PLUS + && (const_ok_for_op (INTVAL (x), outer Sorry, I should have not

Re: initialized out.clauses in read_predicate

On May 30, 2011, Alexandre Oliva wrote: > In an -O3 bootstrap, out.clauses are reported as uninitialized. GCC is > not smart enough to realize accesses to the uninitialized members will > never happen. It shouldn't be too expensive to initialize them all, so > this is what this patch does. Reg

Re: initialized out.clauses in read_predicate

> On May 30, 2011, Alexandre Oliva wrote: > > > In an -O3 bootstrap, out.clauses are reported as uninitialized. GCC is > > not smart enough to realize accesses to the uninitialized members will > > never happen. It shouldn't be too expensive to initialize them all, so > > this is what this patc

Re: AIX net/if_arp.h include fix for struct fc_softc

On 06/05/11 21:16, Peter O'Gorman wrote: Ok for trunk? Peter Index: ChangeLog === --- ChangeLog (revision 174678) 2011-??-?? Peter O'Gorman * inclhack.def (aix_net_if_arp): New fix. * fixincl.x: Regenerate.

Re: initialized out.clauses in read_predicate

> On May 30, 2011, Alexandre Oliva wrote: > > > In an -O3 bootstrap, out.clauses are reported as uninitialized. GCC is > > not smart enough to realize accesses to the uninitialized members will > > never happen. It shouldn't be too expensive to initialize them all, so > > this is what this patc

Re: [pph] Clean up PPH tests (issue4572042)

So, I'm getting this: Running /home/dnovillo/pph/svn/src/gcc/testsuite/g++.dg/pph/pph.exp ... XPASS: g++.dg/pph/c120060625-1.cc -I. (test for bogus messages, line ) XPASS: g++.dg/pph/c1eabi1.cc -I. (test for bogus messages, line ) XPASS: g++.dg/pph/x1autometh.cc -I. (test for bogus messages,

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

On 06/06/2011 05:33 PM, Andrew Stubbs wrote: On 06/06/11 14:26, Dmitry Plotnikov wrote: if (const_ok_for_arm (INTVAL (x)) - || const_ok_for_arm (~INTVAL (x))) + || const_ok_for_arm (~INTVAL (x)) + || (TARGET_THUMB2&& outer == PLUS + && (const_ok_for_op (INTVAL (x), outer

Re: [PATCH] gimple_val_nonnegative_real_p (PR46728 patch 7 of 7)

On Mon, Jun 6, 2011 at 2:12 PM, William J. Schmidt wrote: > On Mon, 2011-06-06 at 13:00 +0200, Richard Guenther wrote: >> On Wed, Jun 1, 2011 at 9:27 PM, William J. Schmidt >> wrote: > > > >> > +/* Return true iff VAL is a gimple expression that is known to be >> > +   non-negative.  Restricted

Re: [lto] Unify decl and type registration (issue4515186)

On Mon, Jun 6, 2011 at 04:50, Richard Guenther wrote: > I'd have it done in the loop that computes canonical types, at this > place we do not gain the advantage that the decl register functions > get completely fixed up trees. Hm, yes, I had forgotten about the call to rest_of_decl_compilation.

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

On Mon, Jun 6, 2011 at 3:04 PM, Richard Sandiford wrote: > Richard Guenther writes: >> Thanks.  I would hope that we eventually can get rid of the >> pattern recognizer ... at least for SSE there is also always >> a scalar variant instruction for each vectorized one. > > AFAIK, that isn't true fo

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

On 06/06/11 15:26, Dmitry Plotnikov wrote: On 06/06/2011 05:33 PM, Andrew Stubbs wrote: On 06/06/11 14:26, Dmitry Plotnikov wrote: if (const_ok_for_arm (INTVAL (x)) - || const_ok_for_arm (~INTVAL (x))) + || const_ok_for_arm (~INTVAL (x)) + || (TARGET_THUMB2&& outer == PLUS + && (const_ok_for_op

[Committed] S/390: longlong.h fix smul_ppmm

Hi, the attached patch fixed a failure with -mzarch -m31. The smul_ppmm implementation in longlong.h uses the registers of the result in the wrong order. A corresponding patch for glibc will be posted soon. Committed to 4.6 and mainline. Bye, -Andreas- 2011-06-06 Andreas Krebbel

[PATCH, C++] Use of a variable with undefined value in cp/parser.c?

Hi, when testing my new IPA-CP with make profiledbootstrap, I stumbled across a warning-made-error in parser.c, claiming that parenthesized_p in cp_parser_parameter_declaration_list is used but undefined. I do not understand the code very much but what happens is that the variable is passed by re

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

On Sat, 4 Jun 2011, Diego Novillo wrote: > On Wed, Jun 1, 2011 at 15:19, Richard Guenther wrote: > > > Yes, I see no benefit of using a global function to get access > > to the address of a global variable. > > There is the minor benefit of being able to control access to it, but > I don't hav

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

On Wed, 1 Jun 2011, 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 global tree in

Re: AIX net/if_arp.h include fix for struct fc_softc

On Mon, Jun 6, 2011 at 12:16 AM, Peter O'Gorman wrote: > Hi, > > We ran across an issue with qt-4.7 built with gcc-4.4 on AIX 5.2, 5.3, > 6.1, and 7.1 where some static constructors were not being called. It > turned out to be a header file issue, see, for example, > https://www.ibm.com/developerw

Re: [lto] Merge streamer hooks from pph branch. (issue4568043)

On Mon, Jun 6, 2011 at 10:50, Richard Guenther wrote: > Do you remember if it was only void_zero_node that causes problems? > We could just special-case it in > lto_input_integer_cst/lto_output_integer_cst.  Or even fix the C family > frontends to not create or use this strange node.  It seems to

Re: PATCH [1/n]: Add initial -x32 support

On Sun, Jun 5, 2011 at 9:54 PM, H.J. Lu wrote: > I'd like to start submitting a series of patches to enable x32: > > https://sites.google.com/site/x32abi/ > > The GCC x32 branch is very stable. There are no unexpected failures in > C, C++, Fortran and Objective C testsuites.  SPEC CPU 2K/2006 com

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

On Mon, Jun 6, 2011 at 4:22 AM, Richard Guenther wrote: > On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li wrote: >> The attached is the split #1 patch that enhances -fenable/disable. >> >> Ok after testing? > > I expect the testcases will be quite fragile, so while I appreciate > test coverage

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

On Mon, Jun 6, 2011 at 5:53 PM, Xinliang David Li wrote: > On Mon, Jun 6, 2011 at 4:22 AM, Richard Guenther > wrote: >> On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li wrote: >>> The attached is the split #1 patch that enhances -fenable/disable. >>> >>> Ok after testing? >> >> I expect the tes

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

On Mon, Jun 6, 2011 at 4:38 AM, Richard Guenther wrote: > On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li wrote: >> This is the version of the patch that walks through pass lists. >> >> Ok with this one? > > +/* Dump all optimization passes.  */ > + > +void > +dump_passes (void) > +{ > +  struc

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

On 06/03/11 18:41, Julian Brown wrote: On Thu, 02 Jun 2011 16:35:01 +0100 Richard Earnshaw wrote: I see Paul has already approved this, but I've just spotted one potential problem that might cause latent bugs sometime in the future. The code to register the libcalls is only run once, the

Re: [ARM] TLS Descriptor support

On 6 June 2011 10:56, Nathan Sidwell wrote: > On 06/06/11 10:53, Ramana Radhakrishnan wrote: >>> >>> If you're asking something else, can you rephrase the question? >> >> Sorry if I wasn't too clear.  My question really should have read "why >> do we have to special case Thumb2" ? The linker shoul

Re: [patch gimplify]: Make sure comparison using boolean-type after gimplification

On Thu, May 26, 2011 at 07:02, Richard Guenther wrote: >  81764   dnovillo       { >  81764   dnovillo         if (TREE_CODE (type) == BOOLEAN_TYPE) >  81764   dnovillo           { >  81764   dnovillo             arg0 = copy_node (arg0); >  81764   dnovillo             TREE_TYPE (arg0) = type; >

Re: AIX net/if_arp.h include fix for struct fc_softc

On Mon, Jun 06, 2011 at 11:05:25AM -0400, David Edelsohn wrote: > > David, do you have any idea if this is what it's supposed to be? > > > > Ok for trunk? > > The header certainly does not make sense as is and does not follow AIX > header file conventions. typedef is the only thing that makes sen

Re: Dump before flag

> > Your patch doesn't really improve this but adds to the confusion. > > +  /* Override dump TODOs.  */ > +  if (dump_file && (pass->todo_flags_finish & TODO_dump_func) > +      && (dump_flags & TDF_BEFORE)) > +    { > +      pass->todo_flags_finish &= ~TODO_dump_func; > +      pass->todo_flags_st

Re: [ARM] TLS Descriptor support

On 05/10/11 07:49, Nathan Sidwell wrote: This patch implements TLS descriptor support in GCC. TLS descriptors are described at http://www.codesourcery.com/publications/RFC-TLSDESC-ARM.txt and blessed by ARM, who have reserved the relocation numbers. Binutils and GLIBC patches are already com

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

Richard Guenther writes: >> 2011-06-01 Richard Guenther >> >> * tree.c (build_common_tree_nodes): Also initialize size_type_node. >> Call set_sizetype from here. >> >> c-family/ >> * c-common.c (c_common_nodes_and_builtins): Do not set >> size_type_node or call set_s

[v3] Small tweak to std::move_if_noexcept

Hi, noticed while working on std::vector. Tested x86_64-linux, committed. Paolo. /// 2011-06-06 Paolo Carlini * include/bits/move.h (move_if_noexcept): Use __and_ and __not_. Index: include/bits/move.h =

[testsuite] Compile gfortran.dg/graphite/vect-pr40979.f90 with -msse2 on x86 (PR tree-optimization/48497)

As described in the PR, gfortran.dg/graphite/vect-pr40979.f90 FAILs on Solaris 8 and 9/x86, which defaults to -march=pentiumpro: FAIL: gfortran.dg/graphite/vect-pr40979.f90 -O scan-tree-dump-times vect "vectorized 1 loops" 1 The dump contains 'vectorized 0 loops' instead. The test passes with

Re: [lto] Unify decl and type registration (issue4515186)

On Mon, Jun 6, 2011 at 10:27, Diego Novillo wrote: > On Mon, Jun 6, 2011 at 04:50, Richard Guenther wrote: > >> I'd have it done in the loop that computes canonical types, at this >> place we do not gain the advantage that the decl register functions >> get completely fixed up trees. > > Hm, yes,

Re: [PATCH] cleanup: local label prefix

On 05/20/11 11:13, Greta Yorsh wrote: This patch replaces a hardcoded "." of local label prefix with LOCAL_LABEL_PREFIX macro in the ARM backend (generating a "trampoline" in thumb mode). The patch also contains a new test to make sure that the local label ".Ltrampoline_start:" is generated corr

Re: [google]Skip target-libiberty for arm*-*-linux-androideabi (issue4564050)

The trunk version has been approved and committed as r174710. Backport it to google/main. The google/main version has the same logic but is slightly different since trunk has a different code structure here. OK for google/main? 2011-06-06 Jing Yu Backport trunk r174710: * con

Re: [PATCH] PR fortran/49268

On 6/2/11 2:42 PM, "Langton, Asher" wrote: >On 6/2/11 2:11 PM, "Steve Kargl" wrote: >>On Thu, Jun 02, 2011 at 01:59:03PM -0700, Langton, Asher wrote: >>> This patch fixes a bug where the wrong code was generated for >>>assumed-size >>> Cray pointees. The fix is simple, but a bit ugly. Bootstrappe

Re: [google]Skip target-libiberty for arm*-*-linux-androideabi (issue4564050)

OK. thanks Carrot On Tue, Jun 7, 2011 at 1:09 AM, wrote: > The trunk version has been approved and committed as r174710. Backport > it to google/main. The google/main version has the same logic but is > slightly different since trunk has a different code structure here. OK > for google/main? >

[PATCH, libstdc++-v3] Add newlib specific ctype_members.cc

Hi, This patch fixes an issue in the ctype implementation when the newlib C library is used. The generic version of ctype::_M_convert_to_wmask() in config/locale/generic/ctype_members.cc assumes that a character type mask is either a bitmask with only 1 bit set or a bitwise-OR result of other cha

RE: [PATCH, libstdc++-v3] Add newlib specific ctype_members.cc

This time with the patch attached. Yufeng > -Original Message- > From: Yufeng Zhang [mailto:yufeng.zh...@arm.com] > Sent: 06 June 2011 18:31 > To: gcc-patches@gcc.gnu.org > Subject: [PATCH, libstdc++-v3] Add newlib specific ctype_members.cc > > Hi, > > This patch fixes an issue in the c

Re: objc/objc++: switch all testcases to Modern Objective-C runtime API

On 6 Jun 2011, at 18:21, Nicola Pero wrote: This patch switches all the testcases in the ObjC/ObjC++ testsuite to use the Modern Objective-C runtime API when executing with the GNU runtime. This will allow me to complete removing the Traditional Objective-C runtime API from libobjc. :-)

Re: [PATCH] Fix fold_stmt ICE (PR c++/49264)

On Mon, Jun 06, 2011 at 01:41:14PM +0200, Richard Guenther wrote: > > I think NULL new_stmt would have the advantage that we wouldn't duplicate > > the complex code looping through all kinds of clones. > > Yeah, I'd prefer that variant. Honza? Ok, after further discussions with Honza on IRC here

[committed] Fix ICE on vector shift in dwarf2out (PR debug/49294)

Hi! Trying to create vector zero_extend is wrong, while mem_loc_descriptor will for now fail for vector modes, the code attempted to expand zero_extend anyway. Fixed thusly, committed to trunk as obvious after bootstrapping/regtesting on x86_64-linux and i686-linux. 2011-06-06 Jakub Jelinek

Re: [PATCH] PR fortran/49268

On Mon, Jun 06, 2011 at 10:14:39AM -0700, Langton, Asher wrote: > On 6/2/11 2:42 PM, "Langton, Asher" wrote: > >On 6/2/11 2:11 PM, "Steve Kargl" wrote: > >>On Thu, Jun 02, 2011 at 01:59:03PM -0700, Langton, Asher wrote: > >>> This patch fixes a bug where the wrong code was generated for > >>>assu

[testsuite] Fix cdtor-1.C testcase (PR testsuite/49288)

Hi! As usually, -g -dA output differs between targets that can't merge debug strings and those that do. Instead of writing regexps to match both, this patch does what most other testcases do. Additionally, it fixes the testcase also for -gdwarf-4 defaulting compilers. Regtested on x86_64-linux

[committed] Fix endless loop in dwarf2out.c (PR debug/49262)

Hi! I forgot to decrement the counter, but given that nobody has hit it until now it is possible that no frontend leaves RANGE_EXPR indexes in CONSTRUCTORs around till late. varasm.c seems to handle them though, so fixed thusly, committed to trunk as obvious. 2011-06-06 Jakub Jelinek

[PATCH] Fix -fprofile-use optimization of a noreturn indirect call (PR gcov-profile/49299)

Hi! When an indirect noreturn call has a likely call target, value-prof.c ICEs on it, as there is no fallthru edge after the call (nor a join bb). Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux, ok for trunk/4.6? 2011-06-06 Jakub Jelinek PR gcov-profile/49299

[Design notes, RFC] Address-lowering prototype design (PR46556)

It's been some time since I last posted about the address lowering issue from PR46556 (http://gcc.gnu.org/bugzilla/show_bug.cgi?id=46556). I've had a basic prototype in place for some time, but I ran into some issues that initially caused performance regressions; I've had to jump through several h

Re: objc/objc++: switch all testcases to Modern Objective-C runtime API

On Jun 6, 2011, at 10:21 AM, Nicola Pero wrote: > This patch switches all the testcases in the ObjC/ObjC++ testsuite to use the > Modern Objective-C runtime API when executing with the GNU runtime. > OK to commit ? Ok, from me with one possible exception, I'd like Iain or Jack to weigh in on weth

Re: [pph] Clean up PPH tests (issue4572042)

On 6/6/11, Diego Novillo wrote: > So, I'm getting this: > > Running /home/dnovillo/pph/svn/src/gcc/testsuite/g++.dg/pph/pph.exp ... > XPASS: g++.dg/pph/c120060625-1.cc -I. (test for bogus messages, line ) > XPASS: g++.dg/pph/c1eabi1.cc -I. (test for bogus messages, line ) > XPASS: g++.dg/pph/x

Re: [lto] Unify decl and type registration (issue4515186)

On Mon, Jun 6, 2011 at 13:02, Diego Novillo wrote: > On Mon, Jun 6, 2011 at 10:27, Diego Novillo wrote: >> On Mon, Jun 6, 2011 at 04:50, Richard Guenther wrote: >> >>> I'd have it done in the loop that computes canonical types, at this >>> place we do not gain the advantage that the decl registe

Re: [PATCH] gimple_val_nonnegative_real_p (PR46728 patch 7 of 7)

Hi Richard, Here's the reworked patch addressing the noted concerns. Regtested for powerpc-linux. OK for trunk? Thanks, Bill 2011-06-06 Bill Schmidt PR tree-optimization/46728 * tree-ssa-math-opts.c (gimple_expand_builtin_pow): Change FIXME to use gimple_val_nonneg

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

Please take a look at the revised one. Thanks, David On Mon, Jun 6, 2011 at 4:22 AM, Richard Guenther wrote: > On Wed, Jun 1, 2011 at 7:24 PM, Xinliang David Li wrote: >> The attached is the split #1 patch that enhances -fenable/disable. >> >> Ok after testing? > > I expect the testcases will

Fix for PR obj-c++/48275 ("getter=namespace failing with .mm")

This patch fixes PR obj-c++/48275. It's a routine parser ingenuity. OK to commit ? Thanks Index: testsuite/ChangeLog === --- testsuite/ChangeLog (revision 174657) +++ testsuite/ChangeLog (working copy) @@ -1,3 +1,9 @@ +2011-06-06

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

This is the patch with max id removed. Thanks, David On Mon, Jun 6, 2011 at 9:00 AM, Xinliang David Li wrote: > On Mon, Jun 6, 2011 at 4:38 AM, Richard Guenther > wrote: >> On Thu, Jun 2, 2011 at 9:12 AM, Xinliang David Li wrote: >>> This is the version of the patch that walks through pass li

Re: [patch, fortran] Some more TRIM optimizations

I wrote: Hello world, the attached patch extends removing trailing TRIMs in assignments for cases like a // trim(b). Regression-tested. OK for trunk? Thomas This time with the test case corrected (cleanup of the *.original file) and a more meaningful Subject line. OK? Thomas 2011-

Re: objc/objc++: switch all testcases to Modern Objective-C runtime API

Nicola, On x86_64-apple-darwin10 I have the following failures with -m32 FAIL: obj-c++.dg/threedotthree-abi-1.mm -fnext-runtime (test for excess errors) FAIL: obj-c++.dg/torture/strings/const-str-10.mm -O0 -fnext-runtime (test for excess errors) FAIL: obj-c++.dg/torture/strings/const-str-10.m

C++ PATCH for c++/49298 (C++0x regression with pointer to member)

potential_constant_expression_1 didn't know how to deal with a pointer-to-member expression in a template. Tested x86_64-pc-linux-gnu, applied to trunk. commit a07ee89ff40f73dcb1fc11fb66931ee79bf8a2d9 Author: Jason Merrill Date: Mon Jun 6 12:05:39 2011 -0400 PR c++/49298 * semantic

Re: [PATCH] Fix PR debug/49130

OK. Jason

Re: objc/objc++: switch all testcases to Modern Objective-C runtime API

On 6 Jun 2011, at 21:07, Nicola Pero wrote: On x86_64-apple-darwin10 I have the following failures with -m32 darwin10 is Mac OS X 10.6, right ? I have access to that. So, how do you test with -m32 ? I thought the testsuite would do that (test both with -m32 and -m64 if they are availabl

  1   2   >