[i386][obvious] Use std::swap instead of manually swapping in a couple of places
authorKyrylo Tkachov <kyrylo.tkachov@arm.com>
Fri, 19 Jun 2015 11:05:02 +0000 (11:05 +0000)
committerKyrylo Tkachov <ktkachov@gcc.gnu.org>
Fri, 19 Jun 2015 11:05:02 +0000 (11:05 +0000)
* config/i386/i386.c (ix86_function_versions): Use std::swap instead
of manually swapping.
(expand_vec_perm_interleave2): Likewise.

From-SVN: r224647

gcc/ChangeLog
gcc/config/i386/i386.c

index e567ed0db715eed89b8a84f778f5a8854d344977..bd9b856cdabd769529bd0cbc39dbd48bd6f18ac1 100644 (file)
@@ -1,3 +1,9 @@
+2015-06-19  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
+
+       * config/i386/i386.c (ix86_function_versions): Use std::swap instead
+       of manually swapping.
+       (expand_vec_perm_interleave2): Likewise.
+
 2015-06-19  Ilya Enkovich  <enkovich.gnu@gmail.com>
 
        * tree-chkp.c (chkp_compute_bounds_for_assignment): Don't
index bd548a969fb24c5334fd8f7c7896ce741257a3ce..24fccfca53d03b46bc665bd77770ecc19cda5111 100644 (file)
@@ -34971,9 +34971,7 @@ ix86_function_versions (tree fn1, tree fn2)
        {
          if (attr2 != NULL_TREE)
            {
-             tree tem = fn1;
-             fn1 = fn2;
-             fn2 = tem;
+             std::swap (fn1, fn2);
              attr1 = attr2;
            }
          error_at (DECL_SOURCE_LOCATION (fn2),
@@ -47990,9 +47988,7 @@ expand_vec_perm_interleave2 (struct expand_vec_perm_d *d)
            {
              /* Attempt to increase the likelihood that dfinal
                 shuffle will be intra-lane.  */
-             char tmph = nonzero_halves[0];
-             nonzero_halves[0] = nonzero_halves[1];
-             nonzero_halves[1] = tmph;
+             std::swap (nonzero_halves[0], nonzero_halves[1]);
            }
 
          /* vperm2f128 or vperm2i128.  */