This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 3/3] Enhance dumps of IVOPTS
- From: Martin LiÅka <mliska at suse dot cz>
- To: "Bin.Cheng" <amker dot cheng at gmail dot com>, Richard Biener <richard dot guenther at gmail dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>, Jan Hubicka <hubicka at ucw dot cz>
- Date: Wed, 11 May 2016 16:17:48 +0200
- Subject: Re: [PATCH 3/3] Enhance dumps of IVOPTS
- Authentication-results: sourceware.org; auth=none
- References: <cover dot 1461931011 dot git dot mliska at suse dot cz> <29780c07dc7da0d8f41aa120665072a4098910d8 dot 1461931011 dot git dot mliska at suse dot cz> <572C61B9 dot 6060001 at suse dot cz> <CAFiYyc3K6ybBht8b2HsLUW00QrOOT1JWFTjvDPB4ET=thCPFmQ at mail dot gmail dot com> <CAHFci2_6qnkz7yCFg_E+XjZ-fgkP-aHhuBfpUB3mDwxew1FfwA at mail dot gmail dot com>
On 05/10/2016 03:16 PM, Bin.Cheng wrote:
> Another way is to remove the use of id for struct iv_inv_expr_ent once
> for all. We can change iv_ca.used_inv_expr and cost_pair.inv_expr_id
> to pointers, and rename iv_inv_expr_ent.id to count and use this to
> record reference number in iv_ca. This if-statement on dump_file can
> be saved. Also I think it simplifies current code a bit. For now,
> there are id <-> struct maps for different structures in IVOPT which
> make it not straightforward.
Sound good to me, I will re-implement dump enhancement in suggested manner.
Martin