[AArch64] Use std::swap instead of manually swapping
authorKyrylo Tkachov <kyrylo.tkachov@arm.com>
Wed, 4 Feb 2015 13:52:55 +0000 (13:52 +0000)
committerKyrylo Tkachov <ktkachov@gcc.gnu.org>
Wed, 4 Feb 2015 13:52:55 +0000 (13:52 +0000)
* config/aarch64/aarch64.c (aarch64_evpc_ext): Use std::swap instead
of manual swapping implementation.
(aarch64_expand_vec_perm_const_1): Likewise.

From-SVN: r220402

gcc/ChangeLog
gcc/config/aarch64/aarch64.c

index 116e20f5e5ff062472fa7261f1e4ac5789b14723..165f521f806bf51c36595e3e645a6e1f95b1d667 100644 (file)
@@ -1,3 +1,9 @@
+2015-02-04  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
+
+       * config/aarch64/aarch64.c (aarch64_evpc_ext): Use std::swap instead
+       of manual swapping implementation.
+       (aarch64_expand_vec_perm_const_1): Likewise.
+
 2015-02-04  James Greenhalgh <james.greenhalgh@arm.com>
 
        * config/aarch64/aarch64.c (NAMED_PARAM): Delete it.
index f082e05bee901b48351ce34d533adf082e95012c..c8e22d5e2c6ae16d495fc26eec3e96126a17febd 100644 (file)
@@ -9832,9 +9832,7 @@ aarch64_evpc_ext (struct expand_vec_perm_d *d)
       /* After setup, we want the high elements of the first vector (stored
          at the LSB end of the register), and the low elements of the second
          vector (stored at the MSB end of the register). So swap.  */
-      rtx temp = d->op0;
-      d->op0 = d->op1;
-      d->op1 = temp;
+      std::swap (d->op0, d->op1);
       /* location != 0 (above), so safe to assume (nelt - location) < nelt.  */
       location = nelt - location;
     }
@@ -10005,15 +10003,12 @@ aarch64_expand_vec_perm_const_1 (struct expand_vec_perm_d *d)
   if (d->perm[0] >= d->nelt)
     {
       unsigned i, nelt = d->nelt;
-      rtx x;
 
       gcc_assert (nelt == (nelt & -nelt));
       for (i = 0; i < nelt; ++i)
        d->perm[i] ^= nelt; /* Keep the same index, but in the other vector.  */
 
-      x = d->op0;
-      d->op0 = d->op1;
-      d->op1 = x;
+      std::swap (d->op0, d->op1);
     }
 
   if (TARGET_SIMD)