[patch] speed up ifcvt:cond_move_convert_if_block

2012-08-05 Thread Steven Bosscher
Hello, In PR54146, ifcvt spends a lot of time just clearing memory. This patch changes the value maps to pointer-maps to fix this issue. Bootstrapped&tested on x86_64-unknown-linux-gnu. OK? Ciao! Steven PR54146_ifcvt.diff Description: Binary data

Re: [PATCH] Improve ifcombine (PR 52005)

2012-08-05 Thread Marc Glisse
Hello, do you have an opinion on this patch (available here: http://gcc.gnu.org/ml/gcc-patches/2012-07/msg01352.html ) ? Or should we go back to my old patch, or Andrew's patch? On Thu, 26 Jul 2012, Marc Glisse wrote: Hello, here is a new version of the ifcombine patch, which handles Andrew

[PATCH] put exception tables for comdat functions in comdat, too

2012-08-05 Thread Sandra Loosemore
For C++ functions that are emitted to a comdat section (template instantiations and the like), the corresponding exception handling information is presently being emitted to the non-comdat .gcc_except_table section. This means that you can end up with multiple copies of the exception tables in

Re: [libiberty] add obstack macros (was Re: PR #53525 - track-macro-expansion performance regression)

2012-08-05 Thread Ian Lance Taylor
On Sun, Aug 5, 2012 at 11:44 AM, Dimitrios Apostolou wrote: > > No problem, if anyone else doesn't object I'll change those (in a second > patch, right?). Can you please resend the patch you propose to check into mainline? Thanks. Ian

Re: [C++ Patch] PR 54161

2012-08-05 Thread Paolo Carlini
On 08/06/2012 01:29 AM, Jason Merrill wrote: On reflection, I think I prefer the status quo. As long as the pedwarn is on by default, I don't mind letting people disable it with -Wno-pointer-arith. I hear you. Then, let's just close the PR. Thanks! Paolo.

Re: [PATCH][2/2] Remove referenced vars

2012-08-05 Thread H.J. Lu
On Thu, Aug 2, 2012 at 1:12 AM, Richard Guenther wrote: > On Wed, 1 Aug 2012, Richard Guenther wrote: > >> >> This is the combined patch, ontop of 2a and 2b this removes all >> gimple_referenced_vars and add_referenced_vars calls. >> >> Bootstrapped on x86_64-unknown-linux-gnu, testing in progress

[PATCH] Remove introduction of undefined overflow in emit_case_bit_test.

2012-08-05 Thread Tom de Vries
Richard, the code in emit_case_bit_tests currently introduces a MINUS_EXPR in signed type (without checking if signed types wrap or not), which could mean introduction of undefined overflow. This patch fixes this problem by performing the MINUS_EXPR in an unsigned type. Doing so also enables the

Re: [C++ Patch] PR 54161

2012-08-05 Thread Jason Merrill
On reflection, I think I prefer the status quo. As long as the pedwarn is on by default, I don't mind letting people disable it with -Wno-pointer-arith. Jason

Re: [C++ Patch] PR 54165

2012-08-05 Thread Jason Merrill
OK. Jason

[Patch, Fortran] PR 35831: checking for dummy procedures

2012-08-05 Thread Janus Weil
Hi all, here is a patch for a rather old PR, which deals with correctness checking for several cases, such as: 1) dummy procedures 2) proc-ptr assignments 3) type-bound procedure overloading The patch adds a new function 'check_result_characteristics' to do various checks on function results. Thi

Re: [libiberty] add obstack macros (was Re: PR #53525 - track-macro-expansion performance regression)

2012-08-05 Thread Dimitrios Apostolou
On Sat, 4 Aug 2012, Ian Lance Taylor wrote: On Fri, 3 Aug 2012, Ian Lance Taylor wrote: I'm not sure where you are looking. I only see one call to _obstack_begin in the gcc directory, and it could easily be replaced with a call to obstack_specify_allocation instead. In libcpp/ mostly, but o

RE: [PATCH, gcc/doc]: Document AMD btver2 enablement

2012-08-05 Thread Kumar, Venkataramanan
Hi Gerald, Thank you. I committed changes as per your review comments. Ref: http://gcc.gnu.org/viewcvs?view=revision&revision=190151 Regards, Venkat. > -Original Message- > From: Gerald Pfeifer [mailto:ger...@pfeifer.com] > Sent: Sunday, August 05, 2012 12:24 AM > To: Kumar, Venkatara

Re: PATCH: PR rtl-optimization/54157: [x32] -maddress-mode=long failures

2012-08-05 Thread Richard Sandiford
For the record, I can't approve this, but... "H.J. Lu" writes: > i386,md has > > (define_expand "extzv" > [(set (match_operand:SI 0 "register_operand") > (zero_extract:SI (match_operand 1 "ext_register_operand") > (match_operand:SI 2 "const8_operand") >