Re: [PATCH] Add option for dumping to stderr (issue6190057)

2012-05-08 Thread Gabriel Dos Reis
On Wed, May 9, 2012 at 1:51 AM, Andrew Pinski wrote: > On Tue, May 8, 2012 at 11:46 PM, Sharad Singhai wrote: >> In response to comments, I have updated the patch to support dumps in >> user provided files via the option -fdump-xxx=. The >> filenames stdout/stderr are treated specially, and are c

Re: [PATCH] Add option for dumping to stderr (issue6190057)

2012-05-08 Thread Andrew Pinski
On Tue, May 8, 2012 at 11:46 PM, Sharad Singhai wrote: > In response to comments, I have updated the patch to support dumps in > user provided files via the option -fdump-xxx=. The > filenames stdout/stderr are treated specially, and are considered > standard streams. I think - should also be tre

Re: patch ping: Add static branch predict heuristic of comparing IV to loop_bound variable

2012-05-08 Thread Jakub Jelinek
On Wed, May 09, 2012 at 09:02:14AM +0800, Dehao Chen wrote: > Sorry for the error. Here is a new patch to fix them: > > gcc/testsuite/ChangeLog: > 2012-05-08 Dehao Chen > > * gcc.dg/predict-1.c: Remove the replicated text in this text. > * gcc.dg/predict-2.c: Likewise. > * gc

[PATCH] Add option for dumping to stderr (issue6190057)

2012-05-08 Thread Sharad Singhai
In response to comments, I have updated the patch to support dumps in user provided files via the option -fdump-xxx=. The filenames stdout/stderr are treated specially, and are considered standard streams. Also updated documentation and a testcase. Okay for trunk? Thanks, Sharad 2012-05-08 Sha

Re: [PATCH] MIPS16: Fix truncated DWARF-2 line information

2012-05-08 Thread Maciej W. Rozycki
On Tue, 8 May 2012, Richard Sandiford wrote: > > Are you using a hard-float multilib for your -mabi=32/-mips16 Linux > > testing? > > Yeah. As an example: > >http://gcc.gnu.org/ml/gcc-testresults/2012-03/msg00393.html > > which doesn't look to bad. Clean fortran results, which I expect

Fix gcc.dg/lower-subreg-1.c failure (was: [C Patch]: pr52543)

