[PATCH] rs6000: r12 copy cleanup

Bill Schmidt wschmidt@linux.ibm.com
Fri Aug 28 16:48:41 GMT 2020


Remove unnecessary tests before copying function address to r12, as
requested by Segher.

Bootstrapped and tested on powerpc64le-unknown-linx-gnu with no
regressions, committed as obvious.

Thanks,
Bill


2020-08-28  Bill Schmidt  <wschmidt@linux.ibm.com>

gcc/
	* config/rs6000/rs6000.c (rs6000_call_aix): Remove test for r12.
	(rs6000_sibcall_aix): Likewise.
---
 gcc/config/rs6000/rs6000.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/gcc/config/rs6000/rs6000.c b/gcc/config/rs6000/rs6000.c
index 09545278dcf..ca5b71ecdd3 100644
--- a/gcc/config/rs6000/rs6000.c
+++ b/gcc/config/rs6000/rs6000.c
@@ -24725,8 +24725,7 @@ rs6000_call_aix (rtx value, rtx func_desc, rtx tlsarg, rtx cookie)
 	  /* A function pointer in the ELFv2 ABI is just a plain address, but
 	     the ABI requires it to be loaded into r12 before the call.  */
 	  func_addr = gen_rtx_REG (Pmode, 12);
-	  if (!rtx_equal_p (func_addr, func))
-	    emit_move_insn (func_addr, func);
+	  emit_move_insn (func_addr, func);
 	  abi_reg = func_addr;
 	  /* Indirect calls via CTR are strongly preferred over indirect
 	     calls via LR, so move the address there.  Needed to mark
@@ -24846,8 +24845,7 @@ rs6000_sibcall_aix (rtx value, rtx func_desc, rtx tlsarg, rtx cookie)
   if (GET_CODE (func_desc) != SYMBOL_REF && DEFAULT_ABI == ABI_ELFv2)
     {
       r12 = gen_rtx_REG (Pmode, 12);
-      if (!rtx_equal_p (r12, func_desc))
-	emit_move_insn (r12, func_desc);
+      emit_move_insn (r12, func_desc);
       func_addr = gen_rtx_REG (Pmode, CTR_REGNO);
       emit_move_insn (func_addr, r12);
     }
-- 
2.17.1



More information about the Gcc-patches mailing list