Re: [PATCH v6] arm: use built-in byte swap function

[Woodhouse, David]

On Fri, 2013-02-22 at 19:40 -0600, Kim Phillips wrote:
> Enable the compiler intrinsic for byte swapping on arch ARM.  This
> allows the compiler to detect and be able to optimize out byte
> swappings, and has a tiny benefit on vmlinux size (Linaro gcc 4.7.3):
> 
>    text    data     bss     dec     hex filename
> 2754100  121144   56520 2931764  2cbc34 vmlinux-lart #orig
> 2754050  121144   56520 2931714  2cbc02 vmlinux-lart #builtin-bswap
> 6282699  307852 5578076 12168627 b9adb3 vmlinux-mxs #orig
> 6282241  307832 5578076 12168149 b9abd5 vmlinux-mxs #builtin-bswap
> 7200193  364180  361748 7926121  78f169 vmlinux-imx_v6_v7 #orig
> 7199515  364188  361748 7925451  78eecb vmlinux-imx_v6_v7 #builtin-bswap
> 
> Signed-off-by: Kim Phillips <kim.phillips@xxxxxxxxxxxxx>

Looks good, thanks.

Acked-by: David Woodhouse <David.Woodhouse@xxxxxxxxx>


-- 
                   Sent with MeeGo's ActiveSync support.

David Woodhouse                            Open Source Technology Centre
David.Woodhouse@xxxxxxxxx                              Intel Corporation

Attachment: smime.p7s
Description: S/MIME cryptographic signature