Re: [patch] speed up ifcvt:cond_move_convert_if_block

2012-08-18 Thread Steven Bosscher
On Thu, Aug 16, 2012 at 12:06 PM, Richard Guenther wrote: > On Thu, Aug 16, 2012 at 1:11 AM, Steven Bosscher > wrote: >> On Mon, Aug 6, 2012 at 1:27 PM, Paolo Bonzini wrote: 2. sparseset has the same problem of memory clearing (for valgrind, see sparseset_alloc). >>> >>> ... only the

Re: [patch] speed up ifcvt:cond_move_convert_if_block

2012-08-16 Thread Richard Guenther
On Thu, Aug 16, 2012 at 1:11 AM, Steven Bosscher wrote: > On Mon, Aug 6, 2012 at 1:27 PM, Paolo Bonzini wrote: >>> 2. sparseset has the same problem of memory clearing (for valgrind, >>> see sparseset_alloc). >> >> ... only the sparse array needs this clearing, but currently we do it >> for both.

Re: [patch] speed up ifcvt:cond_move_convert_if_block

2012-08-15 Thread Steven Bosscher
On Mon, Aug 6, 2012 at 1:27 PM, Paolo Bonzini wrote: >> 2. sparseset has the same problem of memory clearing (for valgrind, >> see sparseset_alloc). > > ... only the sparse array needs this clearing, but currently we do it > for both. And according to the fat comment before the xcalloc, it's not

Re: [patch] speed up ifcvt:cond_move_convert_if_block

2012-08-06 Thread Paolo Bonzini
Il 06/08/2012 13:15, Steven Bosscher ha scritto: > On Mon, Aug 6, 2012 at 1:07 PM, Paolo Bonzini wrote: >> Il 06/08/2012 08:54, Steven Bosscher ha scritto: >>> Hello, >>> >>> In PR54146, ifcvt spends a lot of time just clearing memory. This >>> patch changes the value maps to pointer-maps to fix t

Re: [patch] speed up ifcvt:cond_move_convert_if_block

2012-08-06 Thread Steven Bosscher
On Mon, Aug 6, 2012 at 1:07 PM, Paolo Bonzini wrote: > Il 06/08/2012 08:54, Steven Bosscher ha scritto: >> 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-l

Re: [patch] speed up ifcvt:cond_move_convert_if_block

2012-08-06 Thread Paolo Bonzini
Il 06/08/2012 08:54, Steven Bosscher ha scritto: > 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? Nice, but perhaps we need a sparsemap to do e

Re: [patch] speed up ifcvt:cond_move_convert_if_block

2012-08-06 Thread Richard Guenther
On Mon, Aug 6, 2012 at 8:54 AM, Steven Bosscher wrote: > 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? Ok! Thanks, Richard. > Ciao! > Steven

[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