Re: [PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Dominique Dhumieres
> Ok, slightly updated. How about this? ... It did not work either at -m64, but the following one seems to work (manual testing): --- /opt/gcc/_gcc_clean/gcc/testsuite/gcc.dg/stack-usage-1.c2011-03-28 20:27:57.0 +0200 +++ /opt/gcc/work/gcc/testsuite/gcc.dg/stack-usage-1.c 2011-03-2

[gc-improv] [PATCH] Separate function RTL memory

2011-03-28 Thread Laurynas Biveinis
This is a first attempt to separate function RTL and make it default. This patch is enough for --disable-bootstrap --enable-languages=c to complete on Linux x64, but there still remains some RTL that is allocated on function obstack when it should be permanent instead. The patch disables sharin

[PATCH] gitignore: ignore site.{bak,exp} treewide

2011-03-28 Thread Mike Frysinger
These files are never checked into cvs, and are generated by most testsuites, so ignore them. Signed-off-by: Mike Frysinger 2011-03-28 Mike Frysinger * .gitignore: Ignore site.bak and site.exp. --- .gitignore |2 ++ 1 files changed, 2 insertions(+), 0 deletions(-) diff --git a/

Re: [PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Peter Bergner
On Mon, 2011-03-28 at 23:50 +0200, Dominique Dhumieres wrote: > > Just guessing at the numbers, does the following patch fix the > > failures for you? > > As I am currently bootstrapping gcc, I cannot regtest right now, but running > the > test manually I get 256 at -m32 (I guess this is right) w

Re: [C++0x] Range-based for statements and ADL

2011-03-28 Thread Rodrigo Rivas
Hi again. Here it is my first try at this. I have changed the list to gcc-patches, I don't know if cross post would be correct. Please, note that this patch is not finished: the new test cases are still missing, and expect format mistakes, misspellings and the like... A few comments: 1. I'm not

Re: [committed] Fix PA ior expanders

2011-03-28 Thread John David Anglin
> You've already got an ior_operand, which has a redundant test for CONST_INT. > Actually, this is true of quite a few of the PA predicates... Quite true. The predicates were originally derived from the code in pa.md without a significant review. I'll look at implementing your suggestion. Thank

C++ PATCH for c++/48313 (T&& deduction from function arg)

2011-03-28 Thread Jason Merrill
We were deducing the wrong template argument when passing a function template-id to a T&& parameter, because the code for resolving an overloaded function based on a parameter type wasn't passing in an argument expression to maybe_adjust_parameter_types_for_deduction. The easy fix was to check

Go patch committed: Permit copying hidden fields in method receiver

2011-03-28 Thread Ian Lance Taylor
The Go language now permits copying hidden fields when passing a value to a method receiver. This patch implements that in gccgo. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r ea8e140e8fab go/expressions.cc --- a/go/expressions.cc Mon Mar 28 16

[wwwdocs] PATCH for Re: GCC 4.6.0 Released

2011-03-28 Thread Gerald Pfeifer
Committed. Gerald Index: index.html === RCS file: /cvs/gcc/wwwdocs/htdocs/index.html,v retrieving revision 1.781 diff -u -3 -p -r1.781 index.html --- index.html 25 Mar 2011 19:56:41 - 1.781 +++ index.html 28 Mar 2011 23:31

Go patch committed: Better error message for type switch

2011-03-28 Thread Ian Lance Taylor
This patch to the Go frontend generates a better error message when an invalid variable name is used in a switch statement. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 9f4bdd962616 go/parse.cc --- a/go/parse.cc Mon Mar 28 15:57:46 2011 -0700

Go patch committed: Better error message for missing if condition

2011-03-28 Thread Ian Lance Taylor
This patch to the Go frontend gives a better error message when the condition is omitted from an if statement. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r a05deef47b01 go/parse.cc --- a/go/parse.cc Mon Mar 28 15:48:35 2011 -0700 +++ b/go/par

Go patch committed: Better error message for send as value

2011-03-28 Thread Ian Lance Taylor
This patch to gccgo makes it emit better a error message when using the now obsolete syntax of a send expression as a value. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 60aecfe6ad83 go/parse.cc --- a/go/parse.cc Mon Mar 28 14:49:45 2011 -070

[Patch, Fortran, OOP] PR 48095: Invalid assignment to procedure pointer component not rejected

2011-03-28 Thread Janus Weil
Hi all, the attached patch fixes a problem with assignments to procedure pointer components: The checking for such assignments did only work within one module, but not across modules. The reason for this was that the "ts.interface" field was not being written to the module file. The patch fixes t

Re: [committed] Fix PA ior expanders

2011-03-28 Thread Richard Henderson
On 03/28/2011 03:10 PM, Richard Henderson wrote: > On 03/27/2011 03:17 PM, John David Anglin wrote: >> +(define_predicate "reg_or_ior_operand" >> + (match_code "subreg,reg,const_int") >> +{ >> + return (register_operand (op, mode) >> + || (GET_CODE (op) == CONST_INT && ior_mask_p (INTVAL (op

Re: [committed] Fix PA ior expanders

2011-03-28 Thread Richard Henderson
On 03/27/2011 03:17 PM, John David Anglin wrote: > +(define_predicate "reg_or_ior_operand" > + (match_code "subreg,reg,const_int") > +{ > + return (register_operand (op, mode) > + || (GET_CODE (op) == CONST_INT && ior_mask_p (INTVAL (op; > +}) > + Better written, IMO, as (ior (match

Re: [google] Allow relative profile paths. (issue4280074)

2011-03-28 Thread Diego Novillo
Committed to google/main rev 171635.

Go patch committed: Better errors for missing chan element type

2011-03-28 Thread Ian Lance Taylor
This patch to the Go frontend gives better error messages for the common error of omitting a channel element type. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 5e8176633433 go/parse.cc --- a/go/parse.cc Mon Mar 28 14:34:54 2011 -0700 +++ b/go

Re: [PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Dominique Dhumieres
> Just guessing at the numbers, does the following patch fix the > failures for you? As I am currently bootstrapping gcc, I cannot regtest right now, but running the test manually I get 256 at -m32 (I guess this is right) while I get 272 at -m64 (so the test will fail). Dominique

Go patch committed: Pointer to interface type has no methods

2011-03-28 Thread Ian Lance Taylor
In the patch I just committed, I overlooked that a pointer to a named interface type has no methods, and therefore that you can not use such a type with a method expression. This patch corrects that oversight. To make the error more clear, I added a '*' to the error output for a pointer type. Bo

Interlacing switch labels and compound statements

2011-03-28 Thread Bruce Korb
I stumbled over this code using a source code analyzer. It incorrectly assumed that switch labels cannot be inserted into random places inside of enclosed compound statements. It correctly assumes that you shouldn't be doing that. :) I propose making this change solely for aesthetics: static voi

Re: [PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Peter Bergner
On Mon, 2011-03-28 at 20:48 +0200, Dominique Dhumieres wrote: > The test gcc.dg/stack-usage-1.c already failed on powerpc-apple-darwin9 > before you change with: > > FAIL: gcc.dg/stack-usage-1.c scan-file foo\t(256|264)\tstatic > > and is still failing the same way after it. > > [karma] f90/bug%

Go patch committed: Support method expressions for interface types

2011-03-28 Thread Ian Lance Taylor
This patch to the Go frontend adds support for method expressions for interface types. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 0ed415b7ab0e go/expressions.cc --- a/go/expressions.cc Mon Mar 28 14:18:39 2011 -0700 +++ b/go/expressions.cc

Go patch committed: Don't check for dup interface/named method

2011-03-28 Thread Ian Lance Taylor
A named interface type may not have methods. Therefore, there is no reason to check whether the same name is used for an interface method and a regular method. This patch removes this useless test. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r

Re: [patch] Fix bogus source locations with optimization

2011-03-28 Thread Eric Botcazou
> Oh, I didn't realize curr_location is of type location_t. I'm not very > familiar with these bits. Btw, insn_locators_alloc initializes it with -1 > while it should probably initialize it with UNKNOWN_LCOATION as well. Indeed, will adjust them afterwards. > So it looks like the patch is perfe

[pph] Tidy naming scheme for some functions (issue4291074)

2011-03-28 Thread Diego Novillo
No functional changes. Rename some functions to make them consistent with the other functions in the file. cp/ChangeLog.pph. 2011-03-28 Diego Novillo * pph.c (pph_write_format): Rename from write_pph_format. (pph_write_namespace): Rename from write_pph_namespace. (pp

Re: [patch, fortran] Extend character optimization to LLE and friends

2011-03-28 Thread Thomas Koenig
Am 28.03.2011 22:07, schrieb Richard Henderson: Style nit: remove the unreachable break statements. Fixed, thanks. Any further comments? Thomas

Re: PATCH: Split AVX 32byte unalignd load/store

2011-03-28 Thread H.J. Lu
On Mon, Mar 28, 2011 at 1:17 PM, Richard Henderson wrote: > On 03/27/2011 11:57 AM, H.J. Lu wrote: >> +    { "-mavx256-split-unaligned-stroe",       >> MASK_AVX256_SPLIT_UNALIGNED_STORE}, > > Typo. > I checked in this. Thanks. H.J. --- Index: ChangeLog =

Re: PATCH: Split AVX 32byte unalignd load/store

2011-03-28 Thread Richard Henderson
On 03/27/2011 11:57 AM, H.J. Lu wrote: > +{ "-mavx256-split-unaligned-stroe", > MASK_AVX256_SPLIT_UNALIGNED_STORE}, Typo. r~

Re: [patch, fortran] Extend character optimization to LLE and friends

2011-03-28 Thread Richard Henderson
On 03/28/2011 11:50 AM, Thomas Koenig wrote: > +case GFC_ISYM_LLE: > + return optimize_comparison (e, INTRINSIC_LE); > + break; Style nit: remove the unreachable break statements. r~

Re: [google] Allow relative profile paths. (issue4280074)

2011-03-28 Thread Diego Novillo
On Mon, Mar 28, 2011 at 14:14, wrote: > On 2011/03/28 17:59:32, Diego Novillo wrote: >> >> I'm confused.  why did you take !HAS_DRIVE_SPEC() out? > > Earlier in the function, the following code makes sure we don't start > the fname with a DRIVE_SPEC. > >      if (prefix_length != 0 && HAS_DRIVE_

[patch, fortran] Extend character optimization to LLE and friends

2011-03-28 Thread Thomas Koenig
Hello world, the attached patch extends the character optimizations to the lexical comparison functions (LLE and friends). Regression-tested. OK for trunk? Thomas 2010-03-28 Thomas Koenig * frontend-passes (optimize_lexical_comparison): New function. (optimize_e

Re: [PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Dominique Dhumieres
Peter, The test gcc.dg/stack-usage-1.c already failed on powerpc-apple-darwin9 before you change with: FAIL: gcc.dg/stack-usage-1.c scan-file foo\t(256|264)\tstatic and is still failing the same way after it. [karma] f90/bug% grep foo stack-usage-1.su stack-usage-1.c:54:5:foo272 sta

Re: PING^3: Reducing the overhead of dwarf2 location tracking

2011-03-28 Thread Richard Henderson
On 03/28/2011 02:00 AM, Richard Sandiford wrote: > Third ping for this patch for PR48190 (4.5+): > > http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00255.html > > It fixes a source of quadratic memory use while generating dwarf2 > var-tracking information. Ok. r~

Re: [patch] Split RTL constant/copy propagation off from gcse.c to new cprop.c

2011-03-28 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/28/11 10:25, Steven Bosscher wrote: > Hello, > > This patch splits the CPROP pass (constant/copy propagation for RTL) > out of gcse.c into a new file cprop.c. Originally all the > "Muchnick-passes" (GCSE, PRE, CPROP, HOIST, store motion, ...) li

libgo patch committed: Fix fd_select.go

2011-03-28 Thread Ian Lance Taylor
This patch fixes fd_select.go for the changes in FD handling in the latest libgo update. The code in libgo now takes advantage of the fact that if you change the set of descriptors used by epoll, it is not necessary to wake up the thread waiting in epoll_wait. That is not true of select: if you c

Re: Improve jump threading #1 of N

2011-03-28 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/26/11 11:43, Richard Guenther wrote: > >> Looks good to me apart from not using gsi_start_nondebug_bb in >> thread_around_empty_block. Ah. I wasn't aware of gsi_start_nondebug_bb. I updated the patch to use it, ran another bootstrap and regres

libgo patch committed: Rename fd_rtems.go to fd_select.go

2011-03-28 Thread Ian Lance Taylor
I renamed fd_rtems.go to fd_select.go in libgo, since the file is used for operating systems other than RTEMS. Bootstrapped on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 381b9fce29e2 libgo/Makefile.am --- a/libgo/Makefile.am Mon Mar 28 11:07:46 2011 -0700 +++ b/libgo/Makefile.

Re: [PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Peter Bergner
On Mon, 2011-03-28 at 10:24 -0700, Mike Stump wrote: > On Mar 28, 2011, at 10:12 AM, Peter Bergner wrote: > > I'd like to ping this test suite patch: > > > > [PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc > > > >http://gcc.gnu.org/ml/gcc-patches/2011-02/msg00077.html

go patch committed: Fix useless assertion

2011-03-28 Thread Ian Lance Taylor
This patch to the Go frontend fixes a useless assertion to actually do something useful. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian diff -r 1cab38f05b0f go/expressions.cc --- a/go/expressions.cc Sun Mar 27 12:11:31 2011 -0700 +++ b/go/expressions.c

Re: [google] Allow relative profile paths. (issue4280074)

2011-03-28 Thread dnovillo
OK. A nit and a question below. Diego. http://codereview.appspot.com/4280074/diff/1/gcc/doc/invoke.texi File gcc/doc/invoke.texi (right): http://codereview.appspot.com/4280074/diff/1/gcc/doc/invoke.texi#newcode gcc/doc/invoke.texi:: and its related options. Both absolute and relative

Re: fix for 48208 and 48260 on darwin

2011-03-28 Thread Mike Stump
On Mar 27, 2011, at 8:55 PM, Christian Schüler wrote: > please review the following patch (and besides, bear with me as this is the > first patch proposal from me). For gcc 4.5 and earlier it was possible to > configure xcode via a modified xcplugin to use the newer gcc directly (yes, > the -arc

Re: [PATCH] ENTRY_VALUE fixes (PR debug/48203)

2011-03-28 Thread Richard Henderson
On 03/28/2011 10:32 AM, Jakub Jelinek wrote: > Say vt_add_function_parameter is called for first parameter in %rdi, > cselib_lookup_from_insn gives us VALUE 2:2 for it. We add > (entry_value:DI (reg:DI %rdi)) to list of locations for that VALUE. > The second cselib_lookup_from_insn gives us VALUE

Re: [PATCH] ENTRY_VALUE fixes (PR debug/48203)

2011-03-28 Thread Jakub Jelinek
On Mon, Mar 28, 2011 at 09:58:38AM -0700, Richard Henderson wrote: > > * var-tracking.c (vt_add_function_parameter): Ensure cselib_lookup > > on ENTRY_VALUE is able to find the canonical parameter VALUE. > > I don't really understand what's going on here. Whatever it is, it > could defini

Re: [PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Mike Stump
On Mar 28, 2011, at 10:12 AM, Peter Bergner wrote: > I'd like to ping this test suite patch: > > [PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc > >http://gcc.gnu.org/ml/gcc-patches/2011-02/msg00077.html Normally, I'd prefer the target folks to approve these sorts, b

[PING][PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc

2011-03-28 Thread Peter Bergner
I'd like to ping this test suite patch: [PATCH, testsuite] Update gcc.dg/stack-usage-1.c SIZE values for powerpc http://gcc.gnu.org/ml/gcc-patches/2011-02/msg00077.html Peter

[google] Allow relative profile paths. (issue4280074)

2011-03-28 Thread Martin Thuresson
This patch should go into google/main so that relative paths can be used with FDO. Bootstraped without regression and verified that my benchmarks correctly used the generated profiles. Original patch to trunk included a regression test, but I noticed last week that it failed in certain test scena

Re: [PATCH] ENTRY_VALUE fixes (PR debug/48203)

2011-03-28 Thread Richard Henderson
On 03/20/2011 05:57 AM, Jakub Jelinek wrote: > * cfgexpand.c (expand_debug_expr) : Only > create ENTRY_VALUE if incoming or address of incoming's MEM > is a hard REG. > * dwarf2out.c (mem_loc_descriptor): Don't emit > DW_OP_GNU_entry_value of DW_OP_fbreg. Ok. >

[testsuite] Generalize REs in gcc.dg/debug/dwarf2/pr47939-[1-4].c (PR testsuite/48238)

2011-03-28 Thread Rainer Orth
As described in the PR, four new DWARF-2 tests FAIL on Solaris 2 with Sun as, IRIX 6.5 with gas, and Darwin. Dominique provided the following patch. Tested with the appropriate runtest invocations on i386-pc-solaris2.11 with Sun as and gas and mips-sgi-irix6.5 by myself, and x86_64-apple-darwin10

C++ PATCHes to fix ref binding to array temporaries

2011-03-28 Thread Jason Merrill
In discussion of core issue 1232 we realized that we already do allow array temporaries in some cases (primarily as subobjects of class temporaries), so we ought to handle them consistently. The first patch fixes direct binding to array prvalues, which has been specified for a while. The sec

C++ PATCHes to restrictions on constexpr functions

2011-03-28 Thread Jason Merrill
The resolution of core issue 898 ratifies our current behavior of allowing typedefs and static_assert in constexpr functions. It also allows using decls/directives. We discussed allowing unnecessary braces, but decided not to. So these patches 1) allow using and 2) prohibit extra compound-st

Re: Use gen_int_mode in expand_builtin_memset_args

2011-03-28 Thread Richard Guenther
On Mon, Mar 28, 2011 at 5:47 PM, Richard Sandiford wrote: > This patch fixes a case where expand_builtin_memset_args was creating > a zero- rather than sign-extended character-sized CONST_INT.  The problem > showed up on s390, where a character was being passed as (const_int 171) > rather than (co

C++ PATCH to fix constexpr/noexcept bug

2011-03-28 Thread Jason Merrill
We need to call cxx_constant_value after converting to bool, in case the conversion uses a conversion operator. Tested x86_64-pc-linux-gnu, applying to trunk and 4.6. commit a3fde3e7dcdc6b4f0b92bb273f4267c6bf8f6de0 Author: Jason Merrill Date: Sun Mar 27 21:51:35 2011 -0400 * except.c

Use gen_int_mode in expand_builtin_memset_args

2011-03-28 Thread Richard Sandiford
This patch fixes a case where expand_builtin_memset_args was creating a zero- rather than sign-extended character-sized CONST_INT. The problem showed up on s390, where a character was being passed as (const_int 171) rather than (const_int -85). Tested on x86_64-linux-gnu. Andreas also confirms t

[committed/vms] fix ice for vms/ia64

2011-03-28 Thread Tristan Gingold
Hi, gcc crashes during cross-build of libiberty due to a bad handling of promote_function_mode. This patch fixes that. Committed on trunk. Tristan. 2011-03-28 Tristan Gingold * config/ia64/ia64.c (ia64_promote_function_mode): Fix promotion when for_return is 2. Index: gcc/

Re: C++ PATCH for c++/48289 (-pedantic breaks std::move)

2011-03-28 Thread Jason Merrill
On 03/27/2011 08:17 PM, Eric Botcazou wrote: Tested x86_64-pc-linux-gnu, applied to trunk and 4.5, will apply to 4.6 after 4.6.0. The test doesn't pass on the 4.5 branch: FAIL: g++.dg/cpp0x/move1.C (test for excess errors) Good point. I guess I won't bother trying to fix this in 4.5. Rev

Re: [PATCH] Better .debug_aranges fix (PR debug/48253, take 3)

2011-03-28 Thread Jason Merrill
On 03/28/2011 10:34 AM, Jakub Jelinek wrote: So perhaps if (fde->dw_fde_second_begin&& fde->dw_fde_switch_cfi == NULL) { /* If the first partition contained no CFI adjustments, the CIE opcodes apply to the whole first partition. */ *list_tail = new_loc_list (build

Re: [PATCH] Better .debug_aranges fix (PR debug/48253, take 3)

2011-03-28 Thread Jakub Jelinek
On Mon, Mar 28, 2011 at 10:23:08AM -0400, Jason Merrill wrote: > On 03/25/2011 01:01 PM, Jakub Jelinek wrote: > >+if (node == loc_list->last_before_switch > >+&& (node != loc_list->first || loc_list->first->next) > >+&& current_function_decl) > > Let's store this te

Re: [h8300] Hookize FUNCTION_VALUE, LIBCALL_VALUE and FUNCTION_VALUE_REGNO_P

2011-03-28 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/27/11 14:07, Anatoly Sokolov wrote: > Hello. > > This patch removes obsolete FUNCTION_VALUE, LIBCALL_VALUE and > FUNCTION_VALUE_REGNO_P macros from H8300 back end in the GCC and introduces > equivalent TARGET_FUNCTION_VALUE, TARGET_LIBCALL_VAL

Re: [H8300] Remove ASM_OUTPUT_BSS

2011-03-28 Thread Jeff Law
-BEGIN PGP SIGNED MESSAGE- Hash: SHA1 On 03/27/11 14:32, Anatoly Sokolov wrote: > Hi. > > In config/h8300/h8300.h file both ASM_OUTPUT_BSS and ASM_OUTPUT_ALIGNED_BSS > macros is defined, but the ASM_OUTPUT_BSS macro is not used when > ASM_OUTPUT_ALIGNED_BSS is defined. This patch remo

Re: [PATCH] Better .debug_aranges fix (PR debug/48253, take 3)

2011-03-28 Thread Jason Merrill
On 03/25/2011 01:01 PM, Jakub Jelinek wrote: + if (node == loc_list->last_before_switch + && (node != loc_list->first || loc_list->first->next) + && current_function_decl) Let's store this test in a local variable rather than repeat it a few lines down.

Add support for DragonFlyBSD target to top level configure

2011-03-28 Thread Nick Clifton
Hi Guys, I am propagating the attached patch from the binutils source tree. It adds support for the DragonFlyBSD target to the top level configure script. Cheers Nick ./ChangeLog 2011-03-26 John Marino * configure.ac: Add support for *-*-dragonfly* * configure: Regen

PATCH: PR boehm-gc/48299: FAIL: boehm-gc.c/thread_leak_test.c

2011-03-28 Thread H.J. Lu
Hi, GC_LINUX_THREADS is redefined. This patch fixes it. OK for trunk? Thanks. H.J. diff --git a/boehm-gc/ChangeLog b/boehm-gc/ChangeLog index f6b1865..557167c 100644 --- a/boehm-gc/ChangeLog +++ b/boehm-gc/ChangeLog @@ -1,3 +1,9 @@ +2011-03-28 H.J. Lu + + PR boehm-gc/48299 +

Re: [patch] plug mem leak in gcse

2011-03-28 Thread Richard Guenther
On Mon, Mar 28, 2011 at 3:29 PM, Steven Bosscher wrote: > Hi, > > The bitmap reg_set_bitmap is allocated in alloc_gcse_mem, but it is > never freed. The foul smell of a mem leak... > > Plugged thus. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk? Ok. Thanks, Richard. > Ciao! > St

[patch] plug mem leak in gcse

2011-03-28 Thread Steven Bosscher
Hi, The bitmap reg_set_bitmap is allocated in alloc_gcse_mem, but it is never freed. The foul smell of a mem leak... Plugged thus. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK for trunk? Ciao! Steven * gcse.c (free_gcse_mem): Free reg_set_bitmap. Index: gcse.c =

PATCH: PR testsuite/48276: FAIL: gcc.target/i386/pr47502-2.c

2011-03-28 Thread H.J. Lu
Hi, gcc.target/i386/pr47502-2.c uses EBX which doesn't work with PIC. I am checking in this patch as an obvious fix. H.J. --- diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index a9f70d0..53424d2 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2

[PING] Fix PR46399 - missing mode promotion for libcall args

2011-03-28 Thread Andreas Krebbel
This fixes a wrong code generation bug for sw DFP: http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00141.html Bye, -Andreas-

Re: Use LINK_PLUGIN_SPEC on Darwin (PR target/48245)

2011-03-28 Thread Dominique Dhumieres
Rainer, Ignore my previous comment: I just saw the quotes on the other changed lines. Thanks for the commit. Dominique

Re: Use LINK_PLUGIN_SPEC on Darwin (PR target/48245)

2011-03-28 Thread Dominique Dhumieres
Rainer, In your commit you did not quote LINK_PLUGIN_SPEC. IIRC this breaks bootstrapping. What I have tested is: " LINK_PLUGIN_SPEC " \ TIA Dominique

Re: More toplevel configure.ac pruning

2011-03-28 Thread Paolo Bonzini
On 03/28/2011 01:34 PM, Joseph S. Myers wrote: 2011-03-28 Joseph Myers * configure.ac (i[[3456789]]86-*-msdosdjgpp*): Don't disable libffi on host. (x86_64-*-mingw*, i[[3456789]]86-*-mingw32*): Don't disable newlib on host. (c54x*-*-* | tic54x-*-*): Don't

Re: More toplevel configure.ac pruning

2011-03-28 Thread Joseph S. Myers
On Fri, 25 Mar 2011, Paolo Bonzini wrote: > > Remark: disabling "newlib" (as opposed to target-newlib) for a host seems > > odd, as does disabling "libffi". So maybe the code disabling directories > > for host can be simplified even more. > > Yes, newlib and libffi should never be built as host

Re: [patch ada]: Fix MSG_WAITALL handling for windows native targets

2011-03-28 Thread Kai Tietz
2011/3/28 Arnaud Charlet : > Kai, > > Here are Thomas comments on your patch: > > -- >> Split submitted patch from thread "[patch ada]: Fix issues about >> multilib build of native windows and handle MSG_WAITALL for windows >> native targets" >> into separate ones. >> >> ChangeLog gcc/ada (ada_wt.t

Re: [patch] Fix bogus source locations with optimization

2011-03-28 Thread Richard Guenther
On Mon, Mar 28, 2011 at 1:08 PM, Eric Botcazou wrote: >> This overloads UNKNOWN_LOCATION for both insn_locator and >> source_location, I don't think this is the best idea.  It'll eventually >> break when compiling with C++ anyway. > > Could you elaborate?  UNKNOWN_LOCATION isn't used for INSN_LOCA

Re: [patch] Fix bogus source locations with optimization

2011-03-28 Thread Eric Botcazou
> This overloads UNKNOWN_LOCATION for both insn_locator and > source_location, I don't think this is the best idea. It'll eventually > break when compiling with C++ anyway. Could you elaborate? UNKNOWN_LOCATION isn't used for INSN_LOCATOR at all thanks for the curr_insn_locator hunk. > The exp

Re: [patch ada]: Fix issues about multilib build of native windows

2011-03-28 Thread Kai Tietz
2011/3/28 Arnaud Charlet : >> Split submitted patch from thread "[patch ada]: Fix issues about >> multilib build of native windows and handle MSG_WAITALL for windows >> native targets" >> into separate ones. >> >> ChangeLog gcc/ada (ada_w_multi.txt) >> >> 2011-03-28  Kai Tietz >> >>       * gcc-int

Use LINK_PLUGIN_SPEC on Darwin (PR target/48245)

2011-03-28 Thread Rainer Orth
When developing the patch to use the linker plugin only if the (plugin) linker supports it, I had neglected to check for targets that defined their own version of LD_COMMAND_SPEC. Of course, this couldn't go unpunished, since there are two of them: * Darwin has its own version in config/darwin.h.

Re: [patch ada]: Fix MSG_WAITALL handling for windows native targets

2011-03-28 Thread Arnaud Charlet
Kai, Here are Thomas comments on your patch: -- > Split submitted patch from thread "[patch ada]: Fix issues about > multilib build of native windows and handle MSG_WAITALL for windows > native targets" > into separate ones. > > ChangeLog gcc/ada (ada_wt.txt) > > 2011-03-28 Kai Tietz > >

Re: fix for 48208 and 48260 on darwin

2011-03-28 Thread Joseph S. Myers
On Mon, 28 Mar 2011, Christian Schüler wrote: > PR 48208 > * config/c.opt: Added 'Driver' to -F option This part of the changes is OK with the ChangeLog entry fixed (it should go in c-family/ChangeLog): PR driver/48208 * c.opt (-F): Add Driver. -- Joseph S. Myers j

Re: [patch ada]: Fix issues about multilib build of native windows

2011-03-28 Thread Arnaud Charlet
> Split submitted patch from thread "[patch ada]: Fix issues about > multilib build of native windows and handle MSG_WAITALL for windows > native targets" > into separate ones. > > ChangeLog gcc/ada (ada_w_multi.txt) > > 2011-03-28 Kai Tietz > > * gcc-interface/Makefile.in (SO_LIB): Handl

Re: [ARM] PR 47553: vld1q_lane_u8 & co. don't accept lanes >= 8

2011-03-28 Thread Richard Sandiford
Nick Clifton writes: > Hi Ramana, > >> I am tempted to consider this one for the other release branches since >> this is correcting a test in the interface for an intrinsic which has >> been wrong for a long time but would like the opinion of the other >> maintainers about this. > > I think that I

Re: [build, lto] Only accept -fuse-linker-plugin if linker supports -plugin (PR lto/46944)

2011-03-28 Thread Richard Guenther
On Mon, Mar 28, 2011 at 12:14 PM, Rainer Orth wrote: > Richard Guenther writes: > >>> I think if the plugin linker is different from the normal linker we should >>> set HAVE_LTO_PLUGIN to 2. >> >> I think we should do the linker version checks which relate to linker-plugin >> use on the plugin-li

Re: [build, lto] Only accept -fuse-linker-plugin if linker supports -plugin (PR lto/46944)

2011-03-28 Thread Rainer Orth
Richard Guenther writes: >> I think if the plugin linker is different from the normal linker we should >> set HAVE_LTO_PLUGIN to 2. > > I think we should do the linker version checks which relate to linker-plugin > use on the plugin-linker instead. So if I specify a separate but known I'll see

Re: [patch] Fix bogus source locations with optimization

2011-03-28 Thread Richard Guenther
On Mon, Mar 28, 2011 at 11:00 AM, Eric Botcazou wrote: > Hi, > > when optimization is enabled, especially -O2 and above, you can have lines in > the assembly file with really bogus source location info.  The scenario is as > follows: an optimization pass at the Tree level (typically PRE) creates a

Re: [PATCH 1/4] Remove cgraph_node function and fixup all callers

2011-03-28 Thread Martin Jambor
Hi, On Fri, Mar 25, 2011 at 05:55:24PM +0100, Jan Hubicka wrote: > > > > Index: src/gcc/passes.c > > > > === > > > > --- src.orig/gcc/passes.c 2011-03-19 01:16:23.0 +0100 > > > > +++ src/gcc/passes.c2011-03-19 01:54:

Re: [patch ada]: Prevent warning about PERSONALITY_FUNCTION in raise-gcc.c

2011-03-28 Thread Kai Tietz
2011/3/28 Arnaud Charlet : >> Split submitted patch from thread "[patch ada]: Fix issues about >> multilib build of native windows and handle MSG_WAITALL for windows >> native targets" >> into separate ones. >> >> ChangeLog gcc/ada >> >> 2011-03-28  Kai Tietz >> >>       * raise-gcc.c (PERSONALITY_

Re: [patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets

2011-03-28 Thread Kai Tietz
2011/3/28 Arnaud Charlet : >> I split patch into two. ada_w_multi.txt contains just the >> gcc-interface/Makefile.in change and the ada_wt.txt the MSG_WAITALL >> patch. > > Thanks. Actually raise-gcc.c is yet another issue. > > Could you please resubmit under three different emails/threads your > 3

[patch ada]: Fix issues about multilib build of native windows

2011-03-28 Thread Kai Tietz
Hello, Split submitted patch from thread "[patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets" into separate ones. ChangeLog gcc/ada (ada_w_multi.txt) 2011-03-28 Kai Tietz * gcc-interface/Makefile.in (SO_LIB): Handle multilib

Re: [patch ada] Use Ada output of gnatbind for building tools

2011-03-28 Thread Eric Botcazou
> gnatbind can generate Ada or C output. The C output is being deprecated so > it shouldn't be used to build the tools. This patch updates the Makefiles > so that the Ada output is used instead. Great! Please mention PR ada/44431 in the ChangeLog. -- Eric Botcazou

Re: [patch ada] Use Ada output of gnatbind for building tools

2011-03-28 Thread Arnaud Charlet
> gnatbind can generate Ada or C output. The C output is being deprecated so it > shouldn't be used to build the tools. > This patch updates the Makefiles so that the Ada output is used instead. > > Ok for trunk ? OK, thanks.

Re: [patch ada]: Prevent warning about PERSONALITY_FUNCTION in raise-gcc.c

2011-03-28 Thread Arnaud Charlet
> Split submitted patch from thread "[patch ada]: Fix issues about > multilib build of native windows and handle MSG_WAITALL for windows > native targets" > into separate ones. > > ChangeLog gcc/ada > > 2011-03-28 Kai Tietz > > * raise-gcc.c (PERSONALITY_FUNCTION): Add prototype to >

[patch ada]: Fix MSG_WAITALL handling for windows native targets

2011-03-28 Thread Kai Tietz
Hello, Split submitted patch from thread "[patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets" into separate ones. ChangeLog gcc/ada (ada_wt.txt) 2011-03-28 Kai Tietz * g-socthi-mingw.adb (C_Recvmsg): Handle MSG_WAITALL for

[patch ada] Use Ada output of gnatbind for building tools

2011-03-28 Thread Tristan Gingold
Hi, gnatbind can generate Ada or C output. The C output is being deprecated so it shouldn't be used to build the tools. This patch updates the Makefiles so that the Ada output is used instead. Ok for trunk ? Tristan. gcc/ada 2011-03-28 Tristan Gingold * gcc-interface/Make-lang.in

[patch ada]: Prevent warning about PERSONALITY_FUNCTION in raise-gcc.c

2011-03-28 Thread Kai Tietz
Hello, Split submitted patch from thread "[patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets" into separate ones. ChangeLog gcc/ada 2011-03-28 Kai Tietz * raise-gcc.c (PERSONALITY_FUNCTION): Add prototype to prevent

Re: [RFC PATCH] Typed DWARF stack

2011-03-28 Thread Jakub Jelinek
On Fri, Mar 25, 2011 at 09:44:40AM -0700, Roland McGrath wrote: > It's been a while since I read Cary's proposal, and I am no longer > likely to do much work of my own in this area. So I'll just respond at > the high level. > > I like very much the essential notion of the stack being of typed > e

Re: [patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets

2011-03-28 Thread Arnaud Charlet
> I split patch into two. ada_w_multi.txt contains just the > gcc-interface/Makefile.in change and the ada_wt.txt the MSG_WAITALL > patch. Thanks. Actually raise-gcc.c is yet another issue. Could you please resubmit under three different emails/threads your 3 different patches, so that they can b

[patch] Fix bogus source locations with optimization

2011-03-28 Thread Eric Botcazou
Hi, when optimization is enabled, especially -O2 and above, you can have lines in the assembly file with really bogus source location info. The scenario is as follows: an optimization pass at the Tree level (typically PRE) creates a new statement and inserts it at some place in the dominator t

Re: [patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets

2011-03-28 Thread Kai Tietz
2011/3/28 Kai Tietz : > > ChangeLog gcc/ada (ada_wt.txt) > > 2011-03-28  Kai Tietz > >        * raise-gcc.c (PERSONALITY_FUNCTION): Add prototype to >        prevent warning. >        * g-socthi-mingw.adb (C_Recvmsg): Handle MSG_WAITALL for >        windows native targets. >        * s-oscons-tmplt

PING^3: Reducing the overhead of dwarf2 location tracking

2011-03-28 Thread Richard Sandiford
Third ping for this patch for PR48190 (4.5+): http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00255.html It fixes a source of quadratic memory use while generating dwarf2 var-tracking information. Thanks, Richard

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

2011-03-28 Thread Matthias Klose
On 05.03.2011 15:37, Richard Sandiford wrote: > Jakub Jelinek writes: >> On Fri, Mar 04, 2011 at 01:56:55PM +, Richard Sandiford wrote: >>> * dwarf2out.c (dw_loc_list_node): Add resolved_addr and replaced. >>> (cached_dw_loc_list_def): New structure. >>> (cached_dw_loc_list): New t

Re: [patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets

2011-03-28 Thread Kai Tietz
2011/3/28 Arnaud Charlet : >> this patch fixes some nits about MSG_WAITALL for mingw(-w64) targets. >> Additionally it improves multilib configury for windows native >> targets. > > Can you please split this patch in two, since the MSG_WAITALL is one thing, > and the multilib is a completely differ

Re: [patch ada]: Fix issues about multilib build of native windows and handle MSG_WAITALL for windows native targets

2011-03-28 Thread Arnaud Charlet
> this patch fixes some nits about MSG_WAITALL for mingw(-w64) targets. > Additionally it improves multilib configury for windows native > targets. Can you please split this patch in two, since the MSG_WAITALL is one thing, and the multilib is a completely different patch. Also, can you explain t

  1   2   >