Re: [PATCH][AArch64] Use std::swap instead of manually swapping in aarch64-ldpstp.md

2015-02-04 Thread Marcus Shawcroft
On 4 February 2015 at 12:18, Kyrill Tkachov wrote: > Hi all, > > This patch makes use of std::swap in every peephole2 of aarch64-ldp-stp.md > instead of manually swapping rtxen. > No functional change, just a cleanup. > Bootstrapped and tested on aarch64. > > I'm proposing this for next stage1 tog

Re: [PATCH][AArch64] Use std::swap instead of manually swapping in aarch64-ldpstp.md

2015-02-04 Thread James Greenhalgh
On Wed, Feb 04, 2015 at 12:18:29PM +, Kyrill Tkachov wrote: > Hi all, > > This patch makes use of std::swap in every peephole2 of > aarch64-ldp-stp.md instead of manually swapping rtxen. > No functional change, just a cleanup. > Bootstrapped and tested on aarch64. > > I'm proposing this for

Re: [PATCH][AArch64] Use std::swap instead of manually swapping

2014-12-12 Thread Mike Stump
On Dec 12, 2014, at 3:36 AM, Kyrill Tkachov wrote: > Marcus: Uros pointed out to me that these kinds of changes are considered > obvious > (with precedent athttps://gcc.gnu.org/ml/gcc-patches/2014-12/msg00309.html) > but did you have some concerns about backporting to other branches? So in gener

Re: [PATCH][AArch64] Use std::swap instead of manually swapping

2014-12-12 Thread Kyrill Tkachov
Ping. Marcus: Uros pointed out to me that these kinds of changes are considered obvious (with precedent athttps://gcc.gnu.org/ml/gcc-patches/2014-12/msg00309.html) but did you have some concerns about backporting to other branches? Kyrill On 05/12/14 16:40, Kyrill Tkachov wrote: Ping. https

Re: [PATCH][AArch64] Use std::swap instead of manually swapping

2014-12-05 Thread Kyrill Tkachov
Ping. https://gcc.gnu.org/ml/gcc-patches/2014-11/msg01426.html Thanks, Kyrill On 27/11/14 15:37, Kyrill Tkachov wrote: Ping. Thanks, Kyrill On 13/11/14 09:42, Kyrill Tkachov wrote: Hi all, Following the trend in i386 and alpha, this patch uses std::swap to perform swapping of values in the

Re: [PATCH][AArch64] Use std::swap instead of manually swapping

2014-11-27 Thread Kyrill Tkachov
Ping. Thanks, Kyrill On 13/11/14 09:42, Kyrill Tkachov wrote: Hi all, Following the trend in i386 and alpha, this patch uses std::swap to perform swapping of values in the aarch64 backend instead of declaring temporaries. Tested and bootstrapped on aarch64-linux. Ok for trunk? Thanks, Kyrill