[PATCH][ARM] one_cmpldi2 in NEON

Andrew Stubbs ams@codesourcery.com
Wed Dec 7 20:30:00 GMT 2011


On 07/12/11 16:25, Richard Earnshaw wrote:
>> 2011-12-06  Andrew Stubbs<ams@codesourcery.com>
>>
>> 	gcc/
>> 	* config/arm/arm.md (one_cmpldi2): Rename to ...
>> 	(one_cmpldi2_core): ... this, and modify it to prevent it being
>> 	used for NEON.
>> 	(one_cmpldi2): New define_expand.
>> 	* config/arm/neon.md (one_cmpldi2_neon): New define_insn.
>
>
>> +(define_insn_and_split "*one_cmpldi2_core"
>> +  [(set (match_operand:DI 0 "arm_general_register_operand" "=&r,&r")
>> +	(not:DI (match_operand:DI 1 "arm_general_register_operand" "0,r")))]
>
> Thinking about it, for an operation with one input and one output, there's no need for the
> earlyclobber marker when the input is tied to the output (there's no other operand that can be
> clobbered).
>
> Otherwise OK.

Thanks I'll test and commit the attached updated patch once stage 1 
opens again.

Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: neon-not.patch
Type: text/x-patch
Size: 1736 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20111207/518e0cc6/attachment.bin>


More information about the Gcc-patches mailing list