Re: [PATCH] Fix PR ipa/65087

2015-03-03 Thread Martin Liška
On 03/03/2015 10:50 AM, Marek Polacek wrote: On Tue, Mar 03, 2015 at 10:31:17AM +0100, Martin Liška wrote: @@ -2952,9 +2959,12 @@ sem_item_optimizer::merge_classes (unsigned int prev_class_count) alias->dump_to_file (dump_file); } - source->merge (alias)

Re: [PATCH] Fix PR ipa/65087

2015-03-03 Thread Marek Polacek
On Tue, Mar 03, 2015 at 10:31:17AM +0100, Martin Liška wrote: > @@ -2952,9 +2959,12 @@ sem_item_optimizer::merge_classes (unsigned int > prev_class_count) > alias->dump_to_file (dump_file); > } > > - source->merge (alias); > + if (source->merge (alias))

Re: [PATCH] Fix PR ipa/65087

2015-03-03 Thread Martin Liška
On 03/02/2015 06:20 PM, Jan Hubicka wrote: Hello. Following one line patch is fix for PR ipa/65087. No regression has been seen on x86_64-linux-pc. OK, it won't happend large programs, but can you arrange its execute method to return TODO_remove_functions only if some merging actually happened

Re: [PATCH] Fix PR ipa/65087

2015-03-02 Thread Jan Hubicka
> Hello. > > Following one line patch is fix for PR ipa/65087. No regression has been > seen on x86_64-linux-pc. OK, it won't happend large programs, but can you arrange its execute method to return TODO_remove_functions only if some merging actually happened? Patch is preapproved with this chang