[Patch] catch builtin_bswap16 construct

Christophe Lyon christophe.lyon@linaro.org
Tue Sep 25 11:07:00 GMT 2012


On 25 September 2012 07:00, Segher Boessenkool
<segher@kernel.crashing.org> wrote:
> Christophe, it looks like the zero-extend in the unsigned case is not
> needed on any target?  Assuming the shifts are at least SImode, of
> course (I'm too lazy to check, sorry).
>

It's also present when compiling:
unsigned short swapu16(unsigned short x) {
     return __builtin_bswap16(x);
}

so it's not directly caused by my patch I think.

We have to look at the __builtin_bswap16 expansion with an unsigned argument.

Christophe.



More information about the Gcc-patches mailing list