This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

Re: [PATCH] arm.md: Use the revsh and rev16 instructions for BSWAP:HI RTL in armv6 and up

From: Christophe Lyon <christophe dot lyon at st dot com>
To: Louis Kruger <louisk at gmail dot com>
Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Thu, 23 Aug 2012 17:09:10 +0200
Subject: Re: [PATCH] arm.md: Use the revsh and rev16 instructions for BSWAP:HI RTL in armv6 and up
References: <CAK65ue-_f9c0hBZZML6XDsEp_eEqESb5-0sh7PcSzazdo3kdEw@mail.gmail.com> <CACUk7=X+DZGZwKfJsyd=+okxuFG8aNNtuWJMwTHw+o0HeatscA@mail.gmail.com>

On 19.04.2012 09:05, Ramana Radhakrishnan wrote:

On 16 April 2012 20:35, Louis Kruger <louisk@gmail.com> wrote:

This patch teaches the arm target about the rev16 and revsh
instructions which can
efficiently execute the recently added __builtin_bswap16() function
(BSWAP:HI in RTL)
These instructions are available in thumb and arm modes on armv6
architecture and up

Thank you for your patch but before taking this further , I have to
ask if  you have a valid copyright assignment on file with the FSF .I
would like to also see some testcases for this feature added to the
regression testsuite in gcc.target/arm.


regards,
Ramana

Louis,

Did you have time to sign a valid copyright assignment with FSF since you original patch?

If yes, do you plan to re-submit a patch taking Ramana's comments into account?

If no, do you mind if I propose a patch myself (maybe based on yours) ?

Thanks,

Christophe.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]