This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: builtin_bswap plus enhancements
- From: Ian Lance Taylor <iant at google dot com>
- To: Falk Hueffner <falk at debian dot org>
- Cc: Chris Lattner <clattner at apple dot com>, Eric Christopher <echristo at apple dot com>, "gcc-patches at gcc dot gnu dot org Patches" <gcc-patches at gcc dot gnu dot org>, paul at codesourcery dot com
- Date: 10 Aug 2006 13:52:57 -0700
- Subject: Re: builtin_bswap plus enhancements
- References: <44D29BA2.30206@apple.com> <2895761B-5D52-4C32-894C-B19DDFBD9F4D@apple.com> <87r6zotlzu.fsf@debian.org>
Falk Hueffner <falk@debian.org> writes:
> Chris Lattner <clattner@apple.com> writes:
>
> > Why not add bswap16 as well?
>
> It should be unnecessary, since any attempt to express it should be
> picked up by the rot idiom recognizer, and the backends should then
> emit optimal code for constant-8 rots (and if that doesn't actually
> happen, we should rather fix that).
bswap16 is still necessary, for consistency, and to avoid bug reports.
Eric indicated offline that he was going to work on that, but wanted
to get the first set of patches approved. At least, that is how I
interpreted what he said.
Ian