2012-05-08 Thread Hans-Peter Nilsson
> From: Richard Sandiford > Date: Tue, 1 May 2012 16:46:38 +0200 > To repeat: as things stand, very few targets define proper rtx costs > for SET. IMHO it's wrong to start blaming targets when rtx_cost doesn't take the mode in account in the first place, for the default cost. (Well, except for

[PATCH] Remove -Y option from linker command line on Linux/Sparc.

2012-05-08 Thread David Miller
We never really should have been passing this Solaris compatability option for specifying library paths in the first place. Let them get set the right way when the compiler passes in "-L" options. Committed to master. gcc/ * config/sparc/linux.h (LINK_SPEC): Don't pass "-Y" option.

[PATCH] Fix PR53217

2012-05-08 Thread William J. Schmidt
This fixes another statement-placement issue when reassociating expressions with repeated factors. Multiplies feeding into __builtin_powi calls were not getting placed properly ahead of them in some cases. Bootstrapped and tested on powerpc64-unknown-linux-gnu with no new regressions. I've also

Re: Continue strict-volatile-bitfields fixes

2012-05-08 Thread Thomas Schwinge
Hi! On Fri, 27 Apr 2012 10:29:06 +0200, Jakub Jelinek wrote: > On Fri, Apr 27, 2012 at 12:42:41PM +0800, Thomas Schwinge wrote: > > > > GET_MODE_BITSIZE (lmode)« (8 bits).  (With the current sources, lmode is > > > > VOIDmode.) > > > > > > > > Is emmitting »BIT_FIELD_REF <*common, 32, 0> & 255« w

Re: patch ping: Add static branch predict heuristic of comparing IV to loop_bound variable

2012-05-08 Thread Dehao Chen
Sorry for the error. Here is a new patch to fix them: gcc/testsuite/ChangeLog: 2012-05-08 Dehao Chen * gcc.dg/predict-1.c: Remove the replicated text in this text. * gcc.dg/predict-2.c: Likewise. * gcc.dg/predict-3.c: Likewise. * gcc.dg/predict-4.c: Likewise.

Re: [patch] support for multiarch systems

2012-05-08 Thread Matthias Klose
On 08.05.2012 15:20, Joseph S. Myers wrote: > On Tue, 8 May 2012, Matthias Klose wrote: > >> On 20.08.2011 21:51, Matthias Klose wrote: >>> Multiarch [1] is the term being used to refer to the capability of a system >>> to >>> install and run applications of multiple different binary targets on t

Re: Heads-up, PR53273: testsuite separation and dilution problem. Fix for PR53272

2012-05-08 Thread Hans-Peter Nilsson
> From: Richard Guenther > Date: Tue, 8 May 2012 10:50:43 +0200 > On Tue, May 8, 2012 at 5:39 AM, Hans-Peter Nilsson > wrote: > > The problem was spotted while fixing PR53272, a target bug with > > crisv32-* involving the error-prone notice_update_cc function. > > > > When wrapping up the test-c

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Cary Coutant
>> A suggestion: Make dwarf_name call the demangler, and then a (new?) a >> function that converts a mangled decl to a human-readable string. In >> any case, the pretty-printer does a lot of stuff that is mostly >> useless for just printing a declaration (translation, wrapping, etc.). >> >> Bonus p

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Gabriel Dos Reis
On Tue, May 8, 2012 at 4:38 PM, Manuel López-Ibáñez wrote: > A suggestion: Make dwarf_name call the demangler, and then a (new?) a > function that converts a mangled decl to a human-readable string. In > any case, the pretty-printer does a lot of stuff that is mostly > useless for just printing a

Re: PR 53249: Multiple address modes for same address space

2012-05-08 Thread Richard Henderson
On 05/06/2012 11:41 AM, Richard Sandiford wrote: PR middle-end/53249 * dwarf2out.h (get_address_mode): Move declaration to... * rtl.h: ...here. * dwarf2out.c (get_address_mode): Move definition to... * rtlanal.c: ...here. * var-tracking.c (get_addre

Re: Use C++ in COMPILER_FOR_BUILD if needed (issue6191056)

2012-05-08 Thread Diego Novillo
On 12-05-08 15:46 , Diego Novillo wrote: Found this while testing the C++ conversion for vec.[ch] on the cxx-conversion branch. We do not build the build/*.o files with g++, so I was getting lots of syntax errors while compiling build/vec.o. I am not completely sure if the changes are correct.

gnu-tm: Dont allow assigning transaction_unsafe functions to transaction_safe function pointers (issue6198054)

2012-05-08 Thread Dave Boutcher
Without this patch it is perfectly fine to assign non-transaction_safe functions to function pointers marked as transaction_safe. Unpleasantness happens at run time. e.g. __attribute__((transaction_safe)) long (*compare)(int, int); compare = my_funky_random_function; gcc/c-typeck.c |7

trans-mem: make sure clones for functions referenced indirectly are marked as needed (issue6201064)

2012-05-08 Thread Dave Boutcher
Without this patch we generate calls to TM_GETTMCLONE for functions called indirectly, but we don't actually store the clone mapping in the clone table because we think the functions are not "needed". Compiles fine, dies at runtime. See GCC Bugzilla – Bug 53008 gcc/trans-mem.c | 14 +

Symbol table 19/many: cleanup varpool/front-end/varasm interactions

2012-05-08 Thread Jan Hubicka
Hi, this patch puts varpool more into control of what is output. Traditionally frontends were outputting variables as they parsed them via tree_rest_of_compilation or at the end of compilation via wrapup_global_declarations that went through all the decls, output unused warnings and called tree_r

Use C++ in COMPILER_FOR_BUILD if needed (issue6191056)

2012-05-08 Thread Diego Novillo
Found this while testing the C++ conversion for vec.[ch] on the cxx-conversion branch. We do not build the build/*.o files with g++, so I was getting lots of syntax errors while compiling build/vec.o. I am not completely sure if the changes are correct. But it works for me. Tested on x86_64.

trans-mem: functions making indirect calls are not transformed (issue6194061)

2012-05-08 Thread Dave Boutcher
gcc/trans-mem.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/gcc/trans-mem.c b/gcc/trans-mem.c index 2badf25..24073fa 100644 --- a/gcc/trans-mem.c +++ b/gcc/trans-mem.c @@ -4721,7 +4721,7 @@ ipa_tm_transform_clone (struct cgraph_node *node) /* If this function makes no

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-08 Thread Joseph S. Myers
On Wed, 9 May 2012, Manuel L?pez-Ib??ez wrote: > which looks correct to me. However, the build fails because now > options.h requires input.h which requires line-map.h, which is not > included when building for example libgcc. options.h is included by > tm.h, so it basically appears everywhere. >

Re: [RFC] PR 53063 encode group options in .opt files

2012-05-08 Thread Manuel López-Ibáñez
On 6 May 2012 20:45, Joseph S. Myers wrote: >> >> One idea could be to have an additional auto_handle_option() that is >> generated from the awk scripts and called after all other >> handle_option functions. This function will populate a switch with >> group options and the respective calls to han

[committed] Fix PR c++/52361 ICE in tree_strip_nop_conversions

2012-05-08 Thread Manuel López-Ibáñez
The return value of build_range_check can be NULL and integer_zerop cannot handle that. Bootstrapped and regression tested. John David Anglin tested that it fixes the ICE in hppa. Committed to mainline as obvious. 2012-05-09 Manuel López-Ibáñez PR c++/53261 * c-common.c (warn_l

Re: [committed] Fix lower-subreg cost calculation

2012-05-08 Thread Richard Sandiford
Richard Earnshaw writes: > FTR, this caused > http://gcc.gnu.org/bugzilla/show_bug.cgi?id=53278 Well, this really has been a brown-paper-bag patch. Fixed as below. Tested on x86_64-linux-gnu and applied as obvious. Richard gcc/ PR rtl-optimization/53278 * lower-subreg.c (decom

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Manuel López-Ibáñez
On 8 May 2012 18:20, Sterling Augustine wrote: > On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis > wrote: >> On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine >> wrote: >>> This is the second in the series of patches to make c decl pretty printing >>> more closely match the demangler. A full e

Re: [PR tree-optimization/52558]: RFC: questions on store data race

2012-05-08 Thread Aldy Hernandez
On 05/07/12 19:11, Andrew MacLeod wrote: On 05/07/2012 07:04 PM, Aldy Hernandez wrote: Andrew suggested the correct fix was to add a new pass that was able to do some ?? flow sensitive data flow analysis ?? that could discover these unreachable paths and insert the 0 phis at the start of the b

Re: [PATCH] libgcov support for profile collection in region of interest (issue6186044)

2012-05-08 Thread Jan Hubicka
> On Tue, May 8, 2012 at 10:37 AM, Jan Hubicka wrote: > > > > > Hi Honza, > > > > > > I added L_gcov_reset and L_gcov_dump for the new interfaces, and also > > > added a description into the gcov man page. Let me know if it looks > > > ok now. > > > > > > Bootstrapped and tested on x86_64-unknown-

Re: [testsuite] Fix gcc.target/i386/hle-* testcases with Sun as

2012-05-08 Thread Mike Stump
On May 8, 2012, at 10:19 AM, Rainer Orth wrote: > Several /gcc.target/i386/hle-*.c tests are currently failing on Solaris > 9/x86 with Sun as: > > FAIL: gcc.target/i386/hle-add-acq-1.c scan-assembler lock[ > \\n\\t]+(xacquire|.byte[ \\t]+0xf2)[ \\t\\n]+add > > The .s file has > >lock; >

Re: [patch] HP-PA: use define_c_enum for "unspec" and "unspecv"

2012-05-08 Thread John David Anglin
On 5/8/2012 2:33 PM, Steven Bosscher wrote: Dave, could you please test this for me, and commit it if it is OK? Will do. Thanks, Dave -- John David Anglindave.ang...@bell.net

Re: [PATCH] libgcov support for profile collection in region of interest (issue6186044)

2012-05-08 Thread Teresa Johnson
On Tue, May 8, 2012 at 10:37 AM, Jan Hubicka wrote: > > > Hi Honza, > > > > I added L_gcov_reset and L_gcov_dump for the new interfaces, and also > > added a description into the gcov man page. Let me know if it looks > > ok now. > > > > Bootstrapped and tested on x86_64-unknown-linux-gnu. > > > >

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Marc Glisse
On Tue, 8 May 2012, Richard Sandiford wrote: Marc Glisse writes: Here is a new version. gcc/ChangeLog 2012-05-08 Marc Glisse * simplify-rtx.c (simplify_binary_operation_1): Optimize shuffle of concatenations. OK, thanks. I'll leave an x86 maintainer to review the testca

[patch] HP-PA: use define_c_enum for "unspec" and "unspecv"

2012-05-08 Thread Steven Bosscher
Hello, This patch makes pa.md use define_c_enum, so that UNSPECs are printed with nice strings. Unfortunately I can't bootstrap on gcc61 right now, so this is only tested with a normal "make" up to the failure point. Dave, could you please test this for me, and commit it if it is OK? Ciao! Stev

[PATCH] Fix endless loop in forwprop (PR tree-optimization/53226)

2012-05-08 Thread Jakub Jelinek
Hi! The attached testcase loops endlessly, using more and more memory. The problem is that the prev stmt iterator sometimes references stmts that remove_prop_source_from_use decides to remove, and since Michael's gimple seq changes that seems to be fatal. Fixed by not keeping an iterator, but ins

Re: patch ping: Add static branch predict heuristic of comparing IV to loop_bound variable

2012-05-08 Thread Jakub Jelinek
On Fri, May 04, 2012 at 09:46:22PM +0800, Dehao Chen wrote: > Thanks for the prompt response. Attached is the updated patch. > > Passed bootstrap and all regression tests. All the new testcases fail for me, on both x86_64-linux and i686-linux, apparently because of incorrectly committed patch (ea

Re: RFA: PR target/53120, constraint modifier "+" on operand tied by matching-constraint, "0".

2012-05-08 Thread DJ Delorie
#define q ((char *)0x1234) foo(int x) { *q |= (1 << (char)x); } $ m32c-elf-gcc -S -O3 nick.c .global _foo _foo: mov.w r1,a0; 20 movhi_op/3 bset4660[a0] ; 11

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Richard Sandiford
Marc Glisse writes: > Here is a new version. > > gcc/ChangeLog > 2012-05-08 Marc Glisse > > * simplify-rtx.c (simplify_binary_operation_1): Optimize shuffle > of concatenations. OK, thanks. I'll leave an x86 maintainer to review the testcase, but it looks like it'll need some mark

Re: [PATCH] MIPS16: Remove DWARF-2 location information from GP accesses

2012-05-08 Thread Richard Sandiford
"Maciej W. Rozycki" writes: > gcc-mips16-gp-pseudo-loc.patch > Index: gcc-fsf-trunk-quilt/gcc/config/mips/mips.c > === > --- gcc-fsf-trunk-quilt.orig/gcc/config/mips/mips.c 2012-05-02 > 23:42:46.185566469 +0100 > +++ gcc-fsf-trunk-

Re: [PATCH] MIPS16: Fix truncated DWARF-2 line information

2012-05-08 Thread Richard Sandiford
"Maciej W. Rozycki" writes: >> > and the mips-linux-gnu configuration is not ready yet for MIPS16 testing. >> >> Out of interest, what goes wrong? I've been testing -mabi=32/-mips16 on >> mips64-linux-gnu for some time without difficulty. > > I've thought some pieces are missing upstream, but p

Re: [PATCH] libgcov support for profile collection in region of interest (issue6186044)

2012-05-08 Thread Jan Hubicka
> Hi Honza, > > I added L_gcov_reset and L_gcov_dump for the new interfaces, and also > added a description into the gcov man page. Let me know if it looks > ok now. > > Bootstrapped and tested on x86_64-unknown-linux-gnu. > > Thanks, > Teresa > > 2012-05-08 Teresa Johnson > > * libg

[testsuite] Fix gcc.target/i386/hle-* testcases with Sun as

2012-05-08 Thread Rainer Orth
Several /gcc.target/i386/hle-*.c tests are currently failing on Solaris 9/x86 with Sun as: FAIL: gcc.target/i386/hle-add-acq-1.c scan-assembler lock[ \\n\\t]+(xacquire|.byte[ \\t]+0xf2)[ \\t\\n]+add The .s file has lock; .byte 0xf2 but the scan-assembler regex currently doesn

Re: [google-4_6] fix profile mismatch in stream LIPO (issue6194059)

2012-05-08 Thread Xinliang David Li
Ok. David On Tue, May 8, 2012 at 9:53 AM, Rong Xu wrote: > Hi, > > This patch is for google-4_6 branch only. > > It fixes a profile mismatch in streaming LIPO and recovers the performance > loss > due to this. > > Tested with google internal benchmarks. > > Thanks, > > 2012-05-08   Rong Xu   >

[google-4_6] fix profile mismatch in stream LIPO (issue6194059)

2012-05-08 Thread Rong Xu
Hi, This patch is for google-4_6 branch only. It fixes a profile mismatch in streaming LIPO and recovers the performance loss due to this. Tested with google internal benchmarks. Thanks, 2012-05-08 Rong Xu * ipa-inline.c (fixed_arg_function_p): (better_inline_comdat_functi

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Marc Glisse
Here is a new version. gcc/ChangeLog 2012-05-08 Marc Glisse * simplify-rtx.c (simplify_binary_operation_1): Optimize shuffle of concatenations. gcc/testsuite/ChangeLog 2012-05-08 Marc Glisse * gcc.target/i386/shuf-concat.c: New test. On Tue, 8 May 2012, Richard

Re: [patch, fortran] PR fortran/52537 Optimize string comparisons against empty strings

2012-05-08 Thread Tobias Burnus
Hello Thomas, below a very timely review - your patch is not even a month old and was never pinged, besides, you have chosen an unlucky day. (In other words: Sorry for the slow review.) Thomas Koenig wrote on Fri, 13 Apr 2012: > this patch replaces a != '' with len_trim(a) != 0, to > speed up th

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Gabriel Dos Reis
On Tue, May 8, 2012 at 11:20 AM, Sterling Augustine wrote: > Do you have a suggestion for fixing the disagreement? I would love to > add this as a parameter somewhere, but the decision is very deep in > the internals of the pretty printer. I disagree with your characterization that the pretty-pr

Re: [C++ Patch and pubnames 2/2] Adjust c decl pretty printer to match demangler (issue6195056)

2012-05-08 Thread Sterling Augustine
On Mon, May 7, 2012 at 6:44 PM, Gabriel Dos Reis wrote: > On Mon, May 7, 2012 at 7:10 PM, Sterling Augustine > wrote: >> This is the second in the series of patches to make c decl pretty printing >> more closely match the demangler. A full explanation is here: >> >> http://gcc.gnu.org/ml/gcc-patc

Re: [PATCH] Add -feliminate-malloc to enable/disable elimination of redundant malloc/free pairs

2012-05-08 Thread Xinliang David Li
To be clear, this flag is for malloc implementation (such as tcmalloc) with side effect unknown to the compiler. Using -fno-builtin-xxx is too conservative for that purpose. David On Tue, May 8, 2012 at 7:43 AM, Dehao Chen wrote: > Hello, > > This patch adds a flag to guard the optimization that

Re: RFA: PR target/53120, constraint modifier "+" on operand tied by matching-constraint, "0".

2012-05-08 Thread nick clifton
Hi DJ, Make sure a match_dup will still match the generated pattern later, I've had problems with match_dup not matching two rtx that rtx_equals() says are "the same" but not physically the same. I have tried, but failed, to find a way to trigger the use of the bset_qi pattern. :-( I tried re

Re: [RFC PATCH, i386]: Implement ix86_set_reg_reg_cost, fix PR 53250 "Splitting reg" failure.

2012-05-08 Thread Uros Bizjak
On Mon, May 7, 2012 at 10:40 PM, Uros Bizjak wrote: > 2012-05-07  Uros Bizjak   > >        PR target/53250 >        * config/i386/i386.c (ix86_set_reg_reg_cost): New function. >        (ix86_rtx_costs): Handle SET. > >> Patch was tested on x86_64-pc-linux-gnu {,-m32}. >> >> I have also #define LO

Re: rfa: shrink rtl_bb_info

2012-05-08 Thread Michael Matz
Hi, On Fri, 4 May 2012, Richard Guenther wrote: > Well, as you touch all places that refer to header/footer why not > introduce macros to access them ... > > > Currently regstrapping on x86_64-linux, okay if that passes? > > Ok with the above change. For completeness, this is what I checked in

Re: [PATCH libcpp]: Avoid crash in interpret_float_suffix

2012-05-08 Thread Tom Tromey
> "Tristan" == Tristan Gingold writes: Tristan> 2012-05-04 Tristan Gingold Tristan>* expr.c (interpret_float_suffix): Add a guard. Ok. Tom

Re: [committed] Fix lower-subreg cost calculation

2012-05-08 Thread Richard Earnshaw
On 06/05/12 19:55, Richard Sandiford wrote: > Georg-Johann Lay writes: >> TARGET_RTX_COSTS gets called with x = (const_int 1) and outer = SET >> for example. How do I get SET_DEST from that information? >> >> I don't now if lower-subreg.s ever emits such cost requests, but several >> passes defini

[PATCH][Cilkplus] Handling elemental function for C Compiler.

2012-05-08 Thread Iyer, Balaji V
Hello Everyone, This patch is for the Cilkplus branch affecting mostly the C compiler. This patch will insert elemental functions for C programs. Thanks, Balaji V. Iyer.diff --git a/gcc/ChangeLog.cilk b/gcc/ChangeLog.cilk index d6b28e2..4f01d31 100644 --- a/gcc/ChangeLog.cilk +++ b/gcc/Chang

[PATCH] Add -feliminate-malloc to enable/disable elimination of redundant malloc/free pairs

2012-05-08 Thread Dehao Chen
Hello, This patch adds a flag to guard the optimization that optimize the following code away: free (malloc (4)); In some cases, we'd like this type of malloc/free pairs to remain in the optimized code. Tested with bootstrap, and no regression in the gcc testsuite. Is it ok for mainline? Than

Re: PR 53249: Multiple address modes for same address space

2012-05-08 Thread H.J. Lu
On Sun, May 6, 2012 at 11:41 AM, Richard Sandiford wrote: > x32 uses a mixture of MEM address modes for the same address space. > Some MEMs have SImode addresses, some have DImode.  This means that > the currently common idiom: > >    targetm.addr_space.address_mode (MEM_ADDR_SPACE (mem)) > > isn'

Re: [RS6000] Fix PR53271 powerpc ports

2012-05-08 Thread David Edelsohn
On Tue, May 8, 2012 at 6:32 AM, Alan Modra wrote: > OK, the assert is doing its job.  I wanted to minimize the number of > places that need temporary hard regs, so that tracking of which hard > reg is in use can all be done in rs6000_emit_prologue. > > The problem is that the insns here need reg+

Re: [PATCH] gcc/config/freebsd-spec.h: Fix building PIE executables. Link them with crt{begin,end}S.o and Scrt1.o which are PIC instead of crt{begin,end}.o and crt1.o which are not. Spec synced from g

2012-05-08 Thread Alexis Ballier
For the record, there's a similar logic in FreeBSD's gcc: http://svnweb.freebsd.org/base/head/contrib/gcc/config/freebsd-spec.h?revision=200038&view=markup Regards, Alexis.

[PATCH] gcc/config/freebsd-spec.h: Fix building PIE executables. Link them with crt{begin,end}S.o and Scrt1.o which are PIC instead of crt{begin,end}.o and crt1.o which are not. Spec synced from gnu-u

2012-05-08 Thread Alexis Ballier
gcc/config/i386/freebsd.h: Likewise. --- gcc/config/freebsd-spec.h |9 +++-- gcc/config/i386/freebsd.h |9 +++-- 2 files changed, 6 insertions(+), 12 deletions(-) diff --git a/gcc/config/freebsd-spec.h b/gcc/config/freebsd-spec.h index 770a3d1..2808582 100644 --- a/gcc/config/free

[PATCH] libgcov support for profile collection in region of interest (issue6186044)

2012-05-08 Thread Teresa Johnson
Hi Honza, I added L_gcov_reset and L_gcov_dump for the new interfaces, and also added a description into the gcov man page. Let me know if it looks ok now. Bootstrapped and tested on x86_64-unknown-linux-gnu. Thanks, Teresa 2012-05-08 Teresa Johnson * libgcc/libgcov.c (gcov_clear,

GCC: Add microblaze-*-rtems* (w/ diff)

2012-05-08 Thread Joel Sherrill
Sorry.. missed the attachment This patch adds the microblaze-*-rtems* target to gcc. OK to apply? 2012-05-07 Joel Sherrill * config.gcc (microblaze-*-rtems*): New target. * config/microblaze/rtems.h: New file. -- Joel Sherrill, Ph.D. Director of Research& Development joe

GCC: Add microblaze-*-rtems*

2012-05-08 Thread Joel Sherrill
This patch adds the microblaze-*-rtems* target to gcc. OK to apply? 2012-05-07 Joel Sherrill * config.gcc (microblaze-*-rtems*): New target. * config/microblaze/rtems.h: New file -- Joel Sherrill, Ph.D. Director of Research& Development joel.sherr...@oarcorp.comO

Re: [C++ Patch] PR 53158

2012-05-08 Thread Paolo Carlini
On 05/08/2012 03:00 PM, Jason Merrill wrote: On 05/07/2012 11:28 PM, Paolo Carlini wrote: error: could not convert ‘b.main()::()’ from ‘void’ to ‘bool’ It wouldn't say "operator()"? Nope, it says exactly the above. If you tell me what would be more sensible which remaining informative, I can

Re: [patch] support for multiarch systems

2012-05-08 Thread Joseph S. Myers
On Tue, 8 May 2012, Matthias Klose wrote: > On 20.08.2011 21:51, Matthias Klose wrote: > > Multiarch [1] is the term being used to refer to the capability of a system > > to > > install and run applications of multiple different binary targets on the > > same > > system. > > please find attache

Re: [C++ Patch] PR 53158

2012-05-08 Thread Jason Merrill
On 05/07/2012 11:28 PM, Paolo Carlini wrote: error: could not convert ‘b.main()::()’ from ‘void’ to ‘bool’ It wouldn't say "operator()"? I think I'd leave that alone; it is somewhat more informative (about what b() expands to) and we're moving toward replacing %qE with caret anyway. The pa

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Richard Sandiford
Marc Glisse writes: >> Realised afterwards that both versions need to check >> GET_MODE_NUNITS (mode) == 2, because we're requiring OP0 and OP1 >> to be scalar. Sorry for not noticing first time. > > I thought that was a consequence of > > XVECLEN (trueop1, 0) == 2 > > (in the lines before your f

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Marc Glisse
On Tue, 8 May 2012, Richard Sandiford wrote: Marc Glisse writes: On Tue, 8 May 2012, Richard Sandiford wrote: I know you said that generalising it could be done later, and that's fine, but it looks in some ways like it would be easier to go straight for the more general: && GET_COD

Re: [PATCH] Remove TYPE_IS_SIZETYPE

2012-05-08 Thread Richard Guenther
On Tue, 8 May 2012, Richard Guenther wrote: > > This removes the TYPE_IS_SIZETYPE macro and all its uses (by > assuming it returns zero and applying trivial folding). Sizes > and bitsizes can still be treat specially by means of knowing > what the values represent and by means of using helper fu

[PATCH] Remove TYPE_IS_SIZETYPE

2012-05-08 Thread Richard Guenther
This removes the TYPE_IS_SIZETYPE macro and all its uses (by assuming it returns zero and applying trivial folding). Sizes and bitsizes can still be treat specially by means of knowing what the values represent and by means of using helper functions that assume you are dealing with "sizes" (in pa

[PATCH] Optimize byte_from_pos, pos_from_bit

2012-05-08 Thread Richard Guenther
This optimizes byte_from_pos and pos_from_bit by noting that we operate on sizes whose computations have no intermediate (or final) overflow. This is the single patch necessary to get Ada to bootstrap and test with TYPE_IS_SIZETYPE removed. Rather than amending size_binop (my original plan) I cho

[PATCH] Fold (X * CST1) & CST2

2012-05-08 Thread Richard Guenther
This makes us fold (X * 8) & 5 to zero and (X * 6) & 5 to (X * 6) & 4. It amends the fix for PR52134 in that I noticed we fail to do some simplifications on size expressions (later on SSA, CCP does the above transform, though not (yet) changing of the constant). Bootstrap and regtest ongoing on x

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Richard Sandiford
Marc Glisse writes: > On Tue, 8 May 2012, Richard Sandiford wrote: >> I know you said that generalising it could be done later, >> and that's fine, but it looks in some ways like it would >> be easier to go straight for the more general: >> >>&& GET_CODE (trueop0) == VEC_CONCAT >>&

Re: PATCH: Update longlong.h from GLIBC

2012-05-08 Thread Andreas Jaeger
On Tuesday, May 08, 2012 11:59:34 Richard Earnshaw wrote: > On 08/05/12 10:04, Andreas Jaeger wrote: > > On Tuesday, May 08, 2012 10:43:14 Richard Guenther wrote: > >> On Mon, May 7, 2012 at 11:11 PM, H.J. Lu wrote: > >>> Hi, > >>> > >>> I am preparing to update GLIBC longlong.h from GCC. This p

[RS6000] Fix PR53271 powerpc ports

2012-05-08 Thread Alan Modra
On Mon, May 07, 2012 at 05:23:33PM +0200, Olivier Hainque wrote: > (emit_frame_save): Don't handle reg+reg addressing. > > introduces an assert on which we now trip compiling unwind-dw2.c for SPE > configurations. We now fall into the TARGET_SPE_ABI part of > >/* Some cases that need

Re: Patches to enable -ftrack-macro-expansion by default

2012-05-08 Thread Andreas Krebbel
On 04/30/2012 01:46 PM, Dodji Seketeli wrote: > Dodji Seketeli writes: > >> I am proposing a series of patches which is supposed to address the >> remaining issues (I am aware of) preventing us from enabling the >> -ftrack-macro-expansion by default. >> >> The idea is to address each issue I noti

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Marc Glisse
On Tue, 8 May 2012, Richard Sandiford wrote: I know you said that generalising it could be done later, and that's fine, but it looks in some ways like it would be easier to go straight for the more general: && GET_CODE (trueop0) == VEC_CONCAT && GET_CODE (XEXP (trueop0, 0))

Re: Speed up insn-attrtab.c compilation

2012-05-08 Thread Michael Matz
Hi, On Mon, 7 May 2012, Mike Stump wrote: > On May 7, 2012, at 6:11 AM, Michael Matz wrote: > > I'd like to retain the #if 0 code therein, > > Can you structure this code as > > #define DEBUG 0 > > if (DEBUG) ... > > ? > > If so, that would be a preferable way to structure the code. Sure,

[PATCH] MIPS16: Remove DWARF-2 location information from GP accesses

2012-05-08 Thread Maciej W. Rozycki
Hi, I have been investigating some failures in the MIPS16 GDB test suite that appeared to me that was caused by some weird out-of place line information. For example we get this in gdb.base/break.exp, where a breakpoint is set at main and the beginning of the function (after removing some "

Re: PATCH: Update longlong.h from GLIBC

2012-05-08 Thread Richard Earnshaw
On 08/05/12 10:04, Andreas Jaeger wrote: > On Tuesday, May 08, 2012 10:43:14 Richard Guenther wrote: >> On Mon, May 7, 2012 at 11:11 PM, H.J. Lu wrote: >>> Hi, >>> >>> I am preparing to update GLIBC longlong.h from GCC. This patch >>> updates GCC longlong.h to use a URL instead of an FSF postal a

Re: [rtl, patch] combine concat+shuffle

2012-05-08 Thread Richard Sandiford
Looks like a good idea. Marc Glisse writes: > For the testsuite, since the patch is not in a particular target, it would > be better to have a generic test (in gcc.dg?), but I don't really know how > to write a generic one, so would a test in gcc.target/i386 that scans > the asm for shuf or pe

Re: PATCH: Update longlong.h from GLIBC

2012-05-08 Thread Andreas Jaeger
On Tuesday, May 08, 2012 10:43:14 Richard Guenther wrote: > On Mon, May 7, 2012 at 11:11 PM, H.J. Lu wrote: > > Hi, > > > > I am preparing to update GLIBC longlong.h from GCC. This patch > > updates GCC longlong.h to use a URL instead of an FSF postal address > > and replace spaces with tab. O

Re: [PATCH] Add option for dumping to stderr (issue6190057)

2012-05-08 Thread Richard Guenther
On Tue, May 8, 2012 at 12:02 AM, Gabriel Dos Reis wrote: > On Mon, May 7, 2012 at 4:58 PM, Sharad Singhai wrote: >> This is the first patch for planned improvements to dump >> infrastructure.  Please reference the discussion in >> http://gcc.gnu.org/ml/gcc-patches/2011-10/msg02088.html. >> >> The

Re: Heads-up, PR53273: testsuite separation and dilution problem. Fix for PR53272

2012-05-08 Thread Richard Guenther
On Tue, May 8, 2012 at 5:39 AM, Hans-Peter Nilsson wrote: > The problem was spotted while fixing PR53272, a target bug with > crisv32-* involving the error-prone notice_update_cc function. > > When wrapping up the test-case to use as a run-test, adding main > and auxiliary functions to the reduced

Re: [patch] don't check for execute bits of the liblto plugin

2012-05-08 Thread Richard Guenther
On Tue, May 8, 2012 at 1:07 AM, Matthias Klose wrote: > The lto plugin is installed without x bits set, but gcc-ar.c still checks for > the execute bits. There is no need to have the lto plugin to have the x bits > set, so just check that it is readable. > > Ok for the trunk and the 4.7 branch? O

Re: PATCH: Update longlong.h from GLIBC

2012-05-08 Thread Richard Guenther
On Mon, May 7, 2012 at 11:11 PM, H.J. Lu wrote: > Hi, > > I am preparing to update GLIBC longlong.h from GCC.  This patch updates > GCC longlong.h to use a URL instead of an FSF postal address and  replace > spaces with tab.  OK to install? > > Since I'd like to simply copy longlong.h from GCC rel

Re: [PATCH] MIPS16: Fix truncated DWARF-2 line information

2012-05-08 Thread Maciej W. Rozycki
Hi Richard, Resurrecting the issue now that I have new data. On Wed, 14 Dec 2011, Richard Sandiford wrote: > > After some thinking I decided the simplest approach will be just emitting > > the missing location directive in the context of the MIPS16 thunk being > > built that will apply to th

[ping] 3 pending patches

2012-05-08 Thread Eric Botcazou
Fix debug info of nested inline functions: http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00161.html Emit variable as size attribute in debug info: http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00422.html Implement static stack checking on IA-64: http://gcc.gnu.org/ml/gcc-patches/2012-03/msg00

Re: [SH] PR 51244 - Supplementary patch

2012-05-08 Thread Kaz Kojima
Oleg Endo wrote: > The attached patch is the same as in the PR comment #37. > Tested with > make -k check RUNTESTFLAGS="--target_board=sh-sim > \{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb, > -m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" > > and no new failures. >

[SH] PR 51244 - Supplementary patch

2012-05-08 Thread Oleg Endo
Hello, The attached patch is the same as in the PR comment #37. Tested with make -k check RUNTESTFLAGS="--target_board=sh-sim \{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb, -m4-single/-ml,-m4-single/-mb,-m4a-single/-ml,-m4a-single/-mb}" and no new failures. OK? Cheers, Oleg Change

Re: [PATCH] Add option for dumping to stderr (issue6190057)

2012-05-08 Thread Sharad Singhai
That is certainly a possibility. The original motivation was to implement -fopt-info correctly. If there are other use cases, then I can enhance the patch. Thanks, Sharad On Mon, May 7, 2012 at 3:02 PM, Gabriel Dos Reis wrote: > On Mon, May 7, 2012 at 4:58 PM, Sharad Singhai wrote: >> This is