This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix PR70484, RTL DSE using wrong dependence check
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Richard Biener <rguenther at suse dot de>
- Cc: Bernd Schmidt <bschmidt at redhat dot com>, gcc-patches at gcc dot gnu dot org
- Date: Tue, 5 Apr 2016 11:51:24 +0200
- Subject: Re: [PATCH] Fix PR70484, RTL DSE using wrong dependence check
- Authentication-results: sourceware.org; auth=none
- References: <alpine dot LSU dot 2 dot 11 dot 1604011106140 dot 13384 at t29 dot fhfr dot qr> <56FE931D dot 4030604 at redhat dot com> <alpine dot LSU dot 2 dot 11 dot 1604041116510 dot 13384 at t29 dot fhfr dot qr> <20160404095002 dot GC19207 at tucnak dot redhat dot com> <alpine dot LSU dot 2 dot 11 dot 1604051100460 dot 13384 at t29 dot fhfr dot qr> <20160405091849 dot GK19207 at tucnak dot redhat dot com> <alpine dot LSU dot 2 dot 11 dot 1604051148140 dot 13384 at t29 dot fhfr dot qr>
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
On Tue, Apr 05, 2016 at 11:48:29AM +0200, Richard Biener wrote:
> Like this?
Yeah, thanks.
> 2016-04-05 Richard Biener <rguenther@suse.de>
>
> * dse.c (struct store_info): Remove alias_set member.
> (struct read_info_type): Likewise.
> (clear_alias_group, clear_alias_mode_table, clear_alias_mode_holder,
> spill_deleted, clear_alias_set_lookup): Remove.
> (get_group_info): Remove dead base == NULL_RTX case.
> (dse_step0): Remove initialization of removed variables.
> (delete_dead_store_insn): Reomve alias set dumping.
> (free_read_records): Remove alias_set handling.
> (canon_address): Remove alias_set_out parameter.
> (record_store): Remove spill_alias_set, it's always zero.
> (check_mem_read_rtx): Likewise.
> (dse_step2): Rename from ...
> (dse_step2_nospill): ... this. Adjust.
> (scan_stores): Rename from ...
> (scan_stores_nospill): ... this.
> (scan_reads): Rename from ...
> (scan_reads_nospill): ... this.
> (scan_stores_spill, scan_reads_spill): Remove.
> (dse_step3_scan): Remove for_spills argument which is always false.
> (dse_step3): Likewise.
> (dse_step5): Rename from ...
> (dse_step5_nospill): ... this. Remove alias_set handling.
> (rest_of_handle_dse): Adjust.
Jakub