This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [ARM] Define unspecs using define_c_enum
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