Re: [patch, libgfortran] Fix numerous formatting bugs

2011-04-26 Thread Janne Blomqvist
On Wed, Apr 27, 2011 at 07:09, Jerry DeLisle wrote: > On 04/25/2011 07:36 AM, Janne Blomqvist wrote: >> >> On Mon, Apr 25, 2011 at 14:44, Jerry DeLisle >>  wrote: >>> >>> On 04/25/2011 03:48 AM, Janne Blomqvist wrote: Now, for one of the testcase changes: --- gcc/testsuite/gfor

C++ PATCH to debug_tree for TYPENAME_TYPE and IDENTIFIER

2011-04-26 Thread Jason Merrill
A couple of small changes. It was inconvenient that debug_tree didn't print the name of a TYPENAME_TYPE, and once it did I noticed that the indentation on the IDENTIFIER bits was wrong. Tested x86_64-pc-linux-gnu, applied to trunk. commit 3d660910ee64db8a09091482370ccc1162b781ca Author: Jason

C++ PATCH for c++/42687 (arg-dependent lookup regression)

2011-04-26 Thread Jason Merrill
When we fixed using explicit scope to suppress virtual calling, it broke using parentheses to suppress argument-dependent lookup. Fixed thus. Tested x86_64-pc-linux-gnu, applied to trunk, 4.4, 4.5 and 4.6. commit 2bc3ed125eab1075ea18c7c80e5ab908130861fd Author: Jason Merrill Date: Tue Apr 26

Re: [patch, libgfortran] Fix numerous formatting bugs

2011-04-26 Thread Jerry DeLisle
On 04/26/2011 09:09 PM, Jerry DeLisle wrote: ---snip--- 1.234500019E-06 4284600.00 1.234500019E-06 4284600.00 1.23450002E-06 4284600.00 And without the patch: 1.23450002E-06 4284600.00 1.234500019E-06 4284600.00 1.23450002E-06 4284600.00 d is th

Re: [patch, libgfortran] Fix numerous formatting bugs

2011-04-26 Thread Jerry DeLisle
On 04/25/2011 07:36 AM, Janne Blomqvist wrote: On Mon, Apr 25, 2011 at 14:44, Jerry DeLisle wrote: On 04/25/2011 03:48 AM, Janne Blomqvist wrote: Now, for one of the testcase changes: --- gcc/testsuite/gfortran.dg/char4_iunit_1.f03 (revision 172909) +++ gcc/testsuite/gfortran.dg/char4_iunit_1

Patches ping

2011-04-26 Thread Revital Eres
Hello, http://gcc.gnu.org/ml/gcc-patches/2011-03/msg00350.html - Support closing_branch_deps http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01309.html - Support instructions with REG_INC_NOTE http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01294.html - Avoid considering debug_insn when calculating SC

Go patch committed: Use backend interface for struct types

2011-04-26 Thread Ian Lance Taylor
This patch to the Go frontend uses the backend interface for struct types. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to mainline. Ian 2011-04-26 Ian Lance Taylor * go-gcc.cc (Gcc_backend::struct_type): Implement. Index: gcc/go/go-gcc.cc =

[pph] Tidy pph_output_tree_* (issue4432071)

2011-04-26 Thread Diego Novillo
No functional changes. This patch adds a common tree writing function (pph_output_tree_ref_1) with an additional argument specifying what tracing level to use to trigger a tracer call. Calls to pph_output_tree_or_ref() use a default tracing of 2. For callers that want to use different tracing l

[pph] Write TREE_LANG_FLAG_? fields (issue4430067)

2011-04-26 Thread Diego Novillo
More data we never bothered to stream from GIMPLE but it's important in the front end. Tested on x86_64. Committed to the branch. cp/ChangeLog.pph 2011-04-26 Diego Novillo * pph-streamer-in.c (pph_stream_unpack_value_fields): Read TREE_LANG_FLAG_? fields. * pph-strea

[pph] Fix many segmentation faults when restoring C++ ASTs (issue4441070)

