This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [SH] PR 51244 - Fix defects introduced in 4.8
- From: Oleg Endo <oleg dot endo at t-online dot de>
- To: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 08 Oct 2013 00:21:48 +0200
- Subject: Re: [SH] PR 51244 - Fix defects introduced in 4.8
- Authentication-results: sourceware.org; auth=none
- References: <1380877594 dot 4778 dot 23 dot camel at yam-132-YW-E178-FTW> <20131005 dot 075401 dot 207870028 dot kkojima at rr dot iij4u dot or dot jp> <1381050886 dot 4778 dot 33 dot camel at yam-132-YW-E178-FTW> <20131007 dot 074426 dot 234252419 dot kkojima at rr dot iij4u dot or dot jp>
On Mon, 2013-10-07 at 07:44 +0900, Kaz Kojima wrote:
> Oleg Endo <oleg.endo@t-online.de> wrote:
> > Forgot to handle a case in function can_remove_cstore, thanks for
> > catching it. Fixed in the attached patch and also added test cases.
> > Retested as before without new failures.
>
> Ok for trunk.
>
> > Yeah, right. I've changed 'ifcvt_sh' to 'sh_ifcvt'.
>
> >+ register_pass (make_pass_sh_ifcvt (g, false, "ifcvt1_sh"),
> >+ PASS_POS_INSERT_AFTER, "ce1", 1);
> >+*/
>
> s/ifcvt1_sh/sh_ifcvt1/ might be better even in a comment.
Sorry, I missed one. Will fix and resend the committed patch after we
have agreed on the pass name (see Christian's message).
Cheers,
Oleg