Messages in this thread | | | From | "Woodhouse, David" <> | Subject | Re: [PATCH v6] arm: use built-in byte swap function | Date | Sat, 23 Feb 2013 23:20:56 +0000 |
| |
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@freescale.com>
Looks good, thanks.
Acked-by: David Woodhouse <David.Woodhouse@intel.com>
-- Sent with MeeGo's ActiveSync support.
David Woodhouse Open Source Technology Centre David.Woodhouse@intel.com Intel Corporation
[unhandled content-type:application/x-pkcs7-signature] | |