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

2013-02-23 Thread 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): > >textdata bss dec

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

2013-02-23 Thread 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): textdata bss dec

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

2013-02-22 Thread Nicolas Pitre
On Fri, 22 Feb 2013, Kim Phillips wrote: > On Thu, 21 Feb 2013 22:40:08 -0500 > Nicolas Pitre wrote: > > > On Thu, 21 Feb 2013, Kim Phillips wrote: > > > > > Here's the asm version I'm working on now, based on compiler > > > output of the C version. Haven't tested beyond defconfig builds, > >

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

2013-02-22 Thread Kim Phillips
On Thu, 21 Feb 2013 22:40:08 -0500 Nicolas Pitre wrote: > On Thu, 21 Feb 2013, Kim Phillips wrote: > > > Here's the asm version I'm working on now, based on compiler > > output of the C version. Haven't tested beyond defconfig builds, > > which pass ok. > > > > Is there anything I have to do

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

2013-02-22 Thread Kim Phillips
On Thu, 21 Feb 2013 22:40:08 -0500 Nicolas Pitre n...@fluxnic.net wrote: On Thu, 21 Feb 2013, Kim Phillips wrote: Here's the asm version I'm working on now, based on compiler output of the C version. Haven't tested beyond defconfig builds, which pass ok. Is there anything I have to

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

2013-02-22 Thread Nicolas Pitre
On Fri, 22 Feb 2013, Kim Phillips wrote: On Thu, 21 Feb 2013 22:40:08 -0500 Nicolas Pitre n...@fluxnic.net wrote: On Thu, 21 Feb 2013, Kim Phillips wrote: Here's the asm version I'm working on now, based on compiler output of the C version. Haven't tested beyond defconfig builds,