[obvious] Use std::swap instead of manually swapping in a few more places
authorKyrylo Tkachov <kyrylo.tkachov@arm.com>
Fri, 24 Jul 2015 16:46:04 +0000 (16:46 +0000)
committerKyrylo Tkachov <ktkachov@gcc.gnu.org>
Fri, 24 Jul 2015 16:46:04 +0000 (16:46 +0000)
* alias.c (nonoverlapping_memrefs_p): Use std::swap instead of
manually swapping values.
* cse.c (fold_rtx): Likewise.
* lra-eliminations.c (form_sum): Likewise.

From-SVN: r226179

gcc/ChangeLog
gcc/alias.c
gcc/cse.c
gcc/lra-eliminations.c

index 9f2f87bdc217546d76682dda05d0ff0793388f9a..5a859634702d36e7d773d30082a174075814237a 100644 (file)
@@ -1,3 +1,10 @@
+2015-07-24  Kyrylo Tkachov  <kyrylo.tkachov@arm.com>
+
+       * alias.c (nonoverlapping_memrefs_p): Use std::swap instead of
+       manually swapping values.
+       * cse.c (fold_rtx): Likewise.
+       * lra-eliminations.c (form_sum): Likewise.
+
 2015-07-24  Uros Bizjak  <ubizjak@gmail.com>
 
        PR target/64003
index 69e37326a688a391ad3961f3d11768ad5add542f..4681e3f8b9618563be48d6962bfebffab8838439 100644 (file)
@@ -2466,7 +2466,7 @@ nonoverlapping_memrefs_p (const_rtx x, const_rtx y, bool loop_invariant)
   rtx basex, basey;
   bool moffsetx_known_p, moffsety_known_p;
   HOST_WIDE_INT moffsetx = 0, moffsety = 0;
-  HOST_WIDE_INT offsetx = 0, offsety = 0, sizex, sizey, tem;
+  HOST_WIDE_INT offsetx = 0, offsety = 0, sizex, sizey;
 
   /* Unless both have exprs, we can't tell anything.  */
   if (exprx == 0 || expry == 0)
@@ -2596,8 +2596,8 @@ nonoverlapping_memrefs_p (const_rtx x, const_rtx y, bool loop_invariant)
   /* Put the values of the memref with the lower offset in X's values.  */
   if (offsetx > offsety)
     {
-      tem = offsetx, offsetx = offsety, offsety = tem;
-      tem = sizex, sizex = sizey, sizey = tem;
+      std::swap (offsetx, offsety);
+      std::swap (sizex, sizey);
     }
 
   /* If we don't know the size of the lower-offset value, we can't tell
index 96adf18e6b7abfdd24690279cfa2155055bfaf3b..88c82fc8d74c1616d372b69b1263c172974b8b18 100644 (file)
--- a/gcc/cse.c
+++ b/gcc/cse.c
@@ -3297,9 +3297,8 @@ fold_rtx (rtx x, rtx_insn *insn)
         consistent with the order in X.  */
       if (canonicalize_change_group (insn, x))
        {
-         rtx tem;
-         tem = const_arg0, const_arg0 = const_arg1, const_arg1 = tem;
-         tem = folded_arg0, folded_arg0 = folded_arg1, folded_arg1 = tem;
+         std::swap (const_arg0, const_arg1);
+         std::swap (folded_arg0, folded_arg1);
        }
 
       apply_change_group ();
index c8da0c20deae7fb2952e7d38cbbfe5a102b7fdcb..fdf4179927ef4ded29a974c612d57b45c10ac0b8 100644 (file)
@@ -215,7 +215,6 @@ setup_elimination_map (void)
 static rtx
 form_sum (rtx x, rtx y)
 {
-  rtx tem;
   machine_mode mode = GET_MODE (x);
 
   if (mode == VOIDmode)
@@ -229,7 +228,7 @@ form_sum (rtx x, rtx y)
   else if (CONST_INT_P (y))
     return plus_constant (mode, x, INTVAL (y));
   else if (CONSTANT_P (x))
-    tem = x, x = y, y = tem;
+    std::swap (x, y);
 
   if (GET_CODE (x) == PLUS && CONSTANT_P (XEXP (x, 1)))
     return form_sum (XEXP (x, 0), form_sum (XEXP (x, 1), y));