[patch RFA] Fix PR rtl-optimization/35190

Kaz Kojima kkojima@rr.iij4u.or.jp
Sat Feb 16 00:34:00 GMT 2008


"Steven Bosscher" <stevenb.gcc@gmail.com> wrote:
> This seems like just a way to paper over the problem.
> 
> If local-alloc makes a substitution that results in a bad insn, then
> the validation of the substitution should fail. The SH target specific
> code should just not accept this.   Or, you could test for
> HAS_LONG_UNCOND_BRANCH like bb-reorder.c does.  There is nothing I see
> in this bug report that is limited to hot/cold splitting, you can
> probably trigger this bug with a carefully crafted test case. Finally,
> it's not clear (at least to me) whether there are other substitutions
> that your patch would disable.

Ok, I withdraw this patch and will prepare a SH back end patch.
Thanks for your suggestion.

Regards,
	kaz



More information about the Gcc-patches mailing list