2011-04-26 Thread Diego Novillo
I have been testing the C++ streamer with a collection of 6,576 C test cases taken from the GCC testsuite. These C programs build fine with g++, so I encapsulated them inside a single header file and created a .cc file that simply #includes it. The intent is to make sure we can PPH generate and

Re: C++ PATCH for c++/48281 (ICE with nested initializer_list)

2011-04-26 Thread Jason Merrill
On 03/30/2011 02:05 PM, Jason Merrill wrote: First I noticed that the crash was due to non-constant CONSTRUCTORs with TREE_CONSTANT set. So I fixed that, in constructor-const.patch. It turns out that we need to apply this patch to 4.6 as well, for bug 48726. So I'm doing that. Jason

[pph] Mark C++ codes with proper TS_* type markers (issue4436059)

2011-04-26 Thread Diego Novillo
We were failing to save several fields in C++ specific tree codes because they were not properly marked with TS_* tree structure tags. For instance, we were not streaming EXPR_STMTs because they were not recognized as TS_EXPs in the streamer, so its arguments were never saved. Tested on x86_64.

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Cary Coutant
> Well I think -g1 -g should act like how -O2 -O works.  That is should be -g2. I think so, too, but I guess I was too timid to assume such a change might be acceptable. If the maintainers would go along with that, I'd have no problem revising the patch to make -g1 do line tables and inline info.

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

2011-04-26 Thread Steve Kargl
On Wed, Apr 27, 2011 at 12:23:03AM +0200, Mikael Morin wrote: > On Tuesday 26 April 2011 18:52:58 Steve Kargl wrote: > > On Mon, Apr 25, 2011 at 11:15:35PM +0300, Janne Blomqvist wrote: > > > On Mon, Apr 25, 2011 at 22:45, Steve Kargl > > > > > > wrote: > > > > On Mon, Apr 25, 2011 at 10:26:20PM

