diff mbox series

[committed] hppa: Fix integer REG+D address reloads

Message ID ZV5ih3BNkEn-7As4@mx3210.localdomain
State Unresolved
Headers
Series [committed] hppa: Fix integer REG+D address reloads |

Checks

Context Check Description
snail/gcc-patch-check warning Git am fail log

Commit Message

John David Anglin Nov. 22, 2023, 8:20 p.m. UTC 
  Tested on hppa-unknown-linux-gnu and hppa64-hp-hpux11.11.  Fixes testcase
in PR.  Committed to trunk.

Dave
---

hppa: Fix integer REG+D address reloads

I made a mistake in the previous change to integer_store_memory_operand.
There is no support pa_emit_move sequence to handle secondary reloads of
integer REG+D instructions.  Further, the Q constraint is used for some
non-simple instructions (movb and addib).  Thus, we need to return true
when reload is in progress.

2023-11-22  John David Anglin  <danglin@gcc.gnu.org>

gcc/ChangeLog:

	PR target/112617
	* config/pa/predicates.md (integer_store_memory_operand): Return
	true for REG+D addresses when reload_in_progress is true.
diff mbox series

Patch

diff --git a/gcc/config/pa/predicates.md b/gcc/config/pa/predicates.md
index 1b50020e1de..4c07c0a3828 100644
--- a/gcc/config/pa/predicates.md
+++ b/gcc/config/pa/predicates.md
@@ -308,6 +308,13 @@ 
 
   if (reg_plus_base_memory_operand (op, mode))
     {
+      /* There is no support for handling secondary reloads of integer
+	 REG+D instructions in pa_emit_move_sequence.  Further, the Q
+	 constraint is used in more than simple move instructions.  So,
+	 we must return true and let reload handle the reload.  */
+      if (reload_in_progress)
+	return true;
+
       /* Extract CONST_INT operand.  */
       if (GET_CODE (op) == SUBREG)
 	op = SUBREG_REG (op);