diff options
| author | Kaz Kojima <kkojima@gcc.gnu.org> | 2015-01-08 09:05:06 +0000 |
|---|---|---|
| committer | Kaz Kojima <kkojima@gcc.gnu.org> | 2015-01-08 09:05:06 +0000 |
| commit | ae6b8666c957ea16bc44fd4d4d425b0db8d6bd85 (patch) | |
| tree | 23742a49cff41f90ee9cbce15326654149a49dbf | |
| parent | 946695b779e0ca8c6cb9eabe109465b6c0b599a8 (diff) | |
| download | gcc-ae6b8666c957ea16bc44fd4d4d425b0db8d6bd85.zip gcc-ae6b8666c957ea16bc44fd4d4d425b0db8d6bd85.tar.gz gcc-ae6b8666c957ea16bc44fd4d4d425b0db8d6bd85.tar.bz2 | |
re PR target/64533 ([SH] alloca generates unsafe code)
PR target/64533
* config/sh/sh.md (*addsi3_compact): Use u constraint instead
of r for the second alternative of the destination operand.
From-SVN: r219338
| -rw-r--r-- | gcc/ChangeLog | 6 | ||||
| -rw-r--r-- | gcc/config/sh/sh.md | 5 |
2 files changed, 9 insertions, 2 deletions
diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 5eb1bc6..55ffb7b 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2015-01-08 Kaz Kojima <kkojima@gcc.gnu.org> + + PR target/64533 + * config/sh/sh.md (*addsi3_compact): Use u constraint instead + of r for the second alternative of the destination operand. + 2015-01-07 Segher Boessenkool <segher@kernel.crashing.org> PR target/36557 diff --git a/gcc/config/sh/sh.md b/gcc/config/sh/sh.md index 37d2a20..6e0b97f 100644 --- a/gcc/config/sh/sh.md +++ b/gcc/config/sh/sh.md @@ -2061,9 +2061,10 @@ ;; The problem is that LRA expects something like ;; (set rA (plus rB (const_int N))) ;; to work. We can do that, but we have to split out an additional reg-reg -;; copy before the actual add insn. +;; copy before the actual add insn. Use u constraint for that case to avoid +;; the invalid value in the stack pointer. (define_insn_and_split "*addsi3_compact" - [(set (match_operand:SI 0 "arith_reg_dest" "=r,&r") + [(set (match_operand:SI 0 "arith_reg_dest" "=r,&u") (plus:SI (match_operand:SI 1 "arith_operand" "%0,r") (match_operand:SI 2 "arith_or_int_operand" "rI08,rn")))] "TARGET_SH1 |