Re: [PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Kaz Kojima
"Joseph S. Myers" wrote: > I think you should add a testcase to gcc.c-torture/compile, unless there > is already one that this patch fixes. Ah, indeed. How about the attached testcase? BTW, is it valid C? Regards, kaz -- --- ORIG/trunk/gcc/testsuite/gcc.c-torture/compile/pr48767.c1

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Andrew Pinski
On Tue, Apr 26, 2011 at 3:57 PM, Cary Coutant wrote: > If we do go with extending -g1, though. we want to be able to turn it > on in our build scripts, yet allow a later "-g" option in the user's > compiler options to enable full debug info. In this patch, I've made > sure that "-gmlt -g" is equiv

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Cary Coutant
>> This patch adds a new option, -gmlt, that produces level 1 debug info >> plus line number tables and inlined subroutine information. (The option >> is short for "minimum line tables," taken from a similar feature of >> HP's compilers.) > > What is the difference between -gmlt and -g1?  And why c

Re: [patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Andrew Pinski
On Tue, Apr 26, 2011 at 3:44 PM, Cary Coutant wrote: > This patch adds a new option, -gmlt, that produces level 1 debug info > plus line number tables and inlined subroutine information. (The option > is short for "minimum line tables," taken from a similar feature of > HP's compilers.) What is t

[patch] Add new -gmlt option for min. debug info with line tables (issue4440072)

2011-04-26 Thread Cary Coutant
This patch adds a new option, -gmlt, that produces level 1 debug info plus line number tables and inlined subroutine information. (The option is short for "minimum line tables," taken from a similar feature of HP's compilers.) We've been using this option at Google for about a year now, and have f

PATCH: PR rtl-optimization/48575: RTL vector patterns are limited to 26 elements

2011-04-26 Thread H.J. Lu
On Mon, Apr 4, 2011 at 6:05 PM, H.J. Lu wrote: > On Thu, Mar 31, 2011 at 5:05 AM, Kenneth Zadeck > wrote: >> we hit this limit trying to write the explicit semantics for a >> vec_interleave_evenv32qi. >> >> ;;(define_insn "vec_interleave_evenv32qi" >> ;;  [(set (match_operand:V32QI 0 "register_op

Re: [PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Joseph S. Myers
I think you should add a testcase to gcc.c-torture/compile, unless there is already one that this patch fixes. -- Joseph S. Myers jos...@codesourcery.com

Re: Don't use linux.h for non-Linux targets

2011-04-26 Thread Thomas Schwinge
Hallo! First, Joseph, thanks for working on this clean-up! I'm sorry for the delay, but I have now reviewed all these patches (including those you have committed by now); looks all good. On Tue, 12 Apr 2011 18:37:50 + (UTC), "Joseph S. Myers" wrote: > This patch stops targets not using th

[PATCH Commitred] Fix PR target/48767

2011-04-26 Thread Kaz Kojima
Hi, The attached target specific patch is to fix PR target/48767. In the problematic case, sh.c:sh_gimplify_va_arg_expr calls targetm.calls.must_pass_in_stack with void type as its 2nd argument which is unexpected by the callee. The patch is tested on sh4-unknown-linux-gnu with no new failures. A

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

2011-04-26 Thread Mikael Morin
On Tuesday 26 April 2011 18:52:58 Steve Kargl wrote: > On Mon, Apr 25, 2011 at 11:15:35PM +0300, Janne Blomqvist wrote: > > On Mon, Apr 25, 2011 at 22:45, Steve Kargl > > > > wrote: > > > On Mon, Apr 25, 2011 at 10:26:20PM +0300, Janne Blomqvist wrote: > > >> Hmm, I'd prefer if the warning was is

Re: [patch, fortran] Make block names unique

2011-04-26 Thread Thomas Koenig
Hi Mikael, * decl.c (gfc_match_end): Check that the block name starts with "block@". * parse.c (gfc_build_block_ns): Make block names unique by numbering them. Ping ** 0.25? Thomas OK Waiting for Emacs... Sendingfortran/ChangeLog Sendingfortran/decl.c Sending

Re: Improve DSE in the presence of calls

2011-04-26 Thread Easwaran Raman
On Mon, Apr 25, 2011 at 10:03 AM, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 04/22/11 14:19, Easwaran Raman wrote: >> Hi, >>  This patch improves RTL DSE by not assuming that calls read all >> memory locations. With this patch, calls are assumed to read any >> non-fra

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

2011-04-26 Thread Ralf Wildenhues
Hello Rainer, * Rainer Orth wrote on Tue, Apr 26, 2011 at 05:28:19PM CEST: > it's been a week since I answered your questions on this patch. Could > you please have a look? Sorry for the delay. I'm practically AFK until the weekend or maybe next weekend, whenever I have connectivity again after

[Patch, Fortran] PRs 48112/48279 - fix interface related regressions

2011-04-26 Thread Tobias Burnus
This patch fixes three issues, (a) and (b) are 4.6/4.7 regressions. a) PR48112: Due to incomplete resolution, there was an ICE when writing the module file. Solution: Back-out the patch which introduced the incomplete resolution - and add a similar patch later to avoid printing errors multiple

Re: [patch, fortran] Make block names unique

2011-04-26 Thread Mikael Morin
On Tuesday 26 April 2011 00:40:40 Thomas Koenig wrote: > Am 22.04.2011 22:07, schrieb Thomas Koenig: > > Hello world, > > > > the attached patch makes block names unique, so that > > -fdump-fortran-original dumps are easier to read. > > > > Regression-tested. OK for trunk? > > > > Thomas > > >

Re: [PATCH, C++] Remove unused cp_fold_obj_type_ref

2011-04-26 Thread Jason Merrill
On 04/26/2011 02:07 PM, Rainer Orth wrote: the patch was so obvious that it broke bootstrap: I'm applying this patch to fix bootstrap. commit 922f4a593d65ab65025b0af7b59bb9513d394aac Author: jason Date: Tue Apr 26 21:12:57 2011 + * objc-act.c (objc_fold_objc_type_ref): Remove.

New mirror in Phoenix, US (was: new mirror in Nicosia, Cyprus)

2011-04-26 Thread Gerald Pfeifer
On Mon, 25 Apr 2011, Gerald Pfeifer wrote: > Grigory and me mailed off-list a bit, and I just applied the patch > below (after checking the mirror now seems to be fully active). > > Just one question: is this server _really_ located in Cyprus? My > traceroutes seem to indicate otherwise. It turn

Re: [C PATCH] Avoid nested C_MAYBE_CONST_EXPR in build_binary_op (PR c/48742)

2011-04-26 Thread Joseph S. Myers
On Tue, 26 Apr 2011, Jakub Jelinek wrote: > 2011-04-26 Jakub Jelinek > > PR c/48742 > * c-typeck.c (build_binary_op): Don't wrap arguments if > int_operands is true. > > * gcc.c-torture/compile/pr48742.c: New test. OK. -- Joseph S. Myers jos...@codesourcery.com

C++ PATCH for c++/48530 (hard errors with deleted destructors)

2011-04-26 Thread Jason Merrill
Daniel Krugler has been doing well at ferreting out parts of the compiler that haven't been properly SFINAE'd yet... Tested x86_64-pc-linux-gnu, applied to trunk. commit 93016efbbb141f1bd917c4032b4a94b2b4c85516 Author: Jason Merrill Date: Mon Apr 25 20:07:15 2011 -0400 PR c++/48530

Re: [patch, fortran] Put front-end temporaries into BLOCKs

2011-04-26 Thread Thomas Koenig
Hi Jerry, OK and thanks for patch. Waiting for Emacs... Sende ChangeLog Sende frontend-passes.c Übertrage Daten .. Revision 172983 übertragen. Thanks for the review! Thomas

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-26 Thread David Miller
From: Rainer Orth Date: Tue, 26 Apr 2011 17:25:56 +0200 > David, > >>> Thanks Rainer, if you cons up a patch to add the ".align" directive I'll ACK >>> that as well. >> >> here's the patch that I've been testing over the weekend. It caused no >> change for HAVE_AS_SPARC_GOTDATA_OP on anything b

Re: [PATCH, 4.7] Add float/double vector reductions to VSX

2011-04-26 Thread Michael Meissner
On Sat, Apr 23, 2011 at 04:11:50PM -0400, David Edelsohn wrote: > Except the patch includes a change to pattern vsx_xxpermdi_ that > is not explained nor mentioned in the ChangeLog. Yes, that vsx_xxpermdi_ patch was wrong. I believe that snuck in because I had done the work on an older tree befor

Re: [PATCH, C++] Remove unused cp_fold_obj_type_ref

2011-04-26 Thread Rainer Orth
Martin, > last year I removed the last use of cp_fold_obj_type_ref (which was a > language hook) but forgot to remove the function itself. So I'm about > to do it now. > > I'm currently bootstrapping and testing the following patch. I > consider it obvious enough to commit it myself on Tuesday i

[C PATCH] Avoid nested C_MAYBE_CONST_EXPR in build_binary_op (PR c/48742)

2011-04-26 Thread Jakub Jelinek
Hi! As discussed in the PR, build_binary_op was calling c_wrap_maybe_const which added one C_MAYBE_CONST_EXPR to GT_EXPR operand, then the whole GT_EXPR has been enclosed into another C_MAYBE_CONST_EXPR because int_operands has been true. Fixed thusly, bootstrapped/regtested on x86_64-linux and i

Put inliners info about cgraph edges into its own summary structure

2011-04-26 Thread Jan Hubicka
Hi, this patch moves inliner related callgraph edge info (call_stmt_size/time and loop_nest) into inliner own edge summaries, just like we have for ipa-prop. Primary motivation for doing so is that I don't want to add edge predicates to main cgraph edge and handle them in lto-cgraph, but doing thi

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

2011-04-26 Thread Janne Blomqvist
On Tue, Apr 26, 2011 at 19:52, Steve Kargl wrote: > Yes.  I've implemented in the revised patch, and I've > updated the docs. > > 2011-04-26  Steven G. Kargl   > >        PR fortran/48720 >        * gfortran.texi: Document the 'Q' exponent-letter extension. >        * invoke.texi: Document -Wreal-

Re: [PATCH,Fortran] Handle 'q' exponent-letter in real-literal-constant

2011-04-26 Thread Steve Kargl
On Mon, Apr 25, 2011 at 11:15:35PM +0300, Janne Blomqvist wrote: > On Mon, Apr 25, 2011 at 22:45, Steve Kargl > wrote: > > On Mon, Apr 25, 2011 at 10:26:20PM +0300, Janne Blomqvist wrote: > >> Hmm, I'd prefer if the warning was issued only with -Wsomething which > >> would be included in -Wall. Bu

Re: Ping^2 Re: Target header etc. cleanup patch

2011-04-26 Thread Christopher Faylor
On Thu, Apr 21, 2011 at 11:56:52PM -0400, Jie Zhang wrote: >On 04/21/2011 06:06 AM, Joseph S. Myers wrote: >> On Thu, 21 Apr 2011, Richard Guenther wrote: >> >>> On Wed, Apr 20, 2011 at 11:09 PM, Joseph S. Myers >>> wrote: Ping^2. This patch

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-26 Thread Paolo Bonzini
On 04/26/2011 05:25 PM, Rainer Orth wrote: David, Thanks Rainer, if you cons up a patch to add the ".align" directive I'll ACK that as well. here's the patch that I've been testing over the weekend. It caused no change for HAVE_AS_SPARC_GOTDATA_OP on anything but Solaris 10 with gas and Sun

Re: [debug] PR42288 emit empty .debug_aranges section

2011-04-26 Thread Rainer Orth
Hi Mark, > Thanks for testing the patch. Although I am slightly confused how it > ended up producing a .debug_abbrev section that is actually empty. Are > you sure this isn't an existing issue? it certainly is, and not related to your patch (which I didn't test: my IRIX and Tru64 UNIX boxes are f

Re: [doc] Remove references to mips-tfile on MIPS

2011-04-26 Thread Rainer Orth
Joseph, > Not strictly related to this patch, but there are other cleanups possible > because of the only-used-on-Tru64 nature of mips-tdump/mips-tfile. In > particular, there are seven target macros (all undocumented) used by those > programs and nowhere else in GCC: ALIGN_SYMTABLE_OFFSET COD

Re: Ping^3 Re: Target header etc. cleanup patch

2011-04-26 Thread Joseph S. Myers
On Tue, 26 Apr 2011, Nick Clifton wrote: > Similarly it would be nice to standard abbreviations for CUMULATIVE_ARGS, enum > rtx_code, enum reg_class and const char *. What do you think ? I suspect I think defining an abbreviation for const char * is actively bad, since const char * is a standa

Re: Ping^3 Re: Target header etc. cleanup patch

2011-04-26 Thread Nick Clifton
Hi Joseph, 2011-04-26 Joseph Myers * config/fr30/fr30-protos.h (Mmode): Don't define. * config/fr30/fr30.h (inhibit_libc): Don't define. * config/m32r/m32r-protos.h (Mmode): Don't define. Expand definition where used. * config/mn10300/mn10300-protos.h (

iris6.h cleanup

2011-04-26 Thread Rainer Orth
I noticed a couple of cleanup opportunities in iris6.h: * TARGET_NEWABI and TARGET_IRIX6 are always true in iris6.h. * I'm avoiding an inline expansion of INT_TYPE_SIZE. Bootstrapped without regressions on mips-sgi-irix6.5, installed on mainline. There may be more, though: * Perhaps I can defi

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

2011-04-26 Thread Rainer Orth
Hi Ralf, it's been a week since I answered your questions on this patch. Could you please have a look? Thanks. Rainer >>> I haven't found if there are provisions for in-tree gold, though, and >>> still cannot test that. >> >> I'm not quite sure I understand this statement. I built a c

Re: [build] Avoid ld -v error message with Sun ld on SPARC

2011-04-26 Thread Rainer Orth
David, >> Thanks Rainer, if you cons up a patch to add the ".align" directive I'll ACK >> that as well. > > here's the patch that I've been testing over the weekend. It caused no > change for HAVE_AS_SPARC_GOTDATA_OP on anything but Solaris 10 with gas > and Sun ld, where HAVE_AS_SPARC_GOTDATA_OP

Ping^3 Re: Target header etc. cleanup patch

2011-04-26 Thread Joseph S. Myers
Ping^3. Parts of this patch are still pending review: the parts for targets fr30, m32r, mn10300, rx, v850 (target maintainers CC:ed). This version applies cleanly to current trunk. 2011-04-26 Joseph Myers * config/fr30/fr3

Re: [Patch,AVR]: Solve PR42210

2011-04-26 Thread Georg-Johann Lay
Richard Henderson schrieb: > On 04/26/2011 04:51 AM, Georg-Johann Lay wrote: >> As the SFRs are volatile, insv expander skips >> them and a combine pattern must care of them. Omitting the complicated >> bit-plethora patterns I still see long, slow shift loops for HI. So >> the patch still includes

Re: [Patch,AVR]: Solve PR42210

2011-04-26 Thread Richard Henderson
On 04/26/2011 04:51 AM, Georg-Johann Lay wrote: > As the SFRs are volatile, insv expander skips > them and a combine pattern must care of them. Omitting the complicated > bit-plethora patterns I still see long, slow shift loops for HI. So > the patch still includes them. Why don't we focus on doin

Re: [PATCH] Fix PR48753

2011-04-26 Thread Jakub Jelinek
On Tue, Apr 26, 2011 at 12:02:38PM +0200, Richard Guenther wrote: > > Value-profiling now creates proper integer constants. This still fails on 32-bit HWI hosts, like i686-linux. Fixed thusly, regtested on x86_64-linux and i686-linux, committed to trunk: 2011-04-26 Jakub Jelinek PR

Re: [PATCH] Fix VTA ICE after fixup_noreturn_call (PR debug/48768)

2011-04-26 Thread Richard Guenther
On Tue, Apr 26, 2011 at 3:08 PM, Jakub Jelinek wrote: > Hi! > > fixup_noreturn_call when removing LHS of a noreturn call does: >      /* We need to remove SSA name to avoid checking errors. >         All uses are dominated by the noreturn and thus will >         be removed afterwards. >         We

Re: [PATCH] Fix PR48753

2011-04-26 Thread H.J. Lu
On Tue, Apr 26, 2011 at 3:02 AM, Richard Guenther wrote: > > Value-profiling now creates proper integer constants. > > Installed. > > Richard. > > 2011-04-26  Richard Guenther   > >        PR testsuite/48753 >        * gcc.dg/tree-prof/val-prof-2.c: Adjust. > > Index: gcc/testsuite/gcc.dg/tree-pro

[PATCH] Fix VTA ICE after fixup_noreturn_call (PR debug/48768)

2011-04-26 Thread Jakub Jelinek
Hi! fixup_noreturn_call when removing LHS of a noreturn call does: /* We need to remove SSA name to avoid checking errors. All uses are dominated by the noreturn and thus will be removed afterwards. We proactively remove affected non-PHI statements to avoid

PING: PATCH: PR other/48007: Unwind library doesn't work with UNITS_PER_WORD > sizeof (void *)

2011-04-26 Thread H.J. Lu
On Sat, Apr 9, 2011 at 6:52 PM, H.J. Lu wrote: > On Thu, Mar 24, 2011 at 12:15 AM, H.J. Lu wrote: >> On Wed, Mar 23, 2011 at 12:22 PM, Ulrich Weigand wrote: >>> Richard Henderson wrote: Because, really, if we consider the structure truly public, we can't even change the number of regis

Re: [PATCH] Fix eliminate_redundant_comparison (PR tree-optimization/48734)

2011-04-26 Thread Richard Guenther
On Tue, Apr 26, 2011 at 3:02 PM, Jakub Jelinek wrote: > Hi! > > eliminate_redundant_comparison calls maybe_fold_{and,or}_comparisons, > which calls fold.  Expecting that the result is either INTEGER_CST or > a comparison with gimple vals is just bad assumption, fold can create > all kinds of canon

[PATCH, i386]: Remove ext_QIreg_nomode_operands predicate

2011-04-26 Thread Uros Bizjak
Hello! We can use ext_QIreg_operands instead. These predicates automatically include mode checks with VOIDmode override. Also, clean predicates.md a bit while there. 2011-04-26 Uros Bizjak * config/i386/predicates.md (ext_QIreg_operand): Remove extra mode check. (ext

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

2011-04-26 Thread H.J. Lu
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, Richard Guenther >>> wrote: On Thu, Jan 27, 2011 at 12:12 AM, H.J. Lu wrote: > On Tue, Dec 14, 20

[PATCH] Fix eliminate_redundant_comparison (PR tree-optimization/48734)

2011-04-26 Thread Jakub Jelinek
Hi! eliminate_redundant_comparison calls maybe_fold_{and,or}_comparisons, which calls fold. Expecting that the result is either INTEGER_CST or a comparison with gimple vals is just bad assumption, fold can create all kinds of canonicalizations. While we can perhaps handle a few of them here, I'm

Re: [google]Pass --save-temps to the assembler (issue4436049)

2011-04-26 Thread Diego Novillo
On Mon, Apr 25, 2011 at 15:00, Easwaran Raman wrote: > 2011-04-25  Easwaran Raman   > >        * gcc/gcc.c (asm_options): Pass --save-temps to assembler >        when invoked with -save-temps= option. OK. Diego.

Re: [google] Port self-assign warning to google/main branch (issue4442075)

2011-04-26 Thread Diego Novillo
On Fri, Apr 22, 2011 at 16:08, Le-Chun Wu wrote: > 2011-04-22  Le-Chun Wu   > > gcc/c-family/ChangeLog: >        * c-common.c (check_for_self_assign): New function. >        * c-common.h: New function declaration. >        * c.opt: New option. > > gcc/ChangeLog: >        * c-parser.c (c_parser_dec

[PATCH] Fix PR48731

2011-04-26 Thread Richard Guenther
This adds a missing check for ->local.inlinable to cgraph_flatten. Honza fixed this on trunk by factoring out the checks. Bootstrapped and tested on x86_64-unknown-linux-gnu, installed. Richard. 2011-04-26 Richard Guenther PR tree-optimization/48731 * ipa-inline.c (cgraph_fl

Re: [RFC] Context sensitive inline analysis

2011-04-26 Thread Jan Hubicka
> Honza, > > This patch causes a bootstrap failure when building libstdc++ on AIX: > > In file included from > /farm/dje/src/src/libstdc++-v3/include/precompiled/stdc++.h:94:0: > /tmp/20110423/powerpc-ibm-aix5.3.0.0/libstdc++-v3/include/valarray:1163:1: > internal compiler error: vector VEC(tree,

Re: [Patch,AVR]: Solve PR42210

2011-04-26 Thread Georg-Johann Lay
Richard Henderson schrieb: > On 04/21/2011 05:31 AM, Georg-Johann Lay wrote: >> +;; Some combiner patterns dealing with bits. >> +;; See PR42210 >> + >> +;; Move bit $3.$4 into bit $0.$4 >> +(define_insn "*movbitqi.1-6.a" > ... >> +(define_insn "*movbitqi.1-6.b" > ... >> +(define_insn "*movbitqi.0"

Re: [PATCH] Canonicalize compares in combine [2/3] Modifications to try_combine()

2011-04-26 Thread Chung-Lin Tang
On 2011/4/26 02:08 AM, Jeff Law wrote: > -BEGIN PGP SIGNED MESSAGE- > Hash: SHA1 > > On 04/22/11 09:21, Chung-Lin Tang wrote: >> This patch is the main bulk of this submission. It modifies the compare >> combining part of try_combine(), adding a call of >> CANONICALIZE_COMPARISON into the

Patch ping

2011-04-26 Thread Jakub Jelinek
Hi! http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01117.html - fix unwind info in presence of inline asm, 4.6/4.7 http://gcc.gnu.org/ml/gcc-patches/2011-04/msg01264.html - typed DWARF stack Jakub

Re: libstdc++.exp patch

2011-04-26 Thread Paolo Carlini
On 04/26/2011 10:59 AM, Rainer Orth wrote: Please separate the proc names by commas in the ChangeLog. Indeed. I took care of this. Paolo.

[wwwdocs] PATCH for Re: going from SunOS 5/SparcWorks -> Linux/gcc

2011-04-26 Thread Gerald Pfeifer
Hi Paweł, On Thu, 29 Apr 2010, Paweł Sikora wrote: >> The standard 1st questions are: >> 1) Did you compile with -Wall -Wextra and solve all warnings? >> 2) Did you try with -fno-strict-aliasing? > for legacy code, the '-fwrapv' could be helpful. your mail reminded me that we could/should clarify

Re: [PATCH][ARM] Fix -Werror build failure

2011-04-26 Thread Ramana Radhakrishnan
> > OK? OK. Ramana

[PATCH] Fix PR48753

2011-04-26 Thread Richard Guenther
Value-profiling now creates proper integer constants. Installed. Richard. 2011-04-26 Richard Guenther PR testsuite/48753 * gcc.dg/tree-prof/val-prof-2.c: Adjust. Index: gcc/testsuite/gcc.dg/tree-prof/val-prof-2.c =

[PATCH][ARM] Fix -Werror build failure

2011-04-26 Thread Andrew Stubbs
My previous patch to clean up MOVW support left an unused variable. I tested building a cross-compiler, but not with -Werror. Others have reported native bootstrap failing. :( This patch fixes that problem, and removes the unused variable warning message. OK? Andrew 2011-04-26 Andrew Stubb

Re: [debug] PR42288 emit empty .debug_aranges section

2011-04-26 Thread Mark Wielaard
Hi Rainer, On Tue, 2011-04-26 at 11:22 +0200, Rainer Orth wrote: > Mark Wielaard writes: > > > This PR was put on hold for a while since gdb didn't immediately need > > it. But the consensus seemed that it was a good idea to always output > > the address ranges (.debug_arange section) informatio

Re: [debug] PR42288 emit empty .debug_aranges section

2011-04-26 Thread Rainer Orth
Mark Wielaard writes: > This PR was put on hold for a while since gdb didn't immediately need > it. But the consensus seemed that it was a good idea to always output > the address ranges (.debug_arange section) information if a CU > (.debug_info section) was emitted. Because the consumer has no w

[debug] PR42288 emit empty .debug_aranges section

2011-04-26 Thread Mark Wielaard
Hi, This PR was put on hold for a while since gdb didn't immediately need it. But the consensus seemed that it was a good idea to always output the address ranges (.debug_arange section) information if a CU (.debug_info section) was emitted. Because the consumer has no way to tell the difference b

Re: [doc] Adjust reference to ACATS in sourcebuild.texi

2011-04-26 Thread Rainer Orth
Gerald Pfeifer writes: > Installed on mainline; if someone wants me to, happy to apply to older > branches as well. > > Gerald > > 2011-04-24 Gerald Pfeifer > > * doc/sourcebuild.texi (Ada Tests): Adjust reference to ACATS > testsuite and make it version agnostic. I doubt this is

Re: libstdc++.exp patch

2011-04-26 Thread Rainer Orth
François Dumont writes: > Attached patch applied. > > 2011-04-21 François Dumont > > * testsuite/lib/libstdc++.exp (check_v3_target_time): Discard > unused compilation result thanks to /dev/null. > * testsuite/lib/libstdc++.exp (check_v3_target_debug_mode > check