Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-08-19 Thread Jeff Law
On 08/05/2015 09:16 AM, Martin Liška wrote: 2015-07-09 Martin Liska * dbgcnt.def: Add new debug counter. * ipa-icf-gimple.c (func_checker::compare_ssa_name): Use newly added state flag. (func_checker::compare_memory_operand): Likewise. (func_checker::com

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-08-05 Thread Martin Liška
On 08/03/2015 07:38 PM, Jeff Law wrote: > On 07/16/2015 05:03 AM, Martin Liška wrote: >>> So a general question. We're passing in STRICT to several routines, which >>> is fine. But then we're also checking M_TAIL_MERGE_MODE. What's the >>> difference between the two? Can they be unified? >> >

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-08-03 Thread Jeff Law
On 07/20/2015 01:52 AM, Martin Liška wrote: Due to changes dump file, it will be necessary to amend scanning of dump files. Those changes are fine and can be installed once the prerequisites have been approved/installed. Thanks, Jeff

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-08-03 Thread Jeff Law
On 07/16/2015 05:03 AM, Martin Liška wrote: So a general question. We're passing in STRICT to several routines, which is fine. But then we're also checking M_TAIL_MERGE_MODE. What's the difference between the two? Can they be unified? Hello. I would say that STRICT is a bit generic mecha

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-08-03 Thread Martin Liška
PING. Thanks, Martin On 07/20/2015 09:51 AM, Martin Liška wrote: > On 07/16/2015 01:03 PM, Martin Liška wrote: >> On 07/09/2015 06:24 PM, Jeff Law wrote: >>> On 07/09/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-07-09 Martin Liska * dbgcnt.def: Add new debug co

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-07-20 Thread Martin Liška
On 07/16/2015 01:03 PM, Martin Liška wrote: > On 07/09/2015 06:24 PM, Jeff Law wrote: >> On 07/09/2015 07:56 AM, mliska wrote: >>> gcc/ChangeLog: >>> >>> 2015-07-09 Martin Liska >>> >>> * dbgcnt.def: Add new debug counter. >>> * ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-07-20 Thread Martin Liška
On 07/16/2015 01:03 PM, Martin Liška wrote: > On 07/09/2015 06:24 PM, Jeff Law wrote: >> On 07/09/2015 07:56 AM, mliska wrote: >>> gcc/ChangeLog: >>> >>> 2015-07-09 Martin Liska >>> >>> * dbgcnt.def: Add new debug counter. >>> * ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-07-16 Thread Martin Liška
On 07/09/2015 06:24 PM, Jeff Law wrote: > On 07/09/2015 07:56 AM, mliska wrote: >> gcc/ChangeLog: >> >> 2015-07-09 Martin Liska >> >> * dbgcnt.def: Add new debug counter. >> * ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag >> for strict mode. >> (func_checker::compar

Re: [PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-07-09 Thread Jeff Law
On 07/09/2015 07:56 AM, mliska wrote: gcc/ChangeLog: 2015-07-09 Martin Liska * dbgcnt.def: Add new debug counter. * ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag for strict mode. (func_checker::compare_memory_operand): Likewise. (func_che

[PATCH 1/3] tree-ssa-tail-merge: add IPA ICF infrastructure.

2015-07-09 Thread mliska
gcc/ChangeLog: 2015-07-09 Martin Liska * dbgcnt.def: Add new debug counter. * ipa-icf-gimple.c (func_checker::compare_ssa_name): Add flag for strict mode. (func_checker::compare_memory_operand): Likewise. (func_checker::compare_cst_or_decl): Handle if we