This is the mail archive of the mailing list for the GCC project.

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[PATCH 3/9] reload_reg_free():

   In <URL:>, I described a
case where reload allocates a a register for a RELOAD_FOR_OPADDR_ADDR
and in doing so, picks a register inherited for a 
RELOAD_FOR_OPERAND_ADDRESS, thereby clobbering the register. This patch
fixes it by adding a test for this case.

   This and the other two reload patches in this patch set were bootstrapped
and tested together on x86_64-unknown-linux-gnu with no new failures. I also
built and tested cross compilers for arm-unknown-elf, cris-axis-elf,
m32c-unknown-elf, mipsisa64-unknown-elf, sh-unknown-elf and v850-unknown-elf
with no new failures.

  Ok for trunk?

2007-07-30  Rask Ingemann Lambertsen  <>

	* reload1.c (reload_reg_free_p): A register isn't free to use in
	  RELOAD_FOR_OPADDR_ADDR when already inherited for use in

Index: gcc/reload1.c
--- gcc/reload1.c	(revision 127048)
+++ gcc/reload1.c	(working copy)
@@ -4745,8 +4745,11 @@ reload_reg_free_p (unsigned int regno, i
       for (i = 0; i < reload_n_operands; i++)
 	if (TEST_HARD_REG_BIT (reload_reg_used_in_input[i], regno))
 	  return 0;
+      if (TEST_HARD_REG_BIT (reload_reg_used_for_inherit, regno)
+	  && TEST_HARD_REG_BIT (reload_reg_used_in_op_addr, regno))
+        return 0;
       return (!TEST_HARD_REG_BIT (reload_reg_used_in_op_addr_reload, regno));

Rask Ingemann Lambertsen

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]