On Mon, 2013-10-07 at 07:44 +0900, Kaz Kojima wrote: > Oleg Endo <oleg.e...@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
