This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH][ARM] Improve use of conditional execution in thumb mode.


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  <ams@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.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]