On 14/02/12 18:01, Andrew Stubbs wrote: > And again with the attachment. > > On Tue 14 Feb 2012 18:00:57 GMT, Andrew Stubbs wrote: >> On Tue 14 Feb 2012 17:35:36 GMT, Richard Earnshaw wrote: >>>> Bernds checked in a patch last year (or maybe even before that) to make >>>> the selection of flags clobbered insns run very late (certainly after >>>> condexec had run), so I'm not sure why you're not seeing this. >>> >>> Hm, you mentioned some peepholes. Try removing them.... >> >> Hmmm, it seems you're right. The machine reorg pass now takes care of >> this. Well ... that was easy! >> >> Here's a patch to remove the obsolete peepholes. I've confirmed it >> works with the testcase. >> >> OK? >> >> Andrew >> >> thumb-ifcvt2.patch >> >> >> 2012-02-14 Andrew Stubbs <a...@codesourcery.com> >> >> gcc/ >> * config/arm/thumb2.md: Delete obsolete peepholes. >> >> gcc/testsuite/ >> * gcc.target/arm/thumb-ifcvt.c: New test case. >>
s/obsolete peepholes/obsolete flag-clobbering peepholes/ Otherwise ok for stage1 The 'Similarly' that you delete suggests there might still be more of them... R.
