On 08/03/12 18:03, Richard Henderson wrote:
> On 03/08/12 08:19, Andrew Stubbs wrote:
>> +   (set_attr "arch" "nota8,*,*,onlya8")
>> +   (set_attr_alternative "insn_enabled"
>> +	[(if_then_else (match_test "TARGET_NEON")
>> +		       (const_string "yes") (const_string "no"))
>> +	 (const_string "yes")
>> +	 (const_string "yes")
>> +	 (if_then_else (match_test "TARGET_NEON")
>> +		       (const_string "yes") (const_string "no"))])]
>>   )
>
> While this works, it might be better to add neon/neon_na8/neon_oa8
> alternatives to the arch attribute, and adjust arch_enabled to match.
>
> Obviously this opinion is non-binding; Richard E might have other plans...

No reply from Richard so far ... so here's an update.

OK now?

Andrew