[ARM] Define unspecs using define_c_enum

Richard Sandiford richard.sandiford@linaro.org
Wed Mar 30 15:03:00 GMT 2011


Richard Earnshaw <rearnsha@arm.com> writes:
> On Tue, 2011-03-29 at 15:38 +0100, Richard Sandiford wrote:
>> This ARM patch allows *UNSPEC_* constants to be printed in dump files.
>> It's very much a target decision whether this is worth doing, but just
>> in case...
>> 
>> ...tested on arm-linux-gnueabi.  OK to install?
>> 
>> Richard
>> 
>> 
>> gcc/
>> 	* config/arm/neon.md: Use define_c_enum to define UNSPEC* and
>> 	VUNSPEC* constants.
>> 	* config/arm/arm.md: Likewise.  Move the synchronization ones to...
>> 	* config/arm/sync.md: ...here.
>> 
>
>
> Hmm, seems we have two patches posted to do this.  I've just approved an
> earlier post that came in while we were in pre-release lock-down.

Oops, sorry, I missed Yufeng's patch.  FWIW, the differences in my patch
are that I've got rid of the (documented as) unused UNSPEC_SIN and
UNSPEC_COS and moved the sync stuff to sync.md.  I assumed the old
code did things that way in order to make the numbering more obvious,
but that isn't an issue any more.

Richard



More information about the Gcc-patches mailing list