This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [SH] PR 39423
- From: Richard Henderson <rth at redhat dot com>
- To: Oleg Endo <oleg dot endo at t-online dot de>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Mon, 30 Jul 2012 08:28:38 -0700
- Subject: Re: [SH] PR 39423
- References: <1343602608.2304.15.camel@yam-132-YW-E178-FTW>
On 2012-07-29 15:56, Oleg Endo wrote:
> + "&& can_create_pseudo_p ()"
> + [(set (match_dup 5) (ashift:SI (match_dup 1) (match_dup 2)))
> + (set (match_dup 6) (plus:SI (match_dup 5) (match_dup 3)))
> + (set (match_dup 0) (mem:SI (plus:SI (match_dup 6) (match_dup 4))))]
Don't create new mems like this -- you've lost alias info.
You need to use replace_equiv_address or something on the
original memory. Which means you have to actually capture
the memory operand somehow.
Better to use a custom predicate to match these memories with
these complex addresses, rather than list them out each time:
[(set (match_operand:SI 0 "arith_reg_dest" "=r")
(match_operand:SI 1 "mem_index_disp_operand" "m"))]